Skip to contentSkip to conversation

i386: unexport machine_id

PATCH In review Adrian Bunk 3 participants 3 messages Aug 27, 2007 – Aug 28, 2007

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 i386: unexport machine_id

This patch removes the unused EXPORT_SYMBOL(machine_id).

---

This patch has been sent on:
- 14 Aug 2007

arch/i386/kernel/setup.c +0 -3
@@ -86,9 +86,6 @@ unsigned long mmu_cr4_features;
 
 /* for MCA, but anyone else can use it if they want */
 unsigned int machine_id;
-#ifdef CONFIG_MCA
-EXPORT_SYMBOL(machine_id);
-#endif
 unsigned int machine_submodel_id;
 unsigned int BIOS_revision;
 unsigned int mca_pentium_flag;
#1 lore raw
Quoted 12 lines · On Mon, 27 Aug 2007 23:28:08 +0200 Adrian Bunk <[email protected]> wrote:

This patch removes the unused EXPORT_SYMBOL(machine_id).

---

This patch has been sent on:
- 14 Aug 2007

arch/i386/kernel/setup.c unsigned long mmu_cr4_features; · line 91

Quoted 9 lines
arch/i386/kernel/setup.c +0 -3
@@ -86,9 +86,6 @@ unsigned long mmu_cr4_features;
 
 /* for MCA, but anyone else can use it if they want */
 unsigned int machine_id;
-#ifdef CONFIG_MCA
-EXPORT_SYMBOL(machine_id);
-#endif
 unsigned int machine_submodel_id;
 unsigned int BIOS_revision;
 unsigned int mca_pentium_flag;

You know my position on removal of exports with no warning.

If the relevant maintainer chooses to apply the patch then ho hum I have other things to think about. Your best shot at getting this sort of thing merged would be to cc the appropriate maintainer.

Or, better, use __deprecated_for_modules. That's what it was added for.

#2 lore raw
Quoted 2 lines · On Tue, 28 Aug 2007 01:07:41 -0700 Andrew Morton <[email protected]>…

You know my position on removal of exports with no warning.

I know it (and don't quite agree but ok)

Quoted 7 lines

If the relevant maintainer chooses to apply the patch then ho hum I have other things to think about. Your best shot at getting this sort of thing merged would be to cc the appropriate maintainer.

Or, better, use __deprecated_for_modules. That's what it was added for. -

and/or use EXPORT_UNUSED_SYMBOL so that people at least have the option to not suffer the overhead *now*.