[PATCH 2/2] ARM: mm: mm->context.id fix for big-endian

Will Deacon will.deacon at arm.com
Mon Feb 11 05:51:38 EST 2013


On Mon, Feb 11, 2013 at 10:38:29AM +0000, Ben Dooks wrote:
> On 10/02/13 23:29, Russell King - ARM Linux wrote:
> > On Sun, Feb 10, 2013 at 03:05:11PM +0000, Ben Dooks wrote:
> >> On 10/02/2013 14:57, Ben Dooks wrote:
> >>> Since the new ASID code in [b5466f8728527a05a493cc4abe9e6f034a1bbaab]
> >>> was changed to use 64bit operations it has broken the BE operation due
> >>> to an issue with the MM code accessing sub-fields of mm->context.id.
> >>
> >> Fixed the ref, new patch pushed to git.
> >
> > No you haven't.
> >
> > "Since the new ASID code in b5466f872 (ARM: mm: remove IPI broadcasting
> > on ASID rollover)..."
> >
> > is the correct way to refer to another patch by commit ID - both the ID
> > _and_ its summary line.  You can use a short ID but it's a good idea to
> > add a few additional characters to avoid future hash clashes.
> 
> My repo is showing:
> 
>   Since the new ASID code in b5466f8728527a05a493cc4abe9e6f034a1bbaab
>   ("ARM: mm: remove IPI broadcasting on ASID rollover") was changed to
>   use 64bit operations it has broken the BE operation due to an issue
>   with the MM code accessing sub-fields of mm->context.id.
> 
> Would you like the commit-id shortened?
> Is there anything else before this pair can be pulled?

You can add my acked-by if you like:

  Acked-by: Will Deacon <will.deacon at arm.com>

Given that these are two small patches without any dependencies, it's
probably easier to stick them in the patch system (which will also resolve
any confusion about the form of the final commits).

Cheers,

Will



More information about the linux-arm-kernel mailing list