Date:   Fri, 17 Dec 2021 07:38:17 +0100
From:   "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
To:     "Rantala, Tommi T. (Nokia - FI/Espoo)" <tommi.t.rantala@nokia.com>
Cc:     "stable@vger.kernel.org" <stable@vger.kernel.org>,
        "fw@strlen.de" <fw@strlen.de>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: LTS "Warning: file conntrack_vrf.sh is not executable, correct
 this."
Message-ID: <YbwwWXH4RS+cPn+0@kroah.com>
References: <234d7a6a81664610fdf21ac72730f8bd10d3f46f.camel@nokia.com>
 <YbrzsZf/arqfZ3DT@kroah.com>
 <Ybr1MgpKlNlVHMPq@kroah.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Ybr1MgpKlNlVHMPq@kroah.com>
Precedence: bulk

On Thu, Dec 16, 2021 at 09:13:38AM +0100, gregkh@linuxfoundation.org wrote:
> On Thu, Dec 16, 2021 at 09:07:13AM +0100, gregkh@linuxfoundation.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@strlen.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@strlen.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.

This should now be fixed in the 3 kernel releases that went out
yesterday.  Please let me know if you continue to see this problem.

thanks,

greg k-h