diff for duplicates of <56D6BB7B.70401@redhat.com> diff --git a/a/1.txt b/N1/1.txt index a330c22..5df3c93 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -113,17 +113,17 @@ On 02/03/2016 09:40, Thomas Huth wrote: > + rc = hcall(H_SET_SPRG0, 0xcafebabedeadbeefULL); > + sprg0 = mfspr(SPR_SPRG0); > + report("sprg0 = 0xcafebabedeadbeef", -> + rc = H_SUCCESS && sprg0 = 0xcafebabedeadbeefULL); +> + rc == H_SUCCESS && sprg0 == 0xcafebabedeadbeefULL); > + > + rc = hcall(H_SET_SPRG0, 0xaaaaaaaa55555555ULL); > + sprg0 = mfspr(SPR_SPRG0); > + report("sprg0 = 0xaaaaaaaa55555555", -> + rc = H_SUCCESS && sprg0 = 0xaaaaaaaa55555555ULL); +> + rc == H_SUCCESS && sprg0 == 0xaaaaaaaa55555555ULL); > + > + rc = hcall(H_SET_SPRG0, sprg0_orig); > + sprg0 = mfspr(SPR_SPRG0); > + report("sprg0 = 0x%llx", -> + rc = H_SUCCESS && sprg0 = sprg0_orig, sprg0_orig); +> + rc == H_SUCCESS && sprg0 == sprg0_orig, sprg0_orig); > +} > + > +/** @@ -145,23 +145,23 @@ On 02/03/2016 09:40, Thomas Huth wrote: > + > + memset(dst, 0xaa, PAGE_SIZE); > + rc = hcall(H_PAGE_INIT, H_ZERO_PAGE, dst, src); -> + report("h_zero_page", rc = H_SUCCESS && *(uint64_t*)dst = 0); +> + report("h_zero_page", rc == H_SUCCESS && *(uint64_t*)dst == 0); > + > + *(uint64_t*)src = 0xbeefc0dedeadcafeULL; > + rc = hcall(H_PAGE_INIT, H_COPY_PAGE, dst, src); > + report("h_copy_page", -> + rc = H_SUCCESS && *(uint64_t*)dst = 0xbeefc0dedeadcafeULL); +> + rc == H_SUCCESS && *(uint64_t*)dst == 0xbeefc0dedeadcafeULL); > + > + *(uint64_t*)src = 0x9abcdef012345678ULL; > + rc = hcall(H_PAGE_INIT, H_COPY_PAGE|H_ZERO_PAGE, dst, src); > + report("h_copy_page+h_zero_page", -> + rc = H_SUCCESS && *(uint64_t*)dst = 0x9abcdef012345678ULL); +> + rc == H_SUCCESS && *(uint64_t*)dst == 0x9abcdef012345678ULL); > + > + rc = hcall(H_PAGE_INIT, H_ZERO_PAGE, dst + 0x123, src); -> + report("h_zero_page unaligned dst", rc = H_PARAMETER); +> + report("h_zero_page unaligned dst", rc == H_PARAMETER); > + > + rc = hcall(H_PAGE_INIT, H_COPY_PAGE, dst, src + 0x123); -> + report("h_copy_page unaligned src", rc = H_PARAMETER); +> + report("h_copy_page unaligned src", rc == H_PARAMETER); > +} > + > +static int h_random(uint64_t *val) @@ -193,7 +193,7 @@ have to declare it in the input operands field. > + > + /* H_RANDOM is optional - so check for sane return values first */ > + rc = h_random(&rval); -> + report_xfail("h-call available", rc = H_FUNCTION, rc = H_SUCCESS); +> + report_xfail("h-call available", rc == H_FUNCTION, rc == H_SUCCESS); > + if (rc != H_SUCCESS) > + return; @@ -214,12 +214,12 @@ following report. > + val1 &= rval; > + } while (i-- > 0 && (val0 != ~0ULL || val1 != 0ULL)); > + -> + report("no stuck bits", rc = H_SUCCESS && val0 = ~0ULL && val1 = 0); +> + report("no stuck bits", rc == H_SUCCESS && val0 == ~0ULL && val1 == 0); something like: - report_xfail("no stuck bits", rc = H_FUNCTION, \ - rc = H_SUCCESS && val0 = ~0ULL && val1 = 0); + report_xfail("no stuck bits", rc == H_FUNCTION, \ + rc == H_SUCCESS && val0 == ~0ULL && val1 == 0); Is it possible to use "ibm,hypertasfunctions" of /rtas node to know if it is supported (instead of H_FUNCTION)? @@ -243,12 +243,12 @@ it is supported (instead of H_FUNCTION)? > + > + report_prefix_push("hypercall"); > + -> + if (!argc || (argc = 1 && !strcmp(argv[0], "all"))) +> + if (!argc || (argc == 1 && !strcmp(argv[0], "all"))) > + all = 1; > + > + for (i = 0; hctests[i].name != NULL; i++) { > + report_prefix_push(hctests[i].name); -> + if (all || strcmp(argv[0], hctests[i].name) = 0) { +> + if (all || strcmp(argv[0], hctests[i].name) == 0) { > + hctests[i].func(argc, argv); > + } > + report_prefix_pop(); @@ -262,7 +262,7 @@ it is supported (instead of H_FUNCTION)? > +++ b/powerpc/unittests.cfg > @@ -28,3 +28,6 @@ file = selftest.elf > smp = 2 -> extra_params = -m 256 -append 'setup smp=2 mem%6' +> extra_params = -m 256 -append 'setup smp=2 mem=256' > groups = selftest > + > +[spapr_hcall] diff --git a/a/content_digest b/N1/content_digest index cc1b04b..8178ccf 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,7 +1,7 @@ "ref\01456908043-11184-1-git-send-email-thuth@redhat.com\0" "From\0Laurent Vivier <lvivier@redhat.com>\0" "Subject\0Re: [PATCH v3] powerpc: Add tests for sPAPR h-calls\0" - "Date\0Wed, 02 Mar 2016 10:07:55 +0000\0" + "Date\0Wed, 2 Mar 2016 11:07:55 +0100\0" "To\0Thomas Huth <thuth@redhat.com>" kvm@vger.kernel.org kvm-ppc@vger.kernel.org @@ -124,17 +124,17 @@ "> +\trc = hcall(H_SET_SPRG0, 0xcafebabedeadbeefULL);\n" "> +\tsprg0 = mfspr(SPR_SPRG0);\n" "> +\treport(\"sprg0 = 0xcafebabedeadbeef\",\n" - "> +\t\trc = H_SUCCESS && sprg0 = 0xcafebabedeadbeefULL);\n" + "> +\t\trc == H_SUCCESS && sprg0 == 0xcafebabedeadbeefULL);\n" "> +\n" "> +\trc = hcall(H_SET_SPRG0, 0xaaaaaaaa55555555ULL);\n" "> +\tsprg0 = mfspr(SPR_SPRG0);\n" "> +\treport(\"sprg0 = 0xaaaaaaaa55555555\",\n" - "> +\t\trc = H_SUCCESS && sprg0 = 0xaaaaaaaa55555555ULL);\n" + "> +\t\trc == H_SUCCESS && sprg0 == 0xaaaaaaaa55555555ULL);\n" "> +\n" "> +\trc = hcall(H_SET_SPRG0, sprg0_orig);\n" "> +\tsprg0 = mfspr(SPR_SPRG0);\n" "> +\treport(\"sprg0 = 0x%llx\",\n" - "> +\t\trc = H_SUCCESS && sprg0 = sprg0_orig, sprg0_orig);\n" + "> +\t\trc == H_SUCCESS && sprg0 == sprg0_orig, sprg0_orig);\n" "> +}\n" "> +\n" "> +/**\n" @@ -156,23 +156,23 @@ "> +\n" "> +\tmemset(dst, 0xaa, PAGE_SIZE);\n" "> +\trc = hcall(H_PAGE_INIT, H_ZERO_PAGE, dst, src);\n" - "> +\treport(\"h_zero_page\", rc = H_SUCCESS && *(uint64_t*)dst = 0);\n" + "> +\treport(\"h_zero_page\", rc == H_SUCCESS && *(uint64_t*)dst == 0);\n" "> +\n" "> +\t*(uint64_t*)src = 0xbeefc0dedeadcafeULL;\n" "> +\trc = hcall(H_PAGE_INIT, H_COPY_PAGE, dst, src);\n" "> +\treport(\"h_copy_page\",\n" - "> +\t\trc = H_SUCCESS && *(uint64_t*)dst = 0xbeefc0dedeadcafeULL);\n" + "> +\t\trc == H_SUCCESS && *(uint64_t*)dst == 0xbeefc0dedeadcafeULL);\n" "> +\n" "> +\t*(uint64_t*)src = 0x9abcdef012345678ULL;\n" "> +\trc = hcall(H_PAGE_INIT, H_COPY_PAGE|H_ZERO_PAGE, dst, src);\n" "> +\treport(\"h_copy_page+h_zero_page\",\n" - "> +\t\trc = H_SUCCESS && *(uint64_t*)dst = 0x9abcdef012345678ULL);\n" + "> +\t\trc == H_SUCCESS && *(uint64_t*)dst == 0x9abcdef012345678ULL);\n" "> +\n" "> +\trc = hcall(H_PAGE_INIT, H_ZERO_PAGE, dst + 0x123, src);\n" - "> +\treport(\"h_zero_page unaligned dst\", rc = H_PARAMETER);\n" + "> +\treport(\"h_zero_page unaligned dst\", rc == H_PARAMETER);\n" "> +\n" "> +\trc = hcall(H_PAGE_INIT, H_COPY_PAGE, dst, src + 0x123);\n" - "> +\treport(\"h_copy_page unaligned src\", rc = H_PARAMETER);\n" + "> +\treport(\"h_copy_page unaligned src\", rc == H_PARAMETER);\n" "> +}\n" "> +\n" "> +static int h_random(uint64_t *val)\n" @@ -204,7 +204,7 @@ "> +\n" "> +\t/* H_RANDOM is optional - so check for sane return values first */\n" "> +\trc = h_random(&rval);\n" - "> +\treport_xfail(\"h-call available\", rc = H_FUNCTION, rc = H_SUCCESS);\n" + "> +\treport_xfail(\"h-call available\", rc == H_FUNCTION, rc == H_SUCCESS);\n" "> +\tif (rc != H_SUCCESS)\n" "> +\t\treturn;\n" "\n" @@ -225,12 +225,12 @@ "> +\t\tval1 &= rval;\n" "> +\t} while (i-- > 0 && (val0 != ~0ULL || val1 != 0ULL));\n" "> +\n" - "> +\treport(\"no stuck bits\", rc = H_SUCCESS && val0 = ~0ULL && val1 = 0);\n" + "> +\treport(\"no stuck bits\", rc == H_SUCCESS && val0 == ~0ULL && val1 == 0);\n" "\n" "something like:\n" "\n" - " report_xfail(\"no stuck bits\", rc = H_FUNCTION, \\\n" - " rc = H_SUCCESS && val0 = ~0ULL && val1 = 0);\n" + " report_xfail(\"no stuck bits\", rc == H_FUNCTION, \\\n" + " rc == H_SUCCESS && val0 == ~0ULL && val1 == 0);\n" "\n" "Is it possible to use \"ibm,hypertasfunctions\" of /rtas node to know if\n" "it is supported (instead of H_FUNCTION)?\n" @@ -254,12 +254,12 @@ "> +\n" "> +\treport_prefix_push(\"hypercall\");\n" "> +\n" - "> +\tif (!argc || (argc = 1 && !strcmp(argv[0], \"all\")))\n" + "> +\tif (!argc || (argc == 1 && !strcmp(argv[0], \"all\")))\n" "> +\t\tall = 1;\n" "> +\n" "> +\tfor (i = 0; hctests[i].name != NULL; i++) {\n" "> +\t\treport_prefix_push(hctests[i].name);\n" - "> +\t\tif (all || strcmp(argv[0], hctests[i].name) = 0) {\n" + "> +\t\tif (all || strcmp(argv[0], hctests[i].name) == 0) {\n" "> +\t\t\thctests[i].func(argc, argv);\n" "> +\t\t}\n" "> +\t\treport_prefix_pop();\n" @@ -273,11 +273,11 @@ "> +++ b/powerpc/unittests.cfg\n" "> @@ -28,3 +28,6 @@ file = selftest.elf\n" "> smp = 2\n" - "> extra_params = -m 256 -append 'setup smp=2 mem%6'\n" + "> extra_params = -m 256 -append 'setup smp=2 mem=256'\n" "> groups = selftest\n" "> +\n" "> +[spapr_hcall]\n" "> +file = spapr_hcall.elf\n" > -11d29be40301859451e6d090ad9476ec32612ee41dd8bcf3771bad0a1dcd088f +d16efa809a6d8466aeffb0ea408fdcf2230c9e86fba716ef59e78042bfe1cf3a
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.