Volunteering for stable patch review process

Volunteering for stable patch review process

[Luis R. Rodriguez]

I'm curious what the process is to join the group of volunteers to
review stable patches. Can't seem to find the docs for it. The only
excerpt I see is:

Review committee:

 - This is made up of a number of kernel developers who have volunteered for
   this task, and a few that haven't.

  Luis

Re: Volunteering for stable patch review process

[Chris Wright]

* Luis R. Rodriguez (mcgrof@gmail.com) wrote:
> I'm curious what the process is to join the group of volunteers to
> review stable patches. Can't seem to find the docs for it. The only
> excerpt I see is:
> 
> Review committee:
> 
>  - This is made up of a number of kernel developers who have volunteered for
>    this task, and a few that haven't.

Just send a note (you just did) and we'll put you on the review list
(I just did).

thanks,
-chris

Re: Volunteering for stable patch review process

[Greg KH]

On Mon, Jun 15, 2009 at 10:37:18PM -0700, Chris Wright wrote:
> * Luis R. Rodriguez (mcgrof@gmail.com) wrote:
> > I'm curious what the process is to join the group of volunteers to
> > review stable patches. Can't seem to find the docs for it. The only
> > excerpt I see is:
> > 
> > Review committee:
> > 
> >  - This is made up of a number of kernel developers who have volunteered for
> >    this task, and a few that haven't.
> 
> Just send a note (you just did) and we'll put you on the review list
> (I just did).

Also, the relevant text for how to join is in the 000/XXX message we
send out with every review, that contains the lines:

	These patches are sent out with a number of different people on the
	Cc: line.  If you wish to be a reviewer, please email stable@kernel.org
	to add your name to the list.  If you want to be off the reviewer list,
	also email us.

thanks,

greg k-h

Re: Volunteering for stable patch review process

[Luis R. Rodriguez]

On Mon, Jun 15, 2009 at 10:37 PM, Chris Wright<chrisw@sous-sol.org> wrote:
> * Luis R. Rodriguez (mcgrof@gmail.com) wrote:
>> I'm curious what the process is to join the group of volunteers to
>> review stable patches. Can't seem to find the docs for it. The only
>> excerpt I see is:
>>
>> Review committee:
>>
>>  - This is made up of a number of kernel developers who have volunteered for
>>    this task, and a few that haven't.
>
> Just send a note (you just did) and we'll put you on the review list
> (I just did).

OK thanks, I wasn't particularly looking to volunteer but to get info
as to how to do it. But since you've already added me, lets go through
with it -- I may at least be able to help review wireless stuff.

  Luis

Re: Volunteering for stable patch review process

[Stefan Bader]

Luis R. Rodriguez wrote:
> On Mon, Jun 15, 2009 at 10:37 PM, Chris Wright<chrisw@sous-sol.org> wrote:
>> * Luis R. Rodriguez (mcgrof@gmail.com) wrote:
>>> I'm curious what the process is to join the group of volunteers to
>>> review stable patches. Can't seem to find the docs for it. The only
>>> excerpt I see is:
>>>
>>> Review committee:
>>>
>>>  - This is made up of a number of kernel developers who have volunteered for
>>>    this task, and a few that haven't.
>> Just send a note (you just did) and we'll put you on the review list
>> (I just did).
> 
> OK thanks, I wasn't particularly looking to volunteer but to get info
> as to how to do it. But since you've already added me, lets go through
> with it -- I may at least be able to help review wireless stuff.
> 
>   Luis

Ok, thanks for checking out that, Luis. I'd be interested in getting in 
participating. Which was the part that made Luis ask. Could you add me too?

Thanks,
Stefan

-- 

When all other means of communication fail, try words!

Re: Volunteering for stable patch review process

[Greg KH]

On Tue, Jun 16, 2009 at 08:43:29PM +0200, Stefan Bader wrote:
> Luis R. Rodriguez wrote:
> > On Mon, Jun 15, 2009 at 10:37 PM, Chris Wright<chrisw@sous-sol.org> wrote:
> >> * Luis R. Rodriguez (mcgrof@gmail.com) wrote:
> >>> I'm curious what the process is to join the group of volunteers to
> >>> review stable patches. Can't seem to find the docs for it. The only
> >>> excerpt I see is:
> >>>
> >>> Review committee:
> >>>
> >>>  - This is made up of a number of kernel developers who have volunteered for
> >>>    this task, and a few that haven't.
> >> Just send a note (you just did) and we'll put you on the review list
> >> (I just did).
> > 
> > OK thanks, I wasn't particularly looking to volunteer but to get info
> > as to how to do it. But since you've already added me, lets go through
> > with it -- I may at least be able to help review wireless stuff.
> > 
> >   Luis
> 
> Ok, thanks for checking out that, Luis. I'd be interested in getting in 
> participating. Which was the part that made Luis ask. Could you add me too?

Now added.

Hm, we now have 17 people on the list.  This might be getting a bit
messy for some email systems to have such a large CC: list, perhaps we
should look into setting up an email list, like the stable-commits list
is used.

Or perhaps we should just use the stable-commits list instead?  Chris,
any thoughts?

thanks,

greg k-h

Re: Volunteering for stable patch review process

[Chris Wright]

* Greg KH (greg@kroah.com) wrote:
> Hm, we now have 17 people on the list.  This might be getting a bit
> messy for some email systems to have such a large CC: list, perhaps we
> should look into setting up an email list, like the stable-commits list
> is used.

Heh, was just thinking the same.  Yup, think a list would be better.

> Or perhaps we should just use the stable-commits list instead?  Chris,
> any thoughts?

stable-commits is good from my pov, but it's effectively read only.

thanks,
-chris

Re: Volunteering for stable patch review process

[Greg KH]

On Tue, Jun 16, 2009 at 12:00:03PM -0700, Chris Wright wrote:
> * Greg KH (greg@kroah.com) wrote:
> > Hm, we now have 17 people on the list.  This might be getting a bit
> > messy for some email systems to have such a large CC: list, perhaps we
> > should look into setting up an email list, like the stable-commits list
> > is used.
> 
> Heh, was just thinking the same.  Yup, think a list would be better.
> 
> > Or perhaps we should just use the stable-commits list instead?  Chris,
> > any thoughts?
> 
> stable-commits is good from my pov, but it's effectively read only.

Yeah, it's also higher volume which some reviewers might not like.

Should we create "stable-review" instead?

thanks,

greg k-h

Re: Volunteering for stable patch review process

[Chris Wright]

* Greg KH (greg@kroah.com) wrote:
> Should we create "stable-review" instead?

Yeah, I think so.  I can chase it down if you want.

thanks,
-chris