Re: [PATCH 1/3] mips/kvm: Improve code formatting inarch/mips/kvm/kvm_locore.S

From: Gleb Natapov
Date: Mon Aug 05 2013 - 13:22:30 EST


On Mon, Aug 05, 2013 at 07:06:10PM +0200, Ralf Baechle wrote:
> On Mon, Aug 05, 2013 at 04:43:27PM +0300, Gleb Natapov wrote:
> > Date: Mon, 5 Aug 2013 16:43:27 +0300
> > From: Gleb Natapov <gleb@xxxxxxxxxx>
> > To: Ralf Baechle <ralf@xxxxxxxxxxxxxx>
> > Cc: James Hogan <james.hogan@xxxxxxxxxx>, David Daney
> > <ddaney.cavm@xxxxxxxxx>, linux-mips@xxxxxxxxxxxxxx, kvm@xxxxxxxxxxxxxxx,
> > Sanjay Lal <sanjayl@xxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, David
> > Daney <david.daney@xxxxxxxxxx>
> > Subject: Re: [PATCH 1/3] mips/kvm: Improve code formatting in
> > arch/mips/kvm/kvm_locore.S
> > Content-Type: text/plain; charset=us-ascii
> >
> > On Mon, Aug 05, 2013 at 03:21:57PM +0200, Ralf Baechle wrote:
> > > On Mon, Aug 05, 2013 at 02:17:01PM +0100, James Hogan wrote:
> > >
> > > >
> > > > On 01/08/13 21:22, David Daney wrote:
> > > > > From: David Daney <david.daney@xxxxxxxxxx>
> > > > >
> > > > > No code changes, just reflowing some comments and consistently using
> > > > > tabs and spaces. Object code is verified to be unchanged.
> > > > >
> > > > > Signed-off-by: David Daney <david.daney@xxxxxxxxxx>
> > > > > Acked-by: Ralf Baechle <ralf@xxxxxxxxxxxxxx>
> > > >
> > > >
> > > > > + /* Put the saved pointer to vcpu (s1) back into the DDATA_LO Register */
> > > >
> > > > git am detects a whitespace error here ("space before tab in indent").
> > > > It's got spaces before and after the tab actually.
> > > >
> > > > > /* load the guest context from VCPU and return */
> > > >
> > > > this comment could have it's indentation fixed too
> > > >
> > > > Otherwise, for all 3 patches:
> > > >
> > > > Reviewed-by: James Hogan <james.hogan@xxxxxxxxxx>
> > >
> > > I'm happy with the patch series as well and will fix this issue when
> > > applying the patch.
> > >
> > kvm fixes usually go through kvm.git tree for all arches. Any special
> > reasons you want to get those through mips tree?
>
> MIPS fixes usually go through the MIPS tree ;-)
>
arch/*/kvm/ fixes usually go through the kvm.git though :) KVM arch
code, after it is reasonably stable, usually depends more on kvm common
code then arch code and kvm development suppose to happen against
kvm.git otherwise APIs can go out of sync. I need to get acks of MIPS
people before taking patches of course.

When patch series touches code outside of arch/*/kvm, like David says
the next one will, it make sense to merge it through MIPS tree, just
please take KVM maintainers ACK for kvm part.

> I don't care which tree this stuff goes through - but a general experience
> is that things that affect MIPS systems receive most testing if going
> through the MIPS tree.
>
> Ralf

--
Gleb.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/