From mboxrd@z Thu Jan  1 00:00:00 1970
From: Borislav Petkov <bp@alien8.de>
Subject: Re: [PATCH] x86: svm: use kvm_register_write()/read()
Date: Fri, 20 Feb 2015 21:54:29 +0100
Message-ID: <20150220205429.GC19589@pd.tnic>
References: <20150220183939.16805.54776.stgit@joelvmguard2.amd.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Cc: Gleb Natapov <gleb@kernel.org>,
	Paolo Bonzini <pbonzini@redhat.com>, kvm@vger.kernel.org,
	Joerg Roedel <joro@8bytes.org>, linux-kernel@vger.kernel.org
To: Joel Schopp <joel.schopp@amd.com>
Return-path: <linux-kernel-owner@vger.kernel.org>
Content-Disposition: inline
In-Reply-To: <20150220183939.16805.54776.stgit@joelvmguard2.amd.com>
Sender: linux-kernel-owner@vger.kernel.org
List-Id: kvm.vger.kernel.org

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.kaplan@amd.com>
> Signed-off-by:Joel Schopp <joel.schopp@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.kaplan@amd.com>
[ Did this and that to patch. ]
Signed-off-by: Joel Schopp <joel.schopp@amd.com>

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--