From: Jingbo Xu <jefflexu@linux.alibaba.com>
To: Zhao Chen <winters.zc@antgroup.com>,
linux-fsdevel@vger.kernel.org, Miklos Szeredi <miklos@szeredi.hu>
Subject: Re: [PATCH v4 1/2] fuse: Introduce a new notification type for resend pending requests
Date: Thu, 25 Jan 2024 19:43:05 +0800 [thread overview]
Message-ID: <1611a0cc-6daa-4423-be27-eccca71e2fce@linux.alibaba.com> (raw)
In-Reply-To: <20240109092443.519460-2-winters.zc@antgroup.com>
Hi, Miklos,
On 1/9/24 5:24 PM, Zhao Chen wrote:
> When a FUSE daemon panics and failover, we aim to minimize the impact on
> applications by reusing the existing FUSE connection. During this process,
> another daemon is employed to preserve the FUSE connection's file
> descriptor. The new started FUSE Daemon will takeover the fd and continue
> to provide service.
>
> However, it is possible for some inflight requests to be lost and never
> returned. As a result, applications awaiting replies would become stuck
> forever. To address this, we can resend these pending requests to the
> new started FUSE daemon.
>
> This patch introduces a new notification type "FUSE_NOTIFY_RESEND", which
> can trigger resending of the pending requests, ensuring they are properly
> processed again.
>
> Signed-off-by: Zhao Chen <winters.zc@antgroup.com>
The notification style is more helpful in the cloud native scenarios
comparing to the original fusectl mechanism. Recent days we found the
original fusectl based "resend" interface is un-accessible inside a
non-privileged container in our internal use case, as fusectl is
designed as a singleton and has no ability for user namespace mounting.
The new notification styled interface seems extremely helpful in our case.
Thus may I ask if there is any potential modification as for the user
ABI of the resend feature? I'd like to test and improve it if there's any.
Many thanks!
--
Thanks,
Jingbo
next prev parent reply other threads:[~2024-01-25 11:43 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-09 9:24 [PATCH v4 0/2] fuse: Add support for resend pending requests Zhao Chen
2024-01-09 9:24 ` [PATCH v4 1/2] fuse: Introduce a new notification type " Zhao Chen
2024-01-25 11:43 ` Jingbo Xu [this message]
2024-01-09 9:24 ` [PATCH v4 2/2] fuse: Use the high bit of request ID for indicating resend requests Zhao Chen
2024-03-05 15:29 ` [PATCH v4 0/2] fuse: Add support for resend pending requests Miklos Szeredi
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=1611a0cc-6daa-4423-be27-eccca71e2fce@linux.alibaba.com \
--to=jefflexu@linux.alibaba.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=miklos@szeredi.hu \
--cc=winters.zc@antgroup.com \
/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;
as well as URLs for NNTP newsgroup(s).