lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190917191401.GA4721@agluck-desk2.amr.corp.intel.com>
Date:   Tue, 17 Sep 2019 12:14:01 -0700
From:   "Luck, Tony" <tony.luck@...el.com>
To:     David Laight <David.Laight@...lab.com>
Cc:     Thomas Gleixner <tglx@...utronix.de>,
        Fenghua Yu <fenghua.yu@...el.com>,
        Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>,
        H Peter Anvin <hpa@...or.com>,
        Peter Zijlstra <peterz@...radead.org>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Dave Hansen <dave.hansen@...el.com>,
        Paolo Bonzini <pbonzini@...hat.com>,
        Radim Krcmar <rkrcmar@...hat.com>,
        Sai Praneeth Prakhya <sai.praneeth.prakhya@...el.com>,
        Ravi V Shankar <ravi.v.shankar@...el.com>,
        linux-kernel <linux-kernel@...r.kernel.org>, x86 <x86@...nel.org>
Subject: Re: [PATCH 3/3] x86/split_lock: Align the x86_capability array to
 size of unsigned long

On Tue, Sep 17, 2019 at 08:29:28AM +0000, David Laight wrote:
> From: Tony Luck
> > Sent: 16 September 2019 23:40
> > From: Fenghua Yu <fenghua.yu@...el.com>
> > 
> > The x86_capability array in cpuinfo_x86 is defined as u32 and thus is
> > naturally aligned to 4 bytes. But, set_bit() and clear_bit() require
> > the array to be aligned to size of unsigned long (i.e. 8 bytes in
> > 64-bit).
> > 
> > To fix the alignment issue, align the x86_capability array to size of
> > unsigned long by using unnamed union and 'unsigned long array_align'
> > to force the alignment.
> > 
> > Changing the x86_capability array's type to unsigned long may also fix
> > the issue because the x86_capability array will be naturally aligned
> > to size of unsigned long. But this needs additional code changes.
> > So choose the simpler solution by setting the array's alignment to size
> > of unsigned long.
> > 
> > Suggested-by: David Laight <David.Laight@...lab.com>
> 
> While this is probably the only play where this 'capabilities' array
> has been detected as misaligned, ISTR there are several other places
> where the identical array is defined and used.
> These all need fixing as well.

Agree 100%  These three patches cover the places *detected* so
far. For bisectability reasons they need to be upstream before
the patches that add WARN_ON, or the one that turns on alignment
traps.  As we find other places, we can fix alignments in other
structures too.

If you remember what those other places are, please let us know
so we can push patches to fix those.

If you have a better strategy to find them ... that also would
be very interesting.

-Tony

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ