Re: [PATCH] x86: svm: use kvm_register_write()/read()

2015-02-20 Thread Borislav Petkov
On Fri, Feb 20, 2015 at 12:39:40PM -0600, Joel Schopp wrote:
 KVM has nice wrappers to access the register values, clean up a few places
 that should use them but currently do not.
 
 Signed-off-by:David Kaplan david.kap...@amd.com
 Signed-off-by:Joel Schopp joel.sch...@amd.com

This SOB chain looks strange. If David is the author, you want to have
him in From: at the beginning of the patch.

Stuff you did ontop should be in []-braces before your SOB, like this:

Signed-off-by: David Kaplan david.kap...@amd.com
[ Did this and that to patch. ]
Signed-off-by: Joel Schopp joel.sch...@amd.com

-- 
Regards/Gruss,
Boris.

ECO tip #101: Trim your mails when you reply.
--
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] x86: svm: use kvm_register_write()/read()

2015-02-20 Thread Joel Schopp

On 02/20/2015 02:54 PM, Borislav Petkov wrote:
 On Fri, Feb 20, 2015 at 12:39:40PM -0600, Joel Schopp wrote:
 KVM has nice wrappers to access the register values, clean up a few places
 that should use them but currently do not.

 Signed-off-by:David Kaplan david.kap...@amd.com
 Signed-off-by:Joel Schopp joel.sch...@amd.com
 This SOB chain looks strange. If David is the author, you want to have
 him in From: at the beginning of the patch.

 Stuff you did ontop should be in []-braces before your SOB, like this:
Will resend with From: line and braces for clarification.


 Signed-off-by: David Kaplan david.kap...@amd.com
 [ Did this and that to patch. ]
 Signed-off-by: Joel Schopp joel.sch...@amd.com


--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html