Hi Thomas Gleixner, On 2023/12/6 01:02, Thomas Gleixner wrote: > On Mon, Dec 04 2023 at 13:31, Stefano Stabellini wrote: >> On Mon, 3 Dec 2023, Chen, Jiqian wrote: >>>>> vpci device state when device is reset on dom0 side. >>>>> >>>>> And call that function in pcistub_init_device. Because when >>>>> we use "pci-assignable-add" to assign a passthrough device in >>>>> Xen, it will reset passthrough device and the vpci state will >>>>> out of date, and then device will fail to restore bar state. >>>>> >>>>> Signed-off-by: Jiqian Chen <[email protected]> >>>>> Signed-off-by: Huang Rui <[email protected]> >>>> >>>> This Signed-off-by chain is incorrect. >>>> >>>> Documentation/process/submitting-patches.rst has a full chapter about >>>> S-O-B and the correct usage. >>> I am the author of this series of patches, and Huang Rui transported the v1 >>> to upstream. And now I transport v2. I am not aware that the SOB chain is >>> incorrect. >>> Do you have any suggestions? >> >> I think he means that your Signed-off-by should be the second one of the >> two as you are the one submitting the patch to the LKML > > No. > > Mailfrom: Jiqian Chen <[email protected]> > <body> > > Changelog-text > > Signed-off-by: Huang Rui <[email protected]> > Signed-off-by: Jiqian Chen <[email protected]> > > is equally wrong because that would end up with Chen as author and Huang > as first S-O-B which is required to be the author's S-O-B > > To make the above correct this would require: > > Mailfrom: Jiqian Chen <[email protected]> > <body> > > From: Huang Rui <[email protected]> > > Changelog-text > > Signed-off-by: Huang Rui <[email protected]> > Signed-off-by: Jiqian Chen <[email protected]> > > which tells that Huang is the author and Chen is the 'transporter', > which unfortunately does not reflect reality. > > Or: > > Mailfrom: Jiqian Chen <[email protected]> > <body> > > Changelog-text > > Co-developed-by: Huang Rui <[email protected]> > Signed-off-by: Huang Rui <[email protected]> > Signed-off-by: Jiqian Chen <[email protected]> > > which tells that Checn is the author and Huang co-developed the > patch, which might be true or not. > > > V1 which was sent by Huang has the ordering is correct: > > Mailfrom: Huang Rui <[email protected]> > <body> > > From: Jiqian Chen <[email protected]> > > Changelog-text > > Signed-off-by: Jiqian Chen <[email protected]> > Signed-off-by: Huang Rui <[email protected]> > > i.e. Chen authored and Huang transported > > Now this V2 has not really much to do with V1 and is a new > implementation to solve the problem, which was authored by Chen, so > Huang is not involved at all if I understand correctly. Not exactly, V2 is modified based on the V1. I am the author of this series. Huang Rui upstream the V1, and he also helped me to improve the first patch of this V2 series. Maybe in next version, below is more suitable:
Co-developed-by: Huang Rui <[email protected]> Signed-off-by: Jiqian Chen <[email protected]> Which can reflect that Huang Rui is co-developer, and I am the author and the last people to send patches. > > So what does his S-O-B mean here? Nothing... > > It's very well documented how the whole S-O-B business works and it's > not really rocket science to get it straight. > > It has a meaning and is not just for decoration purposes. > > Thanks, > > tglx -- Best regards, Jiqian Chen.
