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

["Jan Beulich" <JBeulich@xxxxxxxxxx>]

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,

Attachment: acpi-weak-vs-hidden.patch
Description: Text document

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