From: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
To: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Cc: Salvatore Bonaccorso <carnil@debian.org>,
"Mohamed Abuelfotoh, Hazem" <abuehaze@amazon.com>,
kovalev@altlinux.org, Paulo Alcantara <pc@manguebit.com>,
"leonardo@schenkel.net" <leonardo@schenkel.net>,
"linux-cifs@vger.kernel.org" <linux-cifs@vger.kernel.org>,
"m.weissbach@info-gate.de" <m.weissbach@info-gate.de>,
"regressions@lists.linux.dev" <regressions@lists.linux.dev>,
"sairon@sairon.cz" <sairon@sairon.cz>,
"stable@vger.kernel.org" <stable@vger.kernel.org>,
Steve French <smfrench@gmail.com>,
Darren Kenny <darren.kenny@oracle.com>
Subject: Re: [REGRESSION 6.1.70] system calls with CIFS mounts failing with "Resource temporarily unavailable"
Date: Tue, 20 Feb 2024 21:28:26 +0100 [thread overview]
Message-ID: <2024022011-afraid-automated-4677@gregkh> (raw)
In-Reply-To: <61fde07c-f767-42b0-9bfa-ef915b28fb77@oracle.com>
On Tue, Feb 06, 2024 at 01:16:01PM +0530, Harshit Mogalapalli wrote:
> Hi Salvatore,
>
> Adding kovalev here(who backported it to 5.10.y)
>
> On 03/02/24 9:09 pm, Salvatore Bonaccorso wrote:
> > Hi,
> >
> > On Thu, Feb 01, 2024 at 12:58:28PM +0000, Mohamed Abuelfotoh, Hazem wrote:
> > >
> > > On 31/01/2024 17:19, Paulo Alcantara wrote:
> > > > Greg, could you please drop
> > > >
> > > > b3632baa5045 ("cifs: fix off-by-one in SMB2_query_info_init()")
> > > >
> > > > from v5.10.y as suggested by Salvatore?
> > > >
> > > > Thanks.
> > >
> > > Are we dropping b3632baa5045 ("cifs: fix off-by-one in
> > > SMB2_query_info_init()") from v5.10.y while keeping it on v5.15.y? if we are
> > > dropping it from v5.15.y as well then we should backport 06aa6eff7b smb3:
> > > Replace smb2pdu 1-element arrays with flex-arrays to v5.15.y I remember
> > > trying to backport this patch on v5.15.y but there were some merge conflicts
> > > there.
> > >
> > > 06aa6eff7b smb3: Replace smb2pdu 1-element arrays with flex-arrays
> >
> > While I'm not eligible to say what should be done, my understading is
> > that Greg probably would prefer to have the "backport 06aa6eff7b"
> > version. What we know is that having now both commits in the
> > stable-rc/linux-5.10.y queue breaks cifs and the backport variants
> > seens to work fine (Paulo Alcantara probably though can comment best).
> >
> Having both one-liner fix that I have sent and the above commit isn't
> correct.
>
> > As 06aa6eff7b smb3: Replace smb2pdu 1-element arrays with flex-arrays
> > was backportable to 5.10.y it should now work as well for the upper
> > one 5.15.y.
>
> Correct, I agree. I had to send one-liner fix as we have the
> backport("06aa6eff7b smb3: Replace smb2pdu 1-element arrays with
> flex-arrays") missing in 5.15.y and when I tried backporting it to 5.15.y I
> saw many conflicts.
>
> If we have backport for 5.15.y similar to 5.10.y we could ask greg to remove
> one liner fix from both 5.10.y and 5.15.y: https://git.kernel.org/pub/scm/linux/kernel/git/stable/stable-queue.git/tree/queue-5.10/cifs-fix-off-by-one-in-smb2_query_info_init.patch
Someone needs to tell me what to do, as I'm lost.
thanks,
greg k-h
next prev parent reply other threads:[~2024-02-20 20:28 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-15 14:22 [REGRESSION 6.1.70] system calls with CIFS mounts failing with "Resource temporarily unavailable" Mohamed Abuelfotoh, Hazem
2024-01-15 14:28 ` Mohamed Abuelfotoh, Hazem
2024-01-15 15:16 ` gregkh
2024-01-15 15:30 ` Mohamed Abuelfotoh, Hazem
2024-01-31 6:30 ` Salvatore Bonaccorso
2024-01-31 17:19 ` Paulo Alcantara
2024-02-01 12:58 ` Mohamed Abuelfotoh, Hazem
2024-02-03 15:39 ` Salvatore Bonaccorso
2024-02-06 7:46 ` Harshit Mogalapalli
2024-02-06 16:33 ` kovalev
2024-02-20 20:28 ` gregkh [this message]
2024-02-21 2:26 ` Harshit Mogalapalli
2024-02-21 11:09 ` gregkh
2024-01-16 10:23 ` Harshit Mogalapalli
2024-01-26 19:13 ` SeongJae Park
2024-02-23 6:14 ` Linux regression tracking #update (Thorsten Leemhuis)
2024-02-26 14:28 ` Mohamed Abuelfotoh, Hazem
2024-02-26 14:54 ` Linux regression tracking (Thorsten Leemhuis)
2024-02-26 14:55 ` gregkh
2024-02-26 22:54 ` Mohamed Abuelfotoh, Hazem
[not found] <8ad7c20e-0645-40f3-96e6-75257b4bd31a@schenkel.net>
2024-01-08 11:18 ` Jan Čermák
2024-01-08 14:13 ` Greg KH
2024-01-08 14:30 ` Jan Čermák
2024-01-08 14:34 ` Leonardo Brondani Schenkel
2024-01-08 14:52 ` Paulo Alcantara
2024-01-08 15:11 ` Greg Kroah-Hartman
2024-01-08 15:58 ` Paulo Alcantara
2024-01-30 22:43 ` Salvatore Bonaccorso
2024-01-30 22:49 ` Salvatore Bonaccorso
2024-02-20 20:27 ` Greg Kroah-Hartman
2024-02-20 21:25 ` Salvatore Bonaccorso
2024-02-21 11:09 ` Greg Kroah-Hartman
2024-02-22 23:00 ` Paulo Alcantara
2024-02-23 5:50 ` Salvatore Bonaccorso
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=2024022011-afraid-automated-4677@gregkh \
--to=gregkh@linuxfoundation.org \
--cc=abuehaze@amazon.com \
--cc=carnil@debian.org \
--cc=darren.kenny@oracle.com \
--cc=harshit.m.mogalapalli@oracle.com \
--cc=kovalev@altlinux.org \
--cc=leonardo@schenkel.net \
--cc=linux-cifs@vger.kernel.org \
--cc=m.weissbach@info-gate.de \
--cc=pc@manguebit.com \
--cc=regressions@lists.linux.dev \
--cc=sairon@sairon.cz \
--cc=smfrench@gmail.com \
--cc=stable@vger.kernel.org \
/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).