czw., 9 cze 2022 o 16:27 Dave Hansen <dave.hansen@xxxxxxxxx> napisał(a): > > On 6/9/22 04:03, Grzegorz Jaszczyk wrote: > > Co-developed-by: Peter Fang <peter.fang@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx> > > Signed-off-by: Peter Fang <peter.fang@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx> > > Co-developed-by: Tomasz Nowicki <tn@xxxxxxxxxxxx> > > Signed-off-by: Tomasz Nowicki <tn@xxxxxxxxxxxx> > > Signed-off-by: Zide Chen <zide.chen@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx> > > Co-developed-by: Grzegorz Jaszczyk <jaz@xxxxxxxxxxxx> > > Signed-off-by: Grzegorz Jaszczyk <jaz@xxxxxxxxxxxx> > > --- > > Documentation/virt/kvm/x86/hypercalls.rst | 7 +++++++ > > arch/x86/kvm/x86.c | 3 +++ > > drivers/acpi/x86/s2idle.c | 8 ++++++++ > > include/linux/suspend.h | 1 + > > include/uapi/linux/kvm_para.h | 1 + > > kernel/power/suspend.c | 4 ++++ > > 6 files changed, 24 insertions(+) > > What's the deal with these emails? > > zide.chen@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx > > I see a smattering of those in the git logs, but never for Intel folks. I've kept emails as they were in the original patch and I do not think I should change them. This is what Zide and Peter originally used. > > I'll also say that I'm a bit suspicious of a patch that includes 5 > authors for 24 lines of code. Did it really take five of you to write > 24 lines of code? This patch was built iteratively: original patch comes from Zide and Peter, I've squashed it with Tomasz later changes and reworked by myself for upstream. I didn't want to take credentials from any of the above so ended up with Zide as an author and 3 co-developers. Please let me know if that's an issue. Best regards, Grzegorz