[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Ybr1MgpKlNlVHMPq@kroah.com>
Date: Thu, 16 Dec 2021 09:13:38 +0100
From: "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
To: "Rantala, Tommi T. (Nokia - FI/Espoo)" <tommi.t.rantala@...ia.com>
Cc: "stable@...r.kernel.org" <stable@...r.kernel.org>,
"fw@...len.de" <fw@...len.de>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: LTS "Warning: file conntrack_vrf.sh is not executable, correct
this."
On Thu, Dec 16, 2021 at 09:07:13AM +0100, gregkh@...uxfoundation.org wrote:
> On Thu, Dec 16, 2021 at 08:00:30AM +0000, Rantala, Tommi T. (Nokia - FI/Espoo) wrote:
> > Hi Greg,
> >
> > Incorrect 0644 in LTS for new selftest file, seen in 5.4.165 since
> > "selftests: netfilter: add a vrf+conntrack testcase" was added to LTS.
> > It's correct 755 in upstream, but somehow messed up in LTS :-(
>
> That is because quilt/git can not set modes of files.
>
> > linux-5.4.y
> > ===========
> > commit 8d3563ecbca3526fcc6639065c9fb11b2f234706
> > Author: Florian Westphal <fw@...len.de>
> > Date: Mon Oct 18 14:38:13 2021 +0200
> >
> > selftests: netfilter: add a vrf+conntrack testcase
> >
> > commit 33b8aad21ac175eba9577a73eb62b0aa141c241c upstream.
> >
> > [...]
> > diff --git a/tools/testing/selftests/netfilter/conntrack_vrf.sh b/tools/testing/selftests/netfilter/conntrack_vrf.sh
> > new file mode 100644 <----------------
> >
> >
> > linux-5.15.y
> > ============
> > commit cffab968e94e513bdcef25e11402243ce2919803
> > [...]
> > new file mode 100644 <----------------
> >
> >
> > upstream
> > ========
> > $ git show 33b8aad21ac175eba9577a73eb62b0aa141c241c
> >
> > commit 33b8aad21ac175eba9577a73eb62b0aa141c241c
> > Author: Florian Westphal <fw@...len.de>
> > Date: Mon Oct 18 14:38:13 2021 +0200
> >
> > selftests: netfilter: add a vrf+conntrack testcase
> >
> > [...]
> > diff --git a/tools/testing/selftests/netfilter/conntrack_vrf.sh b/tools/testing/selftests/netfilter/conntrack_vrf.sh
> > new file mode 100755 <-----------------------
> >
> >
> > -Tommi
> >
>
> What about the 5.10.y tree? Does it need this change as well?
To answer my own question, yes, it is an issue there too. I'll go do a
release right now with the mode changed.
thanks,
greg k-h
Powered by blists - more mailing lists