Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1508743 > unrolled thread

Re: taint/module: Clean up global and module taint flags handling

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: taint/module: Clean up global and module taint flags handling Rusty Russell <rusty@rustcorp.com.au> - 2016-10-26 03:40 +0200
    Re: taint/module: Clean up global and module taint flags handling Jiri Kosina <jikos@kernel.org> - 2016-10-26 10:20 +0200
      Re: taint/module: Clean up global and module taint flags handling Jessica Yu <jeyu@redhat.com> - 2016-10-27 01:20 +0200

#1508743 — Re: taint/module: Clean up global and module taint flags handling

Jiri Kosina <jikos@kernel.org> writes:
> On Fri, 23 Sep 2016, Jessica Yu wrote:
>
>> Hm, quick question, which tree would this patch go to? Though the
>> cleanup is for modules, there is an indirect cross-tree dependency
>> (taint_flag.module needs to be true for TAINT_LIVEPATCH for Josh's
>> patch to still work as intended). The least complicated thing to do
>> would be to just take this through the livepatch tree (with Rusty's
>> approval :-)), no?
>
> I don't want to be sneaking this behind Rusty's back, but he hasn't 
> reposnded so far.

I finally side-stepped this by appointing Jessica maintainer, thus her
Reviewed-by is sufficient for the module tree.

Lazy, huh?

Sorry for the delay,
Rusty.

> It's not vitally super-crucial to have this present in this very pull 
> request, so I am currently putting this on hold wrt. the upcoming merge 
> window pull request, and we'll then proceeed afterwards once Rusty 
> expressess his (n)ack. If this doesn't happen during the coming weeks, 
> I'll pick this up myself.
>
> Thanks,
>
> -- 
> Jiri Kosina
> SUSE Labs

[toc] | [next] | [standalone]

#1508923

On Wed, 26 Oct 2016, Rusty Russell wrote:

> >> Hm, quick question, which tree would this patch go to? Though the
> >> cleanup is for modules, there is an indirect cross-tree dependency
> >> (taint_flag.module needs to be true for TAINT_LIVEPATCH for Josh's
> >> patch to still work as intended). The least complicated thing to do
> >> would be to just take this through the livepatch tree (with Rusty's
> >> approval :-)), no?
> >
> > I don't want to be sneaking this behind Rusty's back, but he hasn't 
> > reposnded so far.
> 
> I finally side-stepped this by appointing Jessica maintainer, thus her
> Reviewed-by is sufficient for the module tree.

Awesome! :)

> Lazy, huh?

Laziness has always been the main power behind all the progress of the 
mankind :P

Jessica, do you want me to take the patch through livepatching tree still, 
or would you route it yourself now?

Thanks,

-- 
Jiri Kosina
SUSE Labs

[toc] | [prev] | [next] | [standalone]

#1509930

+++ Jiri Kosina [26/10/16 10:17 +0200]:
>On Wed, 26 Oct 2016, Rusty Russell wrote:
>
>> >> Hm, quick question, which tree would this patch go to? Though the
>> >> cleanup is for modules, there is an indirect cross-tree dependency
>> >> (taint_flag.module needs to be true for TAINT_LIVEPATCH for Josh's
>> >> patch to still work as intended). The least complicated thing to do
>> >> would be to just take this through the livepatch tree (with Rusty's
>> >> approval :-)), no?
>> >
>> > I don't want to be sneaking this behind Rusty's back, but he hasn't
>> > reposnded so far.
>>
>> I finally side-stepped this by appointing Jessica maintainer, thus her
>> Reviewed-by is sufficient for the module tree.
>
>Awesome! :)
>
>> Lazy, huh?
>
>Laziness has always been the main power behind all the progress of the
>mankind :P
>
>Jessica, do you want me to take the patch through livepatching tree still,
>or would you route it yourself now?

Hi Jiri,

Once I'm done fumbling with maintainership logistics, I will take this
patch through my tree. :-)

Thanks,
Jessica

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web