2022-04-07 20:34:13

by Christian Brauner

[permalink] [raw]
Subject: Re: [ovl] 30f9ef9479: BUG:kernel_NULL_pointer_dereference,address

On Thu, Apr 07, 2022 at 05:40:23PM +0800, kernel test robot wrote:
>
>
> Greeting,
>
> FYI, we noticed the following commit (built with gcc-11):
>
> commit: 30f9ef94795008e5146f69d2eb043922a512bf85 ("ovl: support idmapped layers")
> https://github.com/ammarfaizi2/linux-block brauner/linux/fs.idmapped.overlayfs.v3

That's an old branch. :)
Anything that has a *.v<idx> appended is basically an old version in my
tree. The base branch is always the branch name without the *.v<idx>
suffix.

Thanks!
Christian


2022-04-12 08:28:35

by kernel test robot

[permalink] [raw]
Subject: Re: [ovl] 30f9ef9479: BUG:kernel_NULL_pointer_dereference,address

hi, Christian,

On Thu, Apr 07, 2022 at 12:26:05PM +0200, Christian Brauner wrote:
> On Thu, Apr 07, 2022 at 05:40:23PM +0800, kernel test robot wrote:
> >
> >
> > Greeting,
> >
> > FYI, we noticed the following commit (built with gcc-11):
> >
> > commit: 30f9ef94795008e5146f69d2eb043922a512bf85 ("ovl: support idmapped layers")
> > https://github.com/ammarfaizi2/linux-block brauner/linux/fs.idmapped.overlayfs.v3
>
> That's an old branch. :)
> Anything that has a *.v<idx> appended is basically an old version in my
> tree. The base branch is always the branch name without the *.v<idx>
> suffix.

got it. so there is no need to test these *.v<idx> branches, whatever build,
boot or function, right? If so, we will disable it.

>
> Thanks!
> Christian

2022-04-13 10:45:45

by Christian Brauner

[permalink] [raw]
Subject: Re: [ovl] 30f9ef9479: BUG:kernel_NULL_pointer_dereference,address

On Mon, Apr 11, 2022 at 05:07:19PM +0800, Oliver Sang wrote:
> hi, Christian,
>
> On Thu, Apr 07, 2022 at 12:26:05PM +0200, Christian Brauner wrote:
> > On Thu, Apr 07, 2022 at 05:40:23PM +0800, kernel test robot wrote:
> > >
> > >
> > > Greeting,
> > >
> > > FYI, we noticed the following commit (built with gcc-11):
> > >
> > > commit: 30f9ef94795008e5146f69d2eb043922a512bf85 ("ovl: support idmapped layers")
> > > https://github.com/ammarfaizi2/linux-block brauner/linux/fs.idmapped.overlayfs.v3
> >
> > That's an old branch. :)
> > Anything that has a *.v<idx> appended is basically an old version in my
> > tree. The base branch is always the branch name without the *.v<idx>
> > suffix.
>
> got it. so there is no need to test these *.v<idx> branches, whatever build,
> boot or function, right? If so, we will disable it.

Yeah, I maintain *.v<idx> branches around. Each time I need to send a
new version of a patchset I do:

git checkout -b base.branch.v<idx> base.branch

and then continue working on

git checkout base.branch

for the new version. :)

I try to make sure to only push base.branch to git.kernel.org once I
know it builds fine. Before that I only push to other code-hosting
platforms. One of the reasons being that I try to avoid upsetting the
various build bots. :)