diff for duplicates of <56EFE3EA.5000900@redhat.com> diff --git a/a/1.txt b/N1/1.txt index 4b5c60b..60504bd 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -30,9 +30,9 @@ On 21.03.2016 12:33, Laurent Vivier wrote: > + * - Data are loaded into 4 low order bytes of registers (Word). > + * - The unfilled bytes are set to 0. > + * - The sequence of registers wraps around to GPR0. -> + * - if n = 0, content of RT is undefined +> + * - if n == 0, content of RT is undefined > + * - RT <= RA or RB < RT + (n + 4) is invalid or result is undefined -> + * - RT = RA = 0 is invalid +> + * - RT == RA == 0 is invalid > + * > + */ > + @@ -64,7 +64,7 @@ On 21.03.2016 12:33, Laurent Vivier wrote: > + : > + "xer", "r11", "r12", "memory"); > + -> + report("partial", regs[0] = 0x01020300 && regs[1] = (uint64_t)-1); +> + report("partial", regs[0] == 0x01020300 && regs[1] == (uint64_t)-1); > + > + /* check an old know bug: the number of bytes is used as > + * the number of registers, so try 32 bytes. @@ -98,11 +98,11 @@ On 21.03.2016 12:33, Laurent Vivier wrote: > + "xer", "r11", "r12", "r13", "r14", "r15", "r16", "r17", > + "r18", "r19", "memory"); > + -> + report("length", regs[0] = 0x01020304 && regs[1] = 0x05060708 && -> + regs[2] = 0x090a0b0c && regs[3] = 0x0d0e0f10 && -> + regs[4] = 0x11121314 && regs[5] = 0x15161718 && -> + regs[6] = 0x191a1b1c && regs[7] = 0x1d1e1f20 && -> + regs[8] = (uint64_t)-1); +> + report("length", regs[0] == 0x01020304 && regs[1] == 0x05060708 && +> + regs[2] == 0x090a0b0c && regs[3] == 0x0d0e0f10 && +> + regs[4] == 0x11121314 && regs[5] == 0x15161718 && +> + regs[6] == 0x191a1b1c && regs[7] == 0x1d1e1f20 && +> + regs[8] == (uint64_t)-1); > + > + /* check wrap around to r0 */ > + @@ -120,8 +120,8 @@ On 21.03.2016 12:33, Laurent Vivier wrote: > + /* modify two registers from r31, wrap around to r0 */ > + "xer", "r31", "r0", "memory"); > + -> + report("wrap around to r0", regs[0] = 0x01020304 && -> + regs[1] = 0x05060708); +> + report("wrap around to r0", regs[0] == 0x01020304 && +> + regs[1] == 0x05060708); > + > + /* check wrap around to r0 over RB doesn't break RB */ > + @@ -155,7 +155,7 @@ series, you could also drop one of the two comments, I think. > + * overwrite the register. > + * In all the cases, the register must stay untouched > + */ -> + report("Don't overwrite Rb", regs[1] = (uint64_t)addr); +> + report("Don't overwrite Rb", regs[1] == (uint64_t)addr); > + > + report_prefix_pop(); > +} diff --git a/a/content_digest b/N1/content_digest index 4467c13..8fd17cb 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,7 +2,7 @@ "ref\01458560014-28862-5-git-send-email-lvivier@redhat.com\0" "From\0Thomas Huth <thuth@redhat.com>\0" "Subject\0Re: [kvm-unit-tests PATCH v2 4/5] powerpc: check lswx\0" - "Date\0Mon, 21 Mar 2016 12:07:06 +0000\0" + "Date\0Mon, 21 Mar 2016 13:07:06 +0100\0" "To\0Laurent Vivier <lvivier@redhat.com>" kvm@vger.kernel.org " kvm-ppc@vger.kernel.org\0" @@ -43,9 +43,9 @@ "> + * - Data are loaded into 4 low order bytes of registers (Word).\n" "> + * - The unfilled bytes are set to 0.\n" "> + * - The sequence of registers wraps around to GPR0.\n" - "> + * - if n = 0, content of RT is undefined\n" + "> + * - if n == 0, content of RT is undefined\n" "> + * - RT <= RA or RB < RT + (n + 4) is invalid or result is undefined\n" - "> + * - RT = RA = 0 is invalid\n" + "> + * - RT == RA == 0 is invalid\n" "> + *\n" "> + */\n" "> +\n" @@ -77,7 +77,7 @@ "> +\t\t :\n" "> +\t\t \"xer\", \"r11\", \"r12\", \"memory\");\n" "> +\n" - "> +\treport(\"partial\", regs[0] = 0x01020300 && regs[1] = (uint64_t)-1);\n" + "> +\treport(\"partial\", regs[0] == 0x01020300 && regs[1] == (uint64_t)-1);\n" "> +\n" "> +\t/* check an old know bug: the number of bytes is used as\n" "> +\t * the number of registers, so try 32 bytes.\n" @@ -111,11 +111,11 @@ "> +\t\t \"xer\", \"r11\", \"r12\", \"r13\", \"r14\", \"r15\", \"r16\", \"r17\",\n" "> +\t\t \"r18\", \"r19\", \"memory\");\n" "> +\n" - "> +\treport(\"length\", regs[0] = 0x01020304 && regs[1] = 0x05060708 &&\n" - "> +\t\t\t regs[2] = 0x090a0b0c && regs[3] = 0x0d0e0f10 &&\n" - "> +\t\t\t regs[4] = 0x11121314 && regs[5] = 0x15161718 &&\n" - "> +\t\t\t regs[6] = 0x191a1b1c && regs[7] = 0x1d1e1f20 &&\n" - "> +\t\t\t regs[8] = (uint64_t)-1);\n" + "> +\treport(\"length\", regs[0] == 0x01020304 && regs[1] == 0x05060708 &&\n" + "> +\t\t\t regs[2] == 0x090a0b0c && regs[3] == 0x0d0e0f10 &&\n" + "> +\t\t\t regs[4] == 0x11121314 && regs[5] == 0x15161718 &&\n" + "> +\t\t\t regs[6] == 0x191a1b1c && regs[7] == 0x1d1e1f20 &&\n" + "> +\t\t\t regs[8] == (uint64_t)-1);\n" "> +\n" "> +\t/* check wrap around to r0 */\n" "> +\n" @@ -133,8 +133,8 @@ "> +\t\t /* modify two registers from r31, wrap around to r0 */\n" "> +\t\t \"xer\", \"r31\", \"r0\", \"memory\");\n" "> +\n" - "> +\treport(\"wrap around to r0\", regs[0] = 0x01020304 &&\n" - "> +\t\t\t regs[1] = 0x05060708);\n" + "> +\treport(\"wrap around to r0\", regs[0] == 0x01020304 &&\n" + "> +\t\t\t regs[1] == 0x05060708);\n" "> +\n" "> +\t/* check wrap around to r0 over RB doesn't break RB */\n" "> +\n" @@ -168,7 +168,7 @@ "> +\t * overwrite the register.\n" "> +\t * In all the cases, the register must stay untouched\n" "> +\t */\n" - "> +\treport(\"Don't overwrite Rb\", regs[1] = (uint64_t)addr);\n" + "> +\treport(\"Don't overwrite Rb\", regs[1] == (uint64_t)addr);\n" "> +\n" "> +\treport_prefix_pop();\n" "> +}\n" @@ -188,4 +188,4 @@ "\n" Reviewed-by: Thomas Huth <thuth@redhat.com> -922b0bd60c7c5d07eab8b1c91f328b1e7173195236a46f1ab4e8aade078642e0 +44283bd32272f2001ce9b86faee1928636789769864210a648d4914141ac9097
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.