[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20170814161829.7kfaqliitdmngtxa@kernel.org>
Date: Mon, 14 Aug 2017 09:18:29 -0700
From: Shaohua Li <shli@...nel.org>
To: Denys Vlasenko <dvlasenk@...hat.com>
Cc: "H. Peter Anvin" <hpa@...or.com>, mingo@...hat.com,
Jim Kukunas <james.t.kukunas@...ux.intel.com>,
Fenghua Yu <fenghua.yu@...el.com>,
Megha Dey <megha.dey@...ux.intel.com>,
Gayatri Kammela <gayatri.kammela@...el.com>,
Shaohua Li <shli@...com>, x86@...nel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] lib/raid6: align AVX512 constants to 512 bits, not bytes
On Sat, Aug 12, 2017 at 07:43:46PM +0200, Denys Vlasenko wrote:
> Signed-off-by: Denys Vlasenko <dvlasenk@...hat.com>
> Cc: H. Peter Anvin <hpa@...or.com>
> Cc: mingo@...hat.com
> Cc: Jim Kukunas <james.t.kukunas@...ux.intel.com>
> Cc: Fenghua Yu <fenghua.yu@...el.com>
> Cc: Megha Dey <megha.dey@...ux.intel.com>
> Cc: Gayatri Kammela <gayatri.kammela@...el.com>
> Cc: Shaohua Li <shli@...com>
> Cc: x86@...nel.org
> Cc: linux-kernel@...r.kernel.org
applied, thanks!
> ---
> avx512.o before:
> Sections:
> Idx Name Size VMA LMA File off Algn
> 4 .rodata 00000240 0000000000000000 0000000000000000 00000c00 2**9
> CONTENTS, ALLOC, LOAD, RELOC, READONLY, DATA
> ...
> Contents of section .rodata:
> 0000 00000000 00000000 00000000 00000000 ................
> 0010 00000000 00000000 00000000 00000000 ................
> 0020 01000000 00000000 00000000 00000000 ................
> 0030 00000000 00000000 00000000 00000000 ................
> 0040 00000000 00000000 00000000 00000000 ................
> 0050 00000000 00000000 00000000 00000000 ................
> 0060 01000000 00000000 00000000 00000000 ................
> 0070 00000000 00000000 00000000 00000000 ................
> 0080 00000000 00000000 00000000 00000000 ................
> 0090 00000000 00000000 00000000 00000000 ................
> 00a0 01000000 00000000 00000000 00000000 ................
> 00b0 00000000 00000000 00000000 00000000 ................
> 00c0 00000000 00000000 00000000 00000000 ................
> 00d0 00000000 00000000 00000000 00000000 ................
> 00e0 00000000 00000000 00000000 00000000 ................
> 00f0 00000000 00000000 00000000 00000000 ................
> 0100 00000000 00000000 00000000 00000000 ................
> 0110 00000000 00000000 00000000 00000000 ................
> 0120 00000000 00000000 00000000 00000000 ................
> 0130 00000000 00000000 00000000 00000000 ................
> 0140 00000000 00000000 00000000 00000000 ................
> 0150 00000000 00000000 00000000 00000000 ................
> 0160 00000000 00000000 00000000 00000000 ................
> 0170 00000000 00000000 00000000 00000000 ................
> 0180 00000000 00000000 00000000 00000000 ................
> 0190 00000000 00000000 00000000 00000000 ................
> 01a0 00000000 00000000 00000000 00000000 ................
> 01b0 00000000 00000000 00000000 00000000 ................
> 01c0 00000000 00000000 00000000 00000000 ................
> 01d0 00000000 00000000 00000000 00000000 ................
> 01e0 00000000 00000000 00000000 00000000 ................
> 01f0 00000000 00000000 00000000 00000000 ................
> 0200 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
> 0210 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
> 0220 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
> 0230 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
>
> avx512.o after:
> Sections:
> Idx Name Size VMA LMA File off Algn
> 4 .rodata 00000100 0000000000000000 0000000000000000 00000b40 2**6
> CONTENTS, ALLOC, LOAD, RELOC, READONLY, DATA
> ...
> Contents of section .rodata:
> 0000 00000000 00000000 00000000 00000000 ................
> 0010 00000000 00000000 00000000 00000000 ................
> 0020 01000000 00000000 00000000 00000000 ................
> 0030 00000000 00000000 00000000 00000000 ................
> 0040 00000000 00000000 00000000 00000000 ................
> 0050 00000000 00000000 00000000 00000000 ................
> 0060 01000000 00000000 00000000 00000000 ................
> 0070 00000000 00000000 00000000 00000000 ................
> 0080 00000000 00000000 00000000 00000000 ................
> 0090 00000000 00000000 00000000 00000000 ................
> 00a0 01000000 00000000 00000000 00000000 ................
> 00b0 00000000 00000000 00000000 00000000 ................
> 00c0 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
> 00d0 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
> 00e0 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
> 00f0 1d1d1d1d 1d1d1d1d 1d1d1d1d 1d1d1d1d ................
>
>
> lib/raid6/avx512.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/raid6/avx512.c b/lib/raid6/avx512.c
> index f524a79..46df797 100644
> --- a/lib/raid6/avx512.c
> +++ b/lib/raid6/avx512.c
> @@ -29,7 +29,7 @@
>
> static const struct raid6_avx512_constants {
> u64 x1d[8];
> -} raid6_avx512_constants __aligned(512) = {
> +} raid6_avx512_constants __aligned(512/8) = {
> { 0x1d1d1d1d1d1d1d1dULL, 0x1d1d1d1d1d1d1d1dULL,
> 0x1d1d1d1d1d1d1d1dULL, 0x1d1d1d1d1d1d1d1dULL,
> 0x1d1d1d1d1d1d1d1dULL, 0x1d1d1d1d1d1d1d1dULL,
> --
> 2.9.2
>
Powered by blists - more mailing lists