Re: [RESEND PATCH] kvm/selftests: Close opened file descriptor in stable_tsc_check_supported()

From: Hao Ge
Date: Thu Apr 06 2023 - 04:08:18 EST


Sorry for bother yours, the computer has updated some configurations,

causing an issue with the email name,so correct it.


On 4/6/23 10:56, gehao@xxxxxxxxxx wrote:
在 4/5/23 22:46, Sean Christopherson 写道:

This is not a RESEND, it is a new version.  From Documentation/process/submitting-patches.rst:

   Don't add "RESEND" when you are submitting a modified version of your
   patch or patch series - "RESEND" only applies to resubmission of a
   patch or patch series which have not been modified in any way from the
   previous submission.

And the "in any way" really does mean in _any_ way.  E.g. if a patch is rebased,
the version needs to be bumped.  RESEND should only ever be used when sending
literally the same patch/email file, e.g. if something went awry in the delivery
of the email, or you forgot to Cc the right people, tec.

On Wed, Apr 05, 2023, Hao Ge wrote:
Close the "current_clocksource" file descriptor before
Wrap closer to ~75 chars, wrapping at ~55 is too aggressive.

returning or exiting from stable_tsc_check_supported()
in vmx_nested_tsc_scaling_test

Vipin provided his Reviewed-by, that should have been captured here.  Please read
through Documentation/process/submitting-patches.rst, guidance on "using" the
various tags is also provided there.  And if you have time, pretty much all of
Documentation/process/ is worth reading.

No need to send a new version, all of this is easy to fixup when applying.

Thank you for your guidance,this has been very helpful to me for submitting

patches.


Thanks

Best Regards

Hao