Linux CIFS filesystem development
 help / color / mirror / Atom feed
From: "Linux regression tracking (Thorsten Leemhuis)" <regressions@leemhuis.info>
To: Christian Heusel <christian@heusel.eu>,
	Andrew Paniakin <apanyaki@amazon.com>
Cc: pc@cjr.nz, stfrench@microsoft.com, sashal@kernel.org,
	pc@manguebit.com, regressions@lists.linux.dev,
	stable@vger.kernel.org, linux-cifs@vger.kernel.org,
	abuehaze@amazon.com, simbarb@amazon.com, benh@amazon.com,
	gregkh@linuxfoundation.org
Subject: Re: [REGRESSION][BISECTED][STABLE] Commit 60e3318e3e900 in stable/linux-6.1.y breaks cifs client failover to another server in DFS namespace
Date: Thu, 11 Jul 2024 11:49:10 +0200	[thread overview]
Message-ID: <7c8d1ec1-7913-45ff-b7e2-ea58d2f04857@leemhuis.info> (raw)
In-Reply-To: <a58625e7-8245-4963-b589-ad69621cb48a@heusel.eu>

On 27.06.24 22:16, Christian Heusel wrote:
> On 24/06/26 03:09PM, Andrew Paniakin wrote:
>> On 25/06/2024, Christian Heusel wrote:
>>> On 24/06/24 10:59AM, Andrew Paniakin wrote:
>>>> On 19/06/2024, Andrew Paniakin wrote:
>>>>> Commit 60e3318e3e900 ("cifs: use fs_context for automounts") was
>>>>> released in v6.1.54 and broke the failover when one of the servers
>>>>> inside DFS becomes unavailable.
>>>> Friendly reminder, did anyone had a chance to look into this report?
>>>
>>> If I understand the report correctly the regression is specific for the
>>> current 6.1.y stable series, so also not much the CIFS devs themselves
>>> can do. Maybe the stable team missed the report with the plethora of
>>> mail that they get.. I'll change the subject to make this more prominent
>>> for them.
>>>
>>> I think a good next step would be to bisect to the commit that fixed the
>>> relevant issue somewhere between v6.1.54..v6.2-rc1 so the stable team
>>> knows what needs backporting .. You can do that somewhat like so[0]:
>>
>> Bisection showed that 7ad54b98fc1f ("cifs: use origin fullpath for
>> automounts") is a first good commit. Applying it on top of 6.1.94 fixed
>> the reported problem. It also passed Amazon Linux kernel regression
>> tests when applied on top of our latest kernel 6.1. Since the code in
>> 6.1.92 is a bit different I updated the original patch:
> 
> I think it might make sense to send the backported version of the patch
> for inclusion to the stable tree directly (see "Option 3" [here][0]).
> 
> [0]: https://www.kernel.org/doc/html/next/process/stable-kernel-rules.html#option-3

Hmmm, unless I'm missing something it seems nobody did so. Andrew, could
you take care of that to get this properly fixed to prevent others from
running into the same problem?

Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)
--
Everything you wanna know about Linux kernel regression tracking:
https://linux-regtracking.leemhuis.info/about/#tldr
If I did something stupid, please tell me, as explained on that page.

#regzbot poke

  reply	other threads:[~2024-07-11  9:49 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-19 18:32 [REGRESSION][BISECTED] Commit 60e3318e3e900 in stable/linux-6.1.y breaks cifs client failover to another server in DFS namespace Andrew Paniakin
2024-06-24 17:59 ` Andrew Paniakin
2024-06-25 11:07   ` [REGRESSION][BISECTED][STABLE] " Christian Heusel
2024-06-26 22:09     ` Andrew Paniakin
2024-06-27 20:16       ` Christian Heusel
2024-07-11  9:49         ` Linux regression tracking (Thorsten Leemhuis) [this message]
2024-07-13  3:22           ` Andrew Paniakin
2024-07-23  0:51             ` Andrew Paniakin
2024-09-27 10:56               ` Linux regression tracking (Thorsten Leemhuis)
2024-10-16 20:49                 ` Andrew Paniakin
2024-11-10 17:59                   ` Andrew Paniakin
2025-03-16 18:34                     ` Andrew Paniakin
2025-05-10  3:05                       ` Andrew Paniakin

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=7c8d1ec1-7913-45ff-b7e2-ea58d2f04857@leemhuis.info \
    --to=regressions@leemhuis.info \
    --cc=abuehaze@amazon.com \
    --cc=apanyaki@amazon.com \
    --cc=benh@amazon.com \
    --cc=christian@heusel.eu \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-cifs@vger.kernel.org \
    --cc=pc@cjr.nz \
    --cc=pc@manguebit.com \
    --cc=regressions@lists.linux.dev \
    --cc=sashal@kernel.org \
    --cc=simbarb@amazon.com \
    --cc=stable@vger.kernel.org \
    --cc=stfrench@microsoft.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