linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).