mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Haifeng Xu <haifeng.xu@shopee.com>
To: Bernd Schubert <bernd.schubert@fastmail.fm>,
	Christian Brauner <brauner@kernel.org>,
	Miklos Szeredi <miklos@szeredi.hu>
Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [RFC] fuse: do not generate interrupt requests for fatal signals
Date: Fri, 21 Jun 2024 10:36:37 +0800	[thread overview]
Message-ID: <03658a84-09e0-40a1-aa30-9f92e82a6b0d@shopee.com> (raw)
In-Reply-To: <db4ed4e5-7c23-468d-8bac-cee215ace19e@fastmail.fm>



On 2024/6/21 05:40, Bernd Schubert wrote:
> 
> 
> On 6/20/24 08:43, Haifeng Xu wrote:
>>
>>
>> On 2024/6/17 15:25, Christian Brauner wrote:
>>> On Fri, Jun 14, 2024 at 12:01:39PM GMT, Miklos Szeredi wrote:
>>>> On Thu, 13 Jun 2024 at 12:44, Haifeng Xu <haifeng.xu@shopee.com> wrote:
>>>>
>>>>> So why the client doesn't get woken up?
>>>>
>>>> Need to find out what the server (lxcfs) is doing.  Can you do a
>>>> strace of lxcfs to see the communication on the fuse device?
>>>
>>> Fwiw, I'm one of the orignal authors and maintainers of LXCFS so if you
>>> have specific questions, I may be able to help.
>>
>> Thanks. All server threads of lcxfs wokrs fine now.
>>
>> So can we add another interface to abort those dead request?
>> If the client thread got killed and wait for relpy, but the fuse sever didn't 
>> send reply for some unknown reason,we can use this interface to wakeup the client thread.
> 
> Isn't that a manual workaround? I.e. an admin or a script needs to trigger it?

Yes.

> 
> There is a discussion in this thread to add request timeouts
> https://urldefense.proofpoint.com/v2/url?u=https-3A__lore.kernel.org_linux-2Dkernel_20240605153552.GB21567-40localhost.localdomain_T_&d=DwIDaQ&c=R1GFtfTqKXCFH-lgEPXWwic6stQkW4U7uVq33mt-crw&r=3uoFsejk1jN2oga47MZfph01lLGODc93n4Zqe7b0NRk&m=8O09nPSMPRZHOnfDnsm3lTwcO7AV93meeZP-F_k_u8w7XO04ISrP36bbcoEMUSrW&s=FRDpgmP8jGWJnoZna3OrFnvx44cCgywsGOeMY3fCeFc&e= 
> I guess for interrupted requests that would be definitely a case where timeouts could be
> applied?

Yes. If the requset can be cancelled until the timeout elapsed, we don't need to abort the dead requests manually.

Thanks!

> 
> 
> Thanks,
> Bernd

  reply	other threads:[~2024-06-21  2:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-13  4:01 Haifeng Xu
2024-06-13  7:55 ` Miklos Szeredi
2024-06-13 10:44   ` Haifeng Xu
2024-06-14 10:01     ` Miklos Szeredi
2024-06-15 12:19       ` Haifeng Xu
2024-08-21 17:38         ` Bernd Schubert
2024-08-27  8:46           ` Haifeng Xu
2024-06-17  7:25       ` Christian Brauner
2024-06-20  6:43         ` Haifeng Xu
2024-06-20 21:40           ` Bernd Schubert
2024-06-21  2:36             ` Haifeng Xu [this message]
2024-06-14 10:31   ` Bernd Schubert
2024-06-15 12:20     ` Haifeng Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=03658a84-09e0-40a1-aa30-9f92e82a6b0d@shopee.com \
    --to=haifeng.xu@shopee.com \
    --cc=bernd.schubert@fastmail.fm \
    --cc=brauner@kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miklos@szeredi.hu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®