Re: [PATCH V4 7/8] powerpc, ptrace: Enable support for miscellaneous debug registers

From: Denis Kirjanov
Date: Fri Nov 14 2014 - 05:00:16 EST


On 11/13/14, Anshuman Khandual <khandual@xxxxxxxxxxxxxxxxxx> wrote:
> On 11/11/2014 10:56 AM, Anshuman Khandual wrote:
>> This patch enables get and set of miscellaneous debug registers through
>> ptrace PTRACE_GETREGSET-PTRACE_SETREGSET interface by implementing new
>> powerpc specific register set REGSET_MISC support corresponding to the
>> new ELF core note NT_PPC_MISC added previously in this regard.
>
> Right now this one does not compile for "ppc64e_defconfig" and
> "pmac32_defconfig" config options. The patch below will fix it
> and would be part of next revision.

It would be great to have a test tool for that under
tools/testing/selftests/ptrace/

> diff --git a/arch/powerpc/kernel/ptrace.c b/arch/powerpc/kernel/ptrace.c
> index 61a2581..be566eb 100644
> --- a/arch/powerpc/kernel/ptrace.c
> +++ b/arch/powerpc/kernel/ptrace.c
> @@ -1326,6 +1326,7 @@ static int tm_cvmx_set(struct task_struct *target,
> }
> #endif /* CONFIG_PPC_TRANSACTIONAL_MEM */
>
> +#ifdef CONFIG_PPC64
> /*
> * get_misc_dbg
> *
> @@ -1339,6 +1340,9 @@ static int tm_cvmx_set(struct task_struct *target,
> * unsigned long ppr;
> * unsigned long tar;
> * };
> + *
> + * The data element 'tar' will be valid only if the
> + * kernel has CONFIG_PPC_BOOK3S_64 config option enabled.
> */
> static int get_misc_dbg(struct task_struct *target,
> const struct user_regset *regset, unsigned int pos,
> @@ -1348,7 +1352,10 @@ static int get_misc_dbg(struct task_struct *target,
>
> /* Build test */
> BUILD_BUG_ON(TSO(dscr) + 2 * sizeof(unsigned long) != TSO(ppr));
> +
> +#ifdef CONFIG_PPC_BOOK3S_64
> BUILD_BUG_ON(TSO(ppr) + sizeof(unsigned long) != TSO(tar));
> +#endif
>
> /* DSCR register */
> ret = user_regset_copyout(&pos, &count, &kbuf, &ubuf,
> @@ -1362,12 +1369,14 @@ static int get_misc_dbg(struct task_struct *target,
> sizeof(unsigned long),
> 2 * sizeof(unsigned long));
>
> +#ifdef CONFIG_PPC_BOOK3S_64
> /* TAR register */
> if (!ret)
> ret = user_regset_copyout(&pos, &count, &kbuf, &ubuf,
> &target->thread.tar,
> 2 * sizeof(unsigned long),
> 3 * sizeof(unsigned long));
> +#endif
> return ret;
> }
>
> @@ -1384,6 +1393,9 @@ static int get_misc_dbg(struct task_struct *target,
> * unsigned long ppr;
> * unsigned long tar;
> * };
> + *
> + * The data element 'tar' will be valid only if the
> + * kernel has CONFIG_PPC_BOOK3S_64 config option enabled.
> */
> static int set_misc_dbg(struct task_struct *target,
> const struct user_regset *regset, unsigned int pos,
> @@ -1394,7 +1406,10 @@ static int set_misc_dbg(struct task_struct *target,
>
> /* Build test */
> BUILD_BUG_ON(TSO(dscr) + 2 * sizeof(unsigned long) != TSO(ppr));
> +
> +#ifdef CONFIG_PPC_BOOK3S_64
> BUILD_BUG_ON(TSO(ppr) + sizeof(unsigned long) != TSO(tar));
> +#endif
>
> /* DSCR register */
> ret = user_regset_copyin(&pos, &count, &kbuf, &ubuf,
> @@ -1407,15 +1422,17 @@ static int set_misc_dbg(struct task_struct *target,
> &target->thread.ppr,
> sizeof(unsigned long),
> 2 * sizeof(unsigned long));
> -
> +#ifdef CONFIG_PPC_BOOK3S_64
> /* TAR register */
> if (!ret)
> ret = user_regset_copyin(&pos, &count, &kbuf, &ubuf,
> &target->thread.tar,
> 2 * sizeof(unsigned long),
> 3 * sizeof(unsigned long));
> +#endif
> return ret;
> }
> +#endif /* CONFIG_PPC64 */
>
> /*
> * These are our native regset flavors.
> @@ -1438,7 +1455,9 @@ enum powerpc_regset {
> REGSET_TM_CFPR, /* TM checkpointed FPR registers */
> REGSET_TM_CVMX, /* TM checkpointed VMX registers */
> #endif
> +#ifdef CONFIG_PPC64
> REGSET_MISC /* Miscellaneous debug registers */
> +#endif
> };
>
> static const struct user_regset native_regsets[] = {
> @@ -1495,11 +1514,13 @@ static const struct user_regset native_regsets[] = {
> .active = tm_cvmx_active, .get = tm_cvmx_get, .set = tm_cvmx_set
> },
> #endif
> +#ifdef CONFIG_PPC64
> [REGSET_MISC] = {
> .core_note_type = NT_PPC_MISC, .n = ELF_NMISCREG,
> .size = sizeof(u64), .align = sizeof(u64),
> .get = get_misc_dbg, .set = set_misc_dbg
> },
> +#endif
> };
>
> static const struct user_regset_view user_ppc_native_view = {
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@xxxxxxxxxxxxxxxx
> https://lists.ozlabs.org/listinfo/linuxppc-dev


--
Regards,
Denis
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/