On 12/17/19 11:51 PM, Sandipan Das wrote:
> write_pkey_reg(pkey_reg);
> - dprintf4("pkey_reg now: %08x\n", read_pkey_reg());
> + dprintf4("pkey_reg now: "PKEY_REG_FMT"\n", read_pkey_reg());
> }
>
> #define ARRAY_SIZE(x) (sizeof(x) / sizeof(*(x)))
> diff --git a/tools/testing/selftests/vm/pkey-x86.h
> b/tools/testing/selftests/vm/pkey-x86.h
> index 2f04ade8ca9c..5f40901219d3 100644
> --- a/tools/testing/selftests/vm/pkey-x86.h
> +++ b/tools/testing/selftests/vm/pkey-x86.h
> @@ -46,6 +46,8 @@
> #define HPAGE_SIZE (1UL<<21)
> #define PAGE_SIZE 4096
> #define MB (1<<20)
> +#define pkey_reg_t u32
> +#define PKEY_REG_FMT "%016x"
How big is the ppc one?
I'd really just rather do %016lx *everywhere* than sprinkle the
PKEY_REG_FMTs around.
BTW, why are you doing a %016lx for a u32?
