[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] don't use weak symbols on x86-64


  • To: Jan Beulich <JBeulich@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir@xxxxxxx>
  • Date: Fri, 07 Jan 2011 15:45:45 +0000
  • Cc:
  • Delivery-date: Fri, 07 Jan 2011 07:47:14 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:user-agent:date:subject:from:to:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=iGm+CIDe2n7UnG2jRDyHylzt9XcdjxsBi5o1gV8ocXSFiUBLA0eMznOe6x87pOZve8 6Rk9D3G4lSmM4VpbQtPAw/+SJXH23cum42PDx65L+npYAeK1iMlUvrqUdRmlJWIyWYmE VQysnO6KRxJDSGdX5UjkF/Gf3d9vBaQeSTmnw=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcuugfJXnfwk3oyYn0Os/KsjhRr2lg==
  • Thread-topic: [Xen-devel] [PATCH] don't use weak symbols on x86-64

We should just disallow use of weak definitions. Your patch plus removing
this one weak attribute achieves that just fine.

 -- Keir

On 07/01/2011 14:30, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:

> Various gcc versions inline functions that are both weak and hidden,
> without even giving a warning.
> 
> Certainly the risk exists that we'll see the problem again when
> another weak function gets introduced, but I don't see a way to
> protect us from that.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
> 
> --- a/xen/drivers/acpi/numa.c
> +++ b/xen/drivers/acpi/numa.c
> @@ -120,6 +120,8 @@ static int __init acpi_parse_slit(struct
> return 0;
>  }
>  
> +#ifndef CONFIG_X86 /* At least gcc 4.3 through 4.5 don't fully honor
> +        weak for functions with non-default visibility. */
>  void __init __attribute__ ((weak))
>  acpi_numa_x2apic_affinity_init(struct acpi_srat_x2apic_cpu_affinity *pa)
>  {
> @@ -127,7 +129,7 @@ acpi_numa_x2apic_affinity_init(struct ac
>       "Found unsupported x2apic [0x%08x] SRAT entry\n", pa->apic_id);
> return;
>  }
> -
> +#endif
>  
>  static int __init
>  acpi_parse_x2apic_affinity(struct acpi_subtable_header *header,
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.