diff for duplicates of <20141008171650.GA7110@us.ibm.com> diff --git a/a/1.txt b/N1/1.txt index 1dfe151..efa362b 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,27 +1,24 @@ Anshuman Khandual [khandual@linux.vnet.ibm.com] wrote: | On 08/28/2014 03:05 AM, Sukadev Bhattiprolu wrote: -| >=20 +| > | > I see these in arch/powerpc/include/asm/processor.h -| >=20 +| > | > #ifdef CONFIG_PPC64 | > unsigned long dscr; | > int dscr_inherit; | > unsigned long ppr; /* used to save/restore SMT priority */ | > #endif -| >=20 +| > | > where there is an 'int' between ppr and dscr. So, should one of | > the above sizeof(unsigned long) be changed to sizeof(int) ? -|=20 +| | Right, I understand that but strangely I get this compile time error | when it is changed to sizeof(int). -|=20 -| error: call to =E2=80=98__compiletime_assert_1350=E2=80=99 declared with= - attribute error: -| BUILD_BUG_ON failed: TSO(dscr) + sizeof(unsigned long) + sizeof(int) != -=3D TSO(ppr) -| BUILD_BUG_ON(TSO(dscr) + sizeof(unsigned long) + sizeof(int) !=3D TSO(p= -pr)); -|=20 +| +| error: call to ‘__compiletime_assert_1350’ declared with attribute error: +| BUILD_BUG_ON failed: TSO(dscr) + sizeof(unsigned long) + sizeof(int) != TSO(ppr) +| BUILD_BUG_ON(TSO(dscr) + sizeof(unsigned long) + sizeof(int) != TSO(ppr)); +| | may be I am missing something here. I guess there is a 4-byte padding after dscr_inherit. We could make that diff --git a/a/content_digest b/N1/content_digest index 55188c7..18fa333 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -6,46 +6,43 @@ "Subject\0Re: [PATCH V3 3/3] powerpc, ptrace: Enable support for miscellaneous registers\0" "Date\0Wed, 8 Oct 2014 10:16:50 -0700\0" "To\0Anshuman Khandual <khandual@linux.vnet.ibm.com>\0" - "Cc\0mikey@neuling.org" + "Cc\0linux-kernel@vger.kernel.org" + linuxppc-dev@ozlabs.org + peterz@infradead.org + akpm@linux-foundation.org + tglx@linutronix.de + mikey@neuling.org james.hogan@imgtec.com avagin@openvz.org Paul.Clothier@imgtec.com - davem@davemloft.net - peterz@infradead.org palves@redhat.com - linux-kernel@vger.kernel.org oleg@redhat.com dhowells@redhat.com - linuxppc-dev@ozlabs.org davej@redhat.com - akpm@linux-foundation.org - " tglx@linutronix.de\0" + " davem@davemloft.net\0" "\00:1\0" "b\0" "Anshuman Khandual [khandual@linux.vnet.ibm.com] wrote:\n" "| On 08/28/2014 03:05 AM, Sukadev Bhattiprolu wrote:\n" - "| >=20\n" + "| > \n" "| > I see these in arch/powerpc/include/asm/processor.h\n" - "| >=20\n" + "| > \n" "| > #ifdef CONFIG_PPC64\n" "| > unsigned long dscr;\n" "| > int dscr_inherit;\n" "| > unsigned long ppr; /* used to save/restore SMT priority */\n" "| > #endif\n" - "| >=20\n" + "| > \n" "| > where there is an 'int' between ppr and dscr. So, should one of\n" "| > the above sizeof(unsigned long) be changed to sizeof(int) ?\n" - "|=20\n" + "| \n" "| Right, I understand that but strangely I get this compile time error\n" "| when it is changed to sizeof(int).\n" - "|=20\n" - "| error: call to =E2=80=98__compiletime_assert_1350=E2=80=99 declared with=\n" - " attribute error:\n" - "| BUILD_BUG_ON failed: TSO(dscr) + sizeof(unsigned long) + sizeof(int) !=\n" - "=3D TSO(ppr)\n" - "| BUILD_BUG_ON(TSO(dscr) + sizeof(unsigned long) + sizeof(int) !=3D TSO(p=\n" - "pr));\n" - "|=20\n" + "| \n" + "| error: call to \342\200\230__compiletime_assert_1350\342\200\231 declared with attribute error:\n" + "| BUILD_BUG_ON failed: TSO(dscr) + sizeof(unsigned long) + sizeof(int) != TSO(ppr)\n" + "| BUILD_BUG_ON(TSO(dscr) + sizeof(unsigned long) + sizeof(int) != TSO(ppr));\n" + "| \n" "| may be I am missing something here.\n" "\n" "I guess there is a 4-byte padding after dscr_inherit. We could make that\n" @@ -55,4 +52,4 @@ "\n" Sukadev -41cd5683c28ae5e9a4159ed281498a86a45451f47c93457bbe8442dc95afc41d +234a96bb2681ea2a8de1dd32af73e7ebdcf74bd1ef0a2163249d214b83b35362
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.