Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1272446 > unrolled thread

[PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

Started byBoris Ostrovsky <boris.ostrovsky@oracle.com>
First post2015-11-18 19:20 +0100
Last post2015-11-20 18:00 +0100
Articles 7 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2015-11-18 19:20 +0100
    Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV  guests Juergen Gross <jgross@suse.com> - 2015-11-19 10:20 +0100
    Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV  guests Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-11-20 12:30 +0100
      Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV  guests Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2015-11-20 16:20 +0100
        Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV  guests Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-11-20 16:40 +0100
          Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV  guests Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2015-11-20 17:40 +0100
            Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV  guests Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2015-11-20 18:00 +0100

#1272446 — [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

FromBoris Ostrovsky <boris.ostrovsky@oracle.com>
Date2015-11-18 19:20 +0100
Subject[PATCH v2] xen/events: Always allocate legacy interrupts on PV guests
Message-ID<qwfuP-6xU-13@gated-at.bofh.it>
After commit 8c058b0b9c34 ("x86/irq: Probe for PIC presence before
allocating descs for legacy IRQs") early_irq_init() will no longer
preallocate descriptors for legacy interrupts if PIC does not
exist, which is the case for Xen PV guests.

Therefore we may need to allocate those descriptors ourselves.

Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Suggested-by: Thomas Gleixner <tglx@linutronix.de>
---

v2: Use nr_legacy_irqs() instead of NR_IRQS_LEGACY (needs definition for ARM)

 arch/arm/include/asm/irq.h       | 4 ++++
 drivers/xen/events/events_base.c | 5 +++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/irq.h b/arch/arm/include/asm/irq.h
index be1d07d..b864f60 100644
--- a/arch/arm/include/asm/irq.h
+++ b/arch/arm/include/asm/irq.h
@@ -2,6 +2,10 @@
 #define __ASM_ARM_IRQ_H
 
 #define NR_IRQS_LEGACY	16
+static inline int nr_legacy_irqs(void)
+{
+	return NR_IRQS_LEGACY;
+}
 
 #ifndef CONFIG_SPARSE_IRQ
 #include <mach/irqs.h>
diff --git a/drivers/xen/events/events_base.c b/drivers/xen/events/events_base.c
index 849500e..524c221 100644
--- a/drivers/xen/events/events_base.c
+++ b/drivers/xen/events/events_base.c
@@ -39,6 +39,7 @@
 #include <asm/irq.h>
 #include <asm/idle.h>
 #include <asm/io_apic.h>
+#include <asm/i8259.h>
 #include <asm/xen/pci.h>
 #endif
 #include <asm/sync_bitops.h>
@@ -420,7 +421,7 @@ static int __must_check xen_allocate_irq_gsi(unsigned gsi)
 		return xen_allocate_irq_dynamic();
 
 	/* Legacy IRQ descriptors are already allocated by the arch. */
-	if (gsi < NR_IRQS_LEGACY)
+	if (gsi < nr_legacy_irqs())
 		irq = gsi;
 	else
 		irq = irq_alloc_desc_at(gsi, -1);
@@ -446,7 +447,7 @@ static void xen_free_irq(unsigned irq)
 	kfree(info);
 
 	/* Legacy IRQ descriptors are managed by the arch. */
-	if (irq < NR_IRQS_LEGACY)
+	if (irq < nr_legacy_irqs())
 		return;
 
 	irq_free_desc(irq);
-- 
2.1.0

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

[toc] | [next] | [standalone]

#1272953 — Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

FromJuergen Gross <jgross@suse.com>
Date2015-11-19 10:20 +0100
SubjectRe: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests
Message-ID<qwtxL-7n4-7@gated-at.bofh.it>
In reply to#1272446
On 18/11/15 19:14, Boris Ostrovsky wrote:
> After commit 8c058b0b9c34 ("x86/irq: Probe for PIC presence before
> allocating descs for legacy IRQs") early_irq_init() will no longer
> preallocate descriptors for legacy interrupts if PIC does not
> exist, which is the case for Xen PV guests.
> 
> Therefore we may need to allocate those descriptors ourselves.
> 
> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Suggested-by: Thomas Gleixner <tglx@linutronix.de>

Tested-by: Juergen Gross <jgross@suse.com>


Juergen

> ---
> 
> v2: Use nr_legacy_irqs() instead of NR_IRQS_LEGACY (needs definition for ARM)
> 
>  arch/arm/include/asm/irq.h       | 4 ++++
>  drivers/xen/events/events_base.c | 5 +++--
>  2 files changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/include/asm/irq.h b/arch/arm/include/asm/irq.h
> index be1d07d..b864f60 100644
> --- a/arch/arm/include/asm/irq.h
> +++ b/arch/arm/include/asm/irq.h
> @@ -2,6 +2,10 @@
>  #define __ASM_ARM_IRQ_H
>  
>  #define NR_IRQS_LEGACY	16
> +static inline int nr_legacy_irqs(void)
> +{
> +	return NR_IRQS_LEGACY;
> +}
>  
>  #ifndef CONFIG_SPARSE_IRQ
>  #include <mach/irqs.h>
> diff --git a/drivers/xen/events/events_base.c b/drivers/xen/events/events_base.c
> index 849500e..524c221 100644
> --- a/drivers/xen/events/events_base.c
> +++ b/drivers/xen/events/events_base.c
> @@ -39,6 +39,7 @@
>  #include <asm/irq.h>
>  #include <asm/idle.h>
>  #include <asm/io_apic.h>
> +#include <asm/i8259.h>
>  #include <asm/xen/pci.h>
>  #endif
>  #include <asm/sync_bitops.h>
> @@ -420,7 +421,7 @@ static int __must_check xen_allocate_irq_gsi(unsigned gsi)
>  		return xen_allocate_irq_dynamic();
>  
>  	/* Legacy IRQ descriptors are already allocated by the arch. */
> -	if (gsi < NR_IRQS_LEGACY)
> +	if (gsi < nr_legacy_irqs())
>  		irq = gsi;
>  	else
>  		irq = irq_alloc_desc_at(gsi, -1);
> @@ -446,7 +447,7 @@ static void xen_free_irq(unsigned irq)
>  	kfree(info);
>  
>  	/* Legacy IRQ descriptors are managed by the arch. */
> -	if (irq < NR_IRQS_LEGACY)
> +	if (irq < nr_legacy_irqs())
>  		return;
>  
>  	irq_free_desc(irq);
> 

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

[toc] | [prev] | [next] | [standalone]

#1273989 — Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

FromStefano Stabellini <stefano.stabellini@eu.citrix.com>
Date2015-11-20 12:30 +0100
SubjectRe: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests
Message-ID<qwS38-6xE-1@gated-at.bofh.it>
In reply to#1272446
On Wed, 18 Nov 2015, Boris Ostrovsky wrote:
> After commit 8c058b0b9c34 ("x86/irq: Probe for PIC presence before
> allocating descs for legacy IRQs") early_irq_init() will no longer
> preallocate descriptors for legacy interrupts if PIC does not
> exist, which is the case for Xen PV guests.
> 
> Therefore we may need to allocate those descriptors ourselves.
> 
> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Suggested-by: Thomas Gleixner <tglx@linutronix.de>

You need to do this for arm64 too.

FYI you can download arm64 cross-compilers from
https://releases.linaro.org/14.04/components/toolchain/binaries/
then you can use them by exporting ARCH=arm64 and
CROSS_COMPILE=/path/to/gcc-linaro-aarch64-linux-gnu-version/bin/aarch64-linux-gnu-


> v2: Use nr_legacy_irqs() instead of NR_IRQS_LEGACY (needs definition for ARM)
> 
>  arch/arm/include/asm/irq.h       | 4 ++++
>  drivers/xen/events/events_base.c | 5 +++--
>  2 files changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/include/asm/irq.h b/arch/arm/include/asm/irq.h
> index be1d07d..b864f60 100644
> --- a/arch/arm/include/asm/irq.h
> +++ b/arch/arm/include/asm/irq.h
> @@ -2,6 +2,10 @@
>  #define __ASM_ARM_IRQ_H
>  
>  #define NR_IRQS_LEGACY	16
> +static inline int nr_legacy_irqs(void)
> +{
> +	return NR_IRQS_LEGACY;
> +}
>  
>  #ifndef CONFIG_SPARSE_IRQ
>  #include <mach/irqs.h>
> diff --git a/drivers/xen/events/events_base.c b/drivers/xen/events/events_base.c
> index 849500e..524c221 100644
> --- a/drivers/xen/events/events_base.c
> +++ b/drivers/xen/events/events_base.c
> @@ -39,6 +39,7 @@
>  #include <asm/irq.h>
>  #include <asm/idle.h>
>  #include <asm/io_apic.h>
> +#include <asm/i8259.h>
>  #include <asm/xen/pci.h>
>  #endif
>  #include <asm/sync_bitops.h>
> @@ -420,7 +421,7 @@ static int __must_check xen_allocate_irq_gsi(unsigned gsi)
>  		return xen_allocate_irq_dynamic();
>  
>  	/* Legacy IRQ descriptors are already allocated by the arch. */
> -	if (gsi < NR_IRQS_LEGACY)
> +	if (gsi < nr_legacy_irqs())
>  		irq = gsi;
>  	else
>  		irq = irq_alloc_desc_at(gsi, -1);
> @@ -446,7 +447,7 @@ static void xen_free_irq(unsigned irq)
>  	kfree(info);
>  
>  	/* Legacy IRQ descriptors are managed by the arch. */
> -	if (irq < NR_IRQS_LEGACY)
> +	if (irq < nr_legacy_irqs())
>  		return;
>  
>  	irq_free_desc(irq);
> -- 
> 2.1.0
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1274171 — Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

FromBoris Ostrovsky <boris.ostrovsky@oracle.com>
Date2015-11-20 16:20 +0100
SubjectRe: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests
Message-ID<qwVDI-uA-29@gated-at.bofh.it>
In reply to#1273989
On 11/20/2015 06:24 AM, Stefano Stabellini wrote:
> On Wed, 18 Nov 2015, Boris Ostrovsky wrote:
>> After commit 8c058b0b9c34 ("x86/irq: Probe for PIC presence before
>> allocating descs for legacy IRQs") early_irq_init() will no longer
>> preallocate descriptors for legacy interrupts if PIC does not
>> exist, which is the case for Xen PV guests.
>>
>> Therefore we may need to allocate those descriptors ourselves.
>>
>> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
>> Suggested-by: Thomas Gleixner <tglx@linutronix.de>
> You need to do this for arm64 too.
>
> FYI you can download arm64 cross-compilers from
> https://releases.linaro.org/14.04/components/toolchain/binaries/
> then you can use them by exporting ARCH=arm64 and
> CROSS_COMPILE=/path/to/gcc-linaro-aarch64-linux-gnu-version/bin/aarch64-linux-gnu-

I did test on arm but didn't realize that arm64 uses different include 
files.

Does arm64 not have any legacy interrupts?

BTW, I got this build error:

   STUBCPY drivers/firmware/efi/libstub/lib-sort.stub.o
0000000000000000 R_AARCH64_ABS64   __efistub_sort
0000000000000008 R_AARCH64_ABS64   .init__ksymtab_strings
drivers/firmware/efi/libstub/lib-sort.stub.o: absolute symbol references 
not allowed in the EFI stub
drivers/firmware/efi/libstub/Makefile:63: recipe for target 
'drivers/firmware/efi/libstub/lib-sort.stub.o' failed
make[4]: *** [drivers/firmware/efi/libstub/lib-sort.stub.o] Error 1
scripts/Makefile.build:403: recipe for target 
'drivers/firmware/efi/libstub' failed
make[3]: *** [drivers/firmware/efi/libstub] Error 2
scripts/Makefile.build:403: recipe for target 'drivers/firmware/efi' failed
make[2]: *** [drivers/firmware/efi] Error 2
scripts/Makefile.build:403: recipe for target 'drivers/firmware' failed
make[1]: *** [drivers/firmware] Error 2
Makefile:943: recipe for target 'drivers' failed
make: *** [drivers] Error 2

I worked around this by removing some of EFI config options but 
presumably this needs to be properly fixed.

-boris


>
>
>> v2: Use nr_legacy_irqs() instead of NR_IRQS_LEGACY (needs definition for ARM)
>>
>>   arch/arm/include/asm/irq.h       | 4 ++++
>>   drivers/xen/events/events_base.c | 5 +++--
>>   2 files changed, 7 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/include/asm/irq.h b/arch/arm/include/asm/irq.h
>> index be1d07d..b864f60 100644
>> --- a/arch/arm/include/asm/irq.h
>> +++ b/arch/arm/include/asm/irq.h
>> @@ -2,6 +2,10 @@
>>   #define __ASM_ARM_IRQ_H
>>   
>>   #define NR_IRQS_LEGACY	16
>> +static inline int nr_legacy_irqs(void)
>> +{
>> +	return NR_IRQS_LEGACY;
>> +}
>>   
>>   #ifndef CONFIG_SPARSE_IRQ
>>   #include <mach/irqs.h>
>> diff --git a/drivers/xen/events/events_base.c b/drivers/xen/events/events_base.c
>> index 849500e..524c221 100644
>> --- a/drivers/xen/events/events_base.c
>> +++ b/drivers/xen/events/events_base.c
>> @@ -39,6 +39,7 @@
>>   #include <asm/irq.h>
>>   #include <asm/idle.h>
>>   #include <asm/io_apic.h>
>> +#include <asm/i8259.h>
>>   #include <asm/xen/pci.h>
>>   #endif
>>   #include <asm/sync_bitops.h>
>> @@ -420,7 +421,7 @@ static int __must_check xen_allocate_irq_gsi(unsigned gsi)
>>   		return xen_allocate_irq_dynamic();
>>   
>>   	/* Legacy IRQ descriptors are already allocated by the arch. */
>> -	if (gsi < NR_IRQS_LEGACY)
>> +	if (gsi < nr_legacy_irqs())
>>   		irq = gsi;
>>   	else
>>   		irq = irq_alloc_desc_at(gsi, -1);
>> @@ -446,7 +447,7 @@ static void xen_free_irq(unsigned irq)
>>   	kfree(info);
>>   
>>   	/* Legacy IRQ descriptors are managed by the arch. */
>> -	if (irq < NR_IRQS_LEGACY)
>> +	if (irq < nr_legacy_irqs())
>>   		return;
>>   
>>   	irq_free_desc(irq);
>> -- 
>> 2.1.0
>>

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

[toc] | [prev] | [next] | [standalone]

#1274189 — Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

FromStefano Stabellini <stefano.stabellini@eu.citrix.com>
Date2015-11-20 16:40 +0100
SubjectRe: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests
Message-ID<qwVX4-Bk-15@gated-at.bofh.it>
In reply to#1274171
On Fri, 20 Nov 2015, Boris Ostrovsky wrote:
> On 11/20/2015 06:24 AM, Stefano Stabellini wrote:
> > On Wed, 18 Nov 2015, Boris Ostrovsky wrote:
> > > After commit 8c058b0b9c34 ("x86/irq: Probe for PIC presence before
> > > allocating descs for legacy IRQs") early_irq_init() will no longer
> > > preallocate descriptors for legacy interrupts if PIC does not
> > > exist, which is the case for Xen PV guests.
> > > 
> > > Therefore we may need to allocate those descriptors ourselves.
> > > 
> > > Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> > > Suggested-by: Thomas Gleixner <tglx@linutronix.de>
> > You need to do this for arm64 too.
> > 
> > FYI you can download arm64 cross-compilers from
> > https://releases.linaro.org/14.04/components/toolchain/binaries/
> > then you can use them by exporting ARCH=arm64 and
> > CROSS_COMPILE=/path/to/gcc-linaro-aarch64-linux-gnu-version/bin/aarch64-linux-gnu-
> 
> I did test on arm but didn't realize that arm64 uses different include files.
> 
> Does arm64 not have any legacy interrupts?
> 
> BTW, I got this build error:
> 
>   STUBCPY drivers/firmware/efi/libstub/lib-sort.stub.o
> 0000000000000000 R_AARCH64_ABS64   __efistub_sort
> 0000000000000008 R_AARCH64_ABS64   .init__ksymtab_strings
> drivers/firmware/efi/libstub/lib-sort.stub.o: absolute symbol references not
> allowed in the EFI stub
> drivers/firmware/efi/libstub/Makefile:63: recipe for target
> 'drivers/firmware/efi/libstub/lib-sort.stub.o' failed
> make[4]: *** [drivers/firmware/efi/libstub/lib-sort.stub.o] Error 1
> scripts/Makefile.build:403: recipe for target 'drivers/firmware/efi/libstub'
> failed
> make[3]: *** [drivers/firmware/efi/libstub] Error 2
> scripts/Makefile.build:403: recipe for target 'drivers/firmware/efi' failed
> make[2]: *** [drivers/firmware/efi] Error 2
> scripts/Makefile.build:403: recipe for target 'drivers/firmware' failed
> make[1]: *** [drivers/firmware] Error 2
> Makefile:943: recipe for target 'drivers' failed
> make: *** [drivers] Error 2
> 
> I worked around this by removing some of EFI config options but presumably
> this needs to be properly fixed.

Interesting. This build error could be something new. Could you please
post your kernel config?


> > > v2: Use nr_legacy_irqs() instead of NR_IRQS_LEGACY (needs definition for
> > > ARM)
> > > 
> > >   arch/arm/include/asm/irq.h       | 4 ++++
> > >   drivers/xen/events/events_base.c | 5 +++--
> > >   2 files changed, 7 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/arch/arm/include/asm/irq.h b/arch/arm/include/asm/irq.h
> > > index be1d07d..b864f60 100644
> > > --- a/arch/arm/include/asm/irq.h
> > > +++ b/arch/arm/include/asm/irq.h
> > > @@ -2,6 +2,10 @@
> > >   #define __ASM_ARM_IRQ_H
> > >     #define NR_IRQS_LEGACY	16
> > > +static inline int nr_legacy_irqs(void)
> > > +{
> > > +	return NR_IRQS_LEGACY;
> > > +}
> > >     #ifndef CONFIG_SPARSE_IRQ
> > >   #include <mach/irqs.h>
> > > diff --git a/drivers/xen/events/events_base.c
> > > b/drivers/xen/events/events_base.c
> > > index 849500e..524c221 100644
> > > --- a/drivers/xen/events/events_base.c
> > > +++ b/drivers/xen/events/events_base.c
> > > @@ -39,6 +39,7 @@
> > >   #include <asm/irq.h>
> > >   #include <asm/idle.h>
> > >   #include <asm/io_apic.h>
> > > +#include <asm/i8259.h>
> > >   #include <asm/xen/pci.h>
> > >   #endif
> > >   #include <asm/sync_bitops.h>
> > > @@ -420,7 +421,7 @@ static int __must_check xen_allocate_irq_gsi(unsigned
> > > gsi)
> > >   		return xen_allocate_irq_dynamic();
> > >     	/* Legacy IRQ descriptors are already allocated by the arch.
> > > */
> > > -	if (gsi < NR_IRQS_LEGACY)
> > > +	if (gsi < nr_legacy_irqs())
> > >   		irq = gsi;
> > >   	else
> > >   		irq = irq_alloc_desc_at(gsi, -1);
> > > @@ -446,7 +447,7 @@ static void xen_free_irq(unsigned irq)
> > >   	kfree(info);
> > >     	/* Legacy IRQ descriptors are managed by the arch. */
> > > -	if (irq < NR_IRQS_LEGACY)
> > > +	if (irq < nr_legacy_irqs())
> > >   		return;
> > >     	irq_free_desc(irq);
> > > -- 
> > > 2.1.0
> > > 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1274238 — Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

FromStefano Stabellini <stefano.stabellini@eu.citrix.com>
Date2015-11-20 17:40 +0100
SubjectRe: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests
Message-ID<qwWT8-1dJ-23@gated-at.bofh.it>
In reply to#1274189
On Fri, 20 Nov 2015, Boris Ostrovsky wrote:
> On 11/20/2015 10:36 AM, Stefano Stabellini wrote:
> > On Fri, 20 Nov 2015, Boris Ostrovsky wrote:
> > > On 11/20/2015 06:24 AM, Stefano Stabellini wrote:
> > > > On Wed, 18 Nov 2015, Boris Ostrovsky wrote:
> > > > > After commit 8c058b0b9c34 ("x86/irq: Probe for PIC presence before
> > > > > allocating descs for legacy IRQs") early_irq_init() will no longer
> > > > > preallocate descriptors for legacy interrupts if PIC does not
> > > > > exist, which is the case for Xen PV guests.
> > > > > 
> > > > > Therefore we may need to allocate those descriptors ourselves.
> > > > > 
> > > > > Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> > > > > Suggested-by: Thomas Gleixner <tglx@linutronix.de>
> > > > You need to do this for arm64 too.
> > > > 
> > > > FYI you can download arm64 cross-compilers from
> > > > https://releases.linaro.org/14.04/components/toolchain/binaries/
> > > > then you can use them by exporting ARCH=arm64 and
> > > > CROSS_COMPILE=/path/to/gcc-linaro-aarch64-linux-gnu-version/bin/aarch64-linux-gnu-
> > > I did test on arm but didn't realize that arm64 uses different include
> > > files.
> > > 
> > > Does arm64 not have any legacy interrupts?
> > > 
> > > BTW, I got this build error:
> > > 
> > >    STUBCPY drivers/firmware/efi/libstub/lib-sort.stub.o
> > > 0000000000000000 R_AARCH64_ABS64   __efistub_sort
> > > 0000000000000008 R_AARCH64_ABS64   .init__ksymtab_strings
> > > drivers/firmware/efi/libstub/lib-sort.stub.o: absolute symbol references
> > > not
> > > allowed in the EFI stub
> > > drivers/firmware/efi/libstub/Makefile:63: recipe for target
> > > 'drivers/firmware/efi/libstub/lib-sort.stub.o' failed
> > > make[4]: *** [drivers/firmware/efi/libstub/lib-sort.stub.o] Error 1
> > > scripts/Makefile.build:403: recipe for target
> > > 'drivers/firmware/efi/libstub'
> > > failed
> > > make[3]: *** [drivers/firmware/efi/libstub] Error 2
> > > scripts/Makefile.build:403: recipe for target 'drivers/firmware/efi'
> > > failed
> > > make[2]: *** [drivers/firmware/efi] Error 2
> > > scripts/Makefile.build:403: recipe for target 'drivers/firmware' failed
> > > make[1]: *** [drivers/firmware] Error 2
> > > Makefile:943: recipe for target 'drivers' failed
> > > make: *** [drivers] Error 2
> > > 
> > > I worked around this by removing some of EFI config options but presumably
> > > this needs to be properly fixed.
> > Interesting. This build error could be something new. Could you please
> > post your kernel config?
> 
> Attached.
> 
> This, BTW, is on a relatively old toolstack:
> gcc version 4.8.2 20131014 (prerelease) (crosstool-NG
> linaro-1.13.1-4.8-2013.10 - Linaro GCC 2013.10)

Interesting. I cannot repro the error with Linaro GCC 4.9-2014.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1274260 — Re: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests

FromBoris Ostrovsky <boris.ostrovsky@oracle.com>
Date2015-11-20 18:00 +0100
SubjectRe: [PATCH v2] xen/events: Always allocate legacy interrupts on PV guests
Message-ID<qwXct-1li-1@gated-at.bofh.it>
In reply to#1274238
On 11/20/2015 11:33 AM, Stefano Stabellini wrote:
>>>
>>>
>>> BTW, I got this build error:
>>>
>>>     STUBCPY drivers/firmware/efi/libstub/lib-sort.stub.o
>>> 0000000000000000 R_AARCH64_ABS64   __efistub_sort
>>> 0000000000000008 R_AARCH64_ABS64   .init__ksymtab_strings
>>> drivers/firmware/efi/libstub/lib-sort.stub.o: absolute symbol references
>>> not
>>> allowed in the EFI stub
>>> drivers/firmware/efi/libstub/Makefile:63: recipe for target
>>> 'drivers/firmware/efi/libstub/lib-sort.stub.o' failed
>>> make[4]: *** [drivers/firmware/efi/libstub/lib-sort.stub.o] Error 1
>>> scripts/Makefile.build:403: recipe for target
>>> 'drivers/firmware/efi/libstub'
>>> failed
>>> make[3]: *** [drivers/firmware/efi/libstub] Error 2
>>> scripts/Makefile.build:403: recipe for target 'drivers/firmware/efi'
>>> failed
>>> make[2]: *** [drivers/firmware/efi] Error 2
>>> scripts/Makefile.build:403: recipe for target 'drivers/firmware' failed
>>> make[1]: *** [drivers/firmware] Error 2
>>> Makefile:943: recipe for target 'drivers' failed
>>> make: *** [drivers] Error 2
>>>
>>> I worked around this by removing some of EFI config options but presumably
>>> this needs to be properly fixed.
>>> Interesting. This build error could be something new. Could you please
>>> post your kernel config?
>> Attached.
>>
>> This, BTW, is on a relatively old toolstack:
>> gcc version 4.8.2 20131014 (prerelease) (crosstool-NG
>> linaro-1.13.1-4.8-2013.10 - Linaro GCC 2013.10)
> Interesting. I cannot repro the error with Linaro GCC 4.9-2014.

This does look like something toolstack-specific: I installed
     gcc version 4.8.3 20140401 (prerelease) (crosstool-NG 
linaro-1.13.1-4.8-2014.04 - Linaro GCC 4.8-2014.04)
from the link you provided above and it built without problems.

-boris

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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web