On Tue, Jun 14, 2016 at 08:52:12AM +0200, Cédric Le Goater wrote: > On 06/14/2016 08:15 AM, David Gibson wrote: > > On Mon, Jun 13, 2016 at 07:24:48AM +0200, Cédric Le Goater wrote: > >> From: Benjamin Herrenschmidt <b...@kernel.crashing.org> > >> > >> And move the code adjusting the MSR mask and calling kvmppc_set_papr() > >> to it. This allows us to add a few more things such as disabling setting > >> of MSR:HV and appropriate LPCR bits which will be used when fixing > >> the exception model. > >> > >> Signed-off-by: Benjamin Herrenschmidt <b...@kernel.crashing.org> > >> Reviewed-by: David Gibson <da...@gibson.dropbear.id.au> > >> [clg: previous commit 26a7f1291bb5 did not include the LPCR setting as > >> it was not needed at the time ] > > > > I see how this came about, but it means the commit message (both long > > and short) is confusingly mismatched from the code now. > > OK. I will work on it. > > For my education, how much can we change the initial changelog of a patch ? > Is it considered as part of the code ?
I'm not entirely sure I follow the question, but AFAIK it's entirely
ok to reword and clarify a commit message as it moves through the
signed-off-by chain.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
