All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
To: Anshuman Khandual <khandual@linux.vnet.ibm.com>
Cc: 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,
	sam.bobroff@au1.ibm.com
Subject: Re: [PATCH V4 1/8] elf: Add new PowerPC specifc core note sections
Date: Tue, 18 Nov 2014 13:28:21 -0800	[thread overview]
Message-ID: <20141118212821.GA21540@us.ibm.com> (raw)
In-Reply-To: <1415683597-22819-2-git-send-email-khandual@linux.vnet.ibm.com>

Anshuman Khandual [khandual@linux.vnet.ibm.com] wrote:
| This patch adds four new core note sections for PowerPC transactional
| memory and one core note section for general miscellaneous debug registers.
| These addition of new elf core note sections extends the existing elf ABI
| without affecting it in any manner.
| 
| Signed-off-by: Anshuman Khandual <khandual@linux.vnet.ibm.com>

Except for some minor comments to one of the patches, the patchset
looks good to me. I was also able to run the included selftest on
a PowerVM guest.

Tested-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>

| ---
|  include/uapi/linux/elf.h | 5 +++++
|  1 file changed, 5 insertions(+)
| 
| diff --git a/include/uapi/linux/elf.h b/include/uapi/linux/elf.h
| index ea9bf25..2260fc0 100644
| --- a/include/uapi/linux/elf.h
| +++ b/include/uapi/linux/elf.h
| @@ -379,6 +379,11 @@ typedef struct elf64_shdr {
|  #define NT_PPC_VMX	0x100		/* PowerPC Altivec/VMX registers */
|  #define NT_PPC_SPE	0x101		/* PowerPC SPE/EVR registers */
|  #define NT_PPC_VSX	0x102		/* PowerPC VSX registers */
| +#define NT_PPC_TM_SPR	0x103		/* PowerPC TM special registers */
| +#define NT_PPC_TM_CGPR	0x104		/* PowerpC TM checkpointed GPR */
| +#define NT_PPC_TM_CFPR	0x105		/* PowerPC TM checkpointed FPR */
| +#define NT_PPC_TM_CVMX	0x106		/* PowerPC TM checkpointed VMX */
| +#define NT_PPC_MISC	0x107		/* PowerPC miscellaneous registers */
|  #define NT_386_TLS	0x200		/* i386 TLS slots (struct user_desc) */
|  #define NT_386_IOPERM	0x201		/* x86 io permission bitmap (1=deny) */
|  #define NT_X86_XSTATE	0x202		/* x86 extended state using xsave */
| -- 
| 1.9.3

WARNING: multiple messages have this Message-ID (diff)
From: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
To: Anshuman Khandual <khandual@linux.vnet.ibm.com>
Cc: linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org,
	peterz@infradead.org, akpm@linux-foundation.org,
	tglx@linutronix.de, james.hogan@imgtec.com, avagin@openvz.org,
	Paul.Clothier@imgtec.com, palves@redhat.com, oleg@redhat.com,
	dhowells@redhat.com, davej@redhat.com, davem@davemloft.net,
	mikey@neuling.org, benh@kernel.crashing.org, mpe@ellerman.id.au,
	sam.bobroff@au1.ibm.com
Subject: Re: [PATCH V4 1/8] elf: Add new PowerPC specifc core note sections
Date: Tue, 18 Nov 2014 13:28:21 -0800	[thread overview]
Message-ID: <20141118212821.GA21540@us.ibm.com> (raw)
In-Reply-To: <1415683597-22819-2-git-send-email-khandual@linux.vnet.ibm.com>

Anshuman Khandual [khandual@linux.vnet.ibm.com] wrote:
| This patch adds four new core note sections for PowerPC transactional
| memory and one core note section for general miscellaneous debug registers.
| These addition of new elf core note sections extends the existing elf ABI
| without affecting it in any manner.
| 
| Signed-off-by: Anshuman Khandual <khandual@linux.vnet.ibm.com>

Except for some minor comments to one of the patches, the patchset
looks good to me. I was also able to run the included selftest on
a PowerVM guest.

Tested-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>

| ---
|  include/uapi/linux/elf.h | 5 +++++
|  1 file changed, 5 insertions(+)
| 
| diff --git a/include/uapi/linux/elf.h b/include/uapi/linux/elf.h
| index ea9bf25..2260fc0 100644
| --- a/include/uapi/linux/elf.h
| +++ b/include/uapi/linux/elf.h
| @@ -379,6 +379,11 @@ typedef struct elf64_shdr {
|  #define NT_PPC_VMX	0x100		/* PowerPC Altivec/VMX registers */
|  #define NT_PPC_SPE	0x101		/* PowerPC SPE/EVR registers */
|  #define NT_PPC_VSX	0x102		/* PowerPC VSX registers */
| +#define NT_PPC_TM_SPR	0x103		/* PowerPC TM special registers */
| +#define NT_PPC_TM_CGPR	0x104		/* PowerpC TM checkpointed GPR */
| +#define NT_PPC_TM_CFPR	0x105		/* PowerPC TM checkpointed FPR */
| +#define NT_PPC_TM_CVMX	0x106		/* PowerPC TM checkpointed VMX */
| +#define NT_PPC_MISC	0x107		/* PowerPC miscellaneous registers */
|  #define NT_386_TLS	0x200		/* i386 TLS slots (struct user_desc) */
|  #define NT_386_IOPERM	0x201		/* x86 io permission bitmap (1=deny) */
|  #define NT_X86_XSTATE	0x202		/* x86 extended state using xsave */
| -- 
| 1.9.3


  reply	other threads:[~2014-11-18 21:28 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-11  5:26 [PATCH V4 0/8] Add new PowerPC specific ELF core notes Anshuman Khandual
2014-11-11  5:26 ` Anshuman Khandual
2014-11-11  5:26 ` [PATCH V4 1/8] elf: Add new PowerPC specifc core note sections Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual
2014-11-18 21:28   ` Sukadev Bhattiprolu [this message]
2014-11-18 21:28     ` Sukadev Bhattiprolu
2014-11-21 23:43   ` Andrew Morton
2014-11-21 23:43     ` Andrew Morton
2014-11-25  5:26     ` Anshuman Khandual
2014-11-25  5:26       ` Anshuman Khandual
2014-11-11  5:26 ` [PATCH V4 2/8] powerpc, process: Add functions flush_tm_state, flush_tmregs_to_thread Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual
2014-11-21 13:13   ` Anshuman Khandual
2014-11-21 13:13     ` Anshuman Khandual
2014-11-11  5:26 ` [PATCH V4 3/8] powerpc, process: Merge functions __switch_to_tm and tm_reclaim_task Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual
2014-11-11  5:26 ` [PATCH V4 4/8] powerpc, ptrace: Enable fpr_(get/set) for transactional memory Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual
2014-11-11  5:26 ` [PATCH V4 5/8] powerpc, ptrace: Enable vr_(get/set) " Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual
2014-11-11  5:26 ` [PATCH V4 6/8] powerpc, ptrace: Enable support for transactional memory register sets Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual
2014-11-18 21:18   ` Sukadev Bhattiprolu
2014-11-18 21:18     ` Sukadev Bhattiprolu
2014-11-21 13:11     ` Anshuman Khandual
2014-11-21 13:11       ` Anshuman Khandual
2014-11-11  5:26 ` [PATCH V4 7/8] powerpc, ptrace: Enable support for miscellaneous debug registers Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual
2014-11-13  9:45   ` Anshuman Khandual
2014-11-13  9:45     ` Anshuman Khandual
2014-11-14 10:00     ` Denis Kirjanov
2014-11-14 10:00       ` Denis Kirjanov
2014-11-21 10:26       ` Anshuman Khandual
2014-11-21 10:26         ` Anshuman Khandual
2014-11-24 14:04         ` Denis Kirjanov
2014-11-24 14:04           ` Denis Kirjanov
2014-11-11  5:26 ` [PATCH V4 8/8] selftests, powerpc: Add new test case for TM related ptrace interfaces Anshuman Khandual
2014-11-11  5:26   ` Anshuman Khandual

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141118212821.GA21540@us.ibm.com \
    --to=sukadev@linux.vnet.ibm.com \
    --cc=Paul.Clothier@imgtec.com \
    --cc=akpm@linux-foundation.org \
    --cc=avagin@openvz.org \
    --cc=davej@redhat.com \
    --cc=davem@davemloft.net \
    --cc=dhowells@redhat.com \
    --cc=james.hogan@imgtec.com \
    --cc=khandual@linux.vnet.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=mikey@neuling.org \
    --cc=oleg@redhat.com \
    --cc=palves@redhat.com \
    --cc=peterz@infradead.org \
    --cc=sam.bobroff@au1.ibm.com \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.