[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] Questions related to the analysis of the Xen mailing lists



Daniel,

I am assuming it is OK to post this one to the devel-list, to get extra answers.

> On 7 Sep 2015, at 16:19, Daniel Izquierdo <dizquierdo@xxxxxxxxxxxx> wrote:
> 
> Hi Lars,
> 
> I promise this is the last email ;).
> 
> We've found that there are several flags launched by different
> developers depending on the status of the patchset.
> 
> By 'flag' I mean comments like 'Signed-off-by: xxx@xxx'.
> 
> However, I think we do not fully understand all of them. I'm listing all
> of them below adding some comments. Please, would you mind double
> checking this and adding others that we may have missed? Thanks in advance!.
> 
> * Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>: This is the author of
> the patch

Correct
See https://www.kernel.org/doc/Documentation/SubmittingPatches section 11 - we 
have exactly the same requirement in Xen

> * Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> : This is the
> reviewer of the patch

Correct
Although the definition is more loose compared to 
https://www.kernel.org/doc/Documentation/SubmittingPatches

> * Acked-by: Roger Pau Monnà <roger.pau@xxxxxxxxxx>: Is this another
> reviewer of the patch, but with more privileges?

Correct
Although the definition is more loose compared to 
https://www.kernel.org/doc/Documentation/SubmittingPatches

One of the things I would like to find out, is whether we do have a lot of 
cases where there are
a) comments
b) changes
for a patch after it was marked Acked-by and whether there are any frequent 
patterns.  

> * Release-acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>: Is this the developer
> in charge of merging the code into master?

That means the release manager signals that this particular patch can be 
committed. It is normally only relevant during times of a feature freeze. In 
other words, a patch can be acked (because it's technical sound), but it might 
not be appropriate to be committed (too much risk vs too little gain). 

I think you should just ignore that tag when counting contributions. But it may 
be worthwhile including it in the model.

> * Reported-by: Roger Pau Monnà <roger.pau@xxxxxxxxxx>: Is this the
> developer that reported the issue?

See https://www.kernel.org/doc/Documentation/SubmittingPatches - section 13. 
Use is informal right now. We don't require it to be used.

> 
> * Tested-by: Roger Pau Monnà <roger.pau@xxxxxxxxxx>: I guess this is a
> tester, but this does not take place that often.

See https://www.kernel.org/doc/Documentation/SubmittingPatches - section 13. 
Use is informal right now. We don't require it to be used.

> 
> Then, we have also noticed some variations such as:
> 
> * Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, with Konrads
> suggestion fixed.

This is common practice when something out of the ordinary happens and is a way 
to highlight it 

> Are you aware of extra 'flags'?

All the ones in https://www.kernel.org/doc/Documentation/SubmittingPatches can 
turn up occasionally, such as suggested-by:, fixes:, ... but these are rare

> In addition to this, we have also noticed several ways to identify a
> patch serie, but pretty similar:
> 
> * [PATCH v0 2/15]
> * [PATCH v0 02/15]
> * [PATCH v1 2 of 15]

I think probably 90% will be covered by the canonical form, with some slight 
variants. Another common variant is [RFC PATCH ...] with the RFC being dropped 
eventually.

Is there usually enough meta-information to map all the patches into a series?

> Given that this process is becoming with some probability more and more
> automated, we're in first place retrieving info for the last year, and
> then expanding the analysis for the rest of the previous years. This
> should help to understand the issue nowadays and then easily escalate
> the problem.
> 
> Regards,
> Daniel.
> 
> 
> 
> --
> Daniel Izquierdo Cortazar, PhD
> Chief Data Officer
> ---------
> "Software Analytics for your peace of mind"
> www.bitergia.com
> @bitergia
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.