Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Thu, Mar 17, 2016 at 11:04:29AM +, Julien Grall wrote: > Hi Konrad, > > On 17/03/16 01:16, Konrad Rzeszutek Wilk wrote: > >And here is the patch. The change for uintXX_t type worked out - same > >size and offset as on 64-bit. Thought I am tempted to add some > >more BUILD_BUG checks. > > >

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

Hi Konrad, On 17/03/16 01:16, Konrad Rzeszutek Wilk wrote: And here is the patch. The change for uintXX_t type worked out - same size and offset as on 64-bit. Thought I am tempted to add some more BUILD_BUG checks. From 7007f1a3fa3a77a725f529420c7aea0e8ebdc9fa Mon Sep 17 00:00:00 2001 From: Ko

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Wed, Mar 16, 2016 at 03:21:51PM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Mar 16, 2016 at 06:08:06PM +, Julien Grall wrote: > > Hi Konrad > > > > On 16/03/2016 17:52, Konrad Rzeszutek Wilk wrote: > > >On Wed, Mar 16, 2016 at 05:32:09PM +, Julien Grall wrote: > > >>Sorry for the late

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Wed, Mar 16, 2016 at 05:32:09PM +, Julien Grall wrote: > Hi Konrad, > > Sorry for the late answer on this patch. I noticed the problem while I was > reviewing your xSplice patch series. > > On 12/02/2016 15:56, Konrad Rzeszutek Wilk wrote: > > From 32a062c119091f2f3f6a4c540a8098e97c273dd2

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

Hi Konrad On 16/03/2016 17:52, Konrad Rzeszutek Wilk wrote: On Wed, Mar 16, 2016 at 05:32:09PM +, Julien Grall wrote: Sorry for the late answer on this patch. I noticed the problem while I was reviewing your xSplice patch series. On 12/02/2016 15:56, Konrad Rzeszutek Wilk wrote: From 32a

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

Hi Konrad, Sorry for the late answer on this patch. I noticed the problem while I was reviewing your xSplice patch series. On 12/02/2016 15:56, Konrad Rzeszutek Wilk wrote: From 32a062c119091f2f3f6a4c540a8098e97c273dd2 Mon Sep 17 00:00:00 2001 From: Konrad Rzeszutek Wilk Date: Fri, 5 Feb 20

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Wed, Mar 16, 2016 at 06:08:06PM +, Julien Grall wrote: > Hi Konrad > > On 16/03/2016 17:52, Konrad Rzeszutek Wilk wrote: > >On Wed, Mar 16, 2016 at 05:32:09PM +, Julien Grall wrote: > >>Sorry for the late answer on this patch. I noticed the problem while I was > >>reviewing your xSplice

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Fri, Feb 12, 2016 at 03:57:25PM +, Stefano Stabellini wrote: > On Fri, 12 Feb 2016, Konrad Rzeszutek Wilk wrote: > > On Fri, Feb 12, 2016 at 03:26:14PM +, Stefano Stabellini wrote: > > > On Fri, 12 Feb 2016, Jan Beulich wrote: > > > > >>> On 12.02.16 at 15:17, wrote: > > > > > --- a/xen

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Fri, 12 Feb 2016, Konrad Rzeszutek Wilk wrote: > On Fri, Feb 12, 2016 at 03:26:14PM +, Stefano Stabellini wrote: > > On Fri, 12 Feb 2016, Jan Beulich wrote: > > > >>> On 12.02.16 at 15:17, wrote: > > > > --- a/xen/include/asm-arm/config.h > > > > +++ b/xen/include/asm-arm/config.h > > > > @

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Fri, Feb 12, 2016 at 03:26:14PM +, Stefano Stabellini wrote: > On Fri, 12 Feb 2016, Jan Beulich wrote: > > >>> On 12.02.16 at 15:17, wrote: > > > --- a/xen/include/asm-arm/config.h > > > +++ b/xen/include/asm-arm/config.h > > > @@ -15,8 +15,10 @@ > > > > > > #if defined(CONFIG_ARM_64) >

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Fri, 12 Feb 2016, Jan Beulich wrote: > >>> On 12.02.16 at 15:17, wrote: > > --- a/xen/include/asm-arm/config.h > > +++ b/xen/include/asm-arm/config.h > > @@ -15,8 +15,10 @@ > > > > #if defined(CONFIG_ARM_64) > > # define LONG_BYTEORDER 3 > > +# define ELFSIZE 64 > > #else > > # define LON

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

>>> On 12.02.16 at 15:17, wrote: > --- a/xen/include/asm-arm/config.h > +++ b/xen/include/asm-arm/config.h > @@ -15,8 +15,10 @@ > > #if defined(CONFIG_ARM_64) > # define LONG_BYTEORDER 3 > +# define ELFSIZE 64 > #else > # define LONG_BYTEORDER 2 > +# define ELFSIZE 64 > #endif Leaving the

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Fri, Feb 12, 2016 at 11:26:10AM +, Stefano Stabellini wrote: > On Thu, 11 Feb 2016, Konrad Rzeszutek Wilk wrote: > > Otherwise any code that tries to use Elf_* macros instead of > > Elf32_ or Elf_64 fails to compile. > > > > CC: ian.campb...@citrix.com > > CC: wei.l...@citrix.com > > CC: st

Re: [Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

On Thu, 11 Feb 2016, Konrad Rzeszutek Wilk wrote: > Otherwise any code that tries to use Elf_* macros instead of > Elf32_ or Elf_64 fails to compile. > > CC: ian.campb...@citrix.com > CC: wei.l...@citrix.com > CC: stefano.stabell...@citrix.com > Signed-off-by: Konrad Rzeszutek Wilk > --- > xen/i

[Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

Otherwise any code that tries to use Elf_* macros instead of Elf32_ or Elf_64 fails to compile. CC: ian.campb...@citrix.com CC: wei.l...@citrix.com CC: stefano.stabell...@citrix.com Signed-off-by: Konrad Rzeszutek Wilk --- xen/include/asm-arm/config.h | 2 ++ 1 file changed, 2 insertions(+) dif