Date: Mon, 24 Sep 2007 10:49:40 -0700
From: Randy Dunlap <randy.dunlap@oracle.com>
To: Jonathan Campbell <jon@nerdgrounds.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Patches for tiny 386 kernels, again. Linux kernel
 2.6.22.7
Message-Id: <20070924104940.50a52cf3.randy.dunlap@oracle.com>
In-Reply-To: <46F77A85.6020202@nerdgrounds.com>
References: <46F77A85.6020202@nerdgrounds.com>
Organization: Oracle Linux Eng.
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2758
Lines: 81

On Mon, 24 Sep 2007 01:51:17 -0700 Jonathan Campbell wrote:

> Sorry for the long delay, been very busy since I last posted the 386 
> kernel patches back in July.
> Now that I have more free time I remade the patches in a cleaner manner, 
> broken down into
> smaller patches, with fewer #ifdefs all over the place. most #ifdefs are 
> in the include/asm-i386 headers
> now.

Regarding only the DMI-disable patch:

> linux-2.6.22.7-embedded-dmi-disable-option.patch:
>     * Allows you to remove DMI (Desktop Management Interface) parsing. 
> Older computers do
>        not have DMI structures in the BIOS, on these computers the DMI 
> code is a waste of space.
>        In my tests with applying this patch and disabling DMI removes 
> 6KB from
>        arch/i386/boot/bzImage after compile. Apparently there is already 
> a CONFIG_DMI out there,
>        and #ifdef's in some parts to check this, so all this patch does 
> is make CONFIG_DMI visible
>        from "make menuconfig" while patching up other parts (like ACPI) 
> that simply assume DMI
>        functions are there. Depends on CONFIG_EMBEDDED.


This seems reasonable, so I tried to use it.  Here are the results
and comments and meta-comments.


1.  Please forcibly wrap text lines in mail body at around column 70-72.

2.  Put patches inline in the mail body, not as attachments.

3.  Make patches against current mainline (e.g., 2.6.23-rc7), not
    against the stable tree.  Mainline is where someone would end
    up applying them.

4.  The DMI patch does not build against current mainline.  It needs
    the additional patch below (at end of mail).

5.  The Kconfig part of the DMI patch contains a trailing space.
    No brownie today.

6.  The Kconfig entry for "config DMI" should not be listed under
    "Processor types and features".  It should just go into the
    main EMBEDDED menu:

menuconfig EMBEDDED
	bool "Configure standard kernel features (for small systems)"


After all of these fixes, I can ack the patch.

---

From: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
---
 drivers/acpi/thermal.c |    2 ++
 1 file changed, 2 insertions(+)

--- linux-2.6.23-rc7.orig/drivers/acpi/thermal.c
+++ linux-2.6.23-rc7/drivers/acpi/thermal.c
@@ -1440,7 +1440,9 @@ static int __init acpi_thermal_init(void
 {
 	int result = 0;
 
+#ifdef CONFIG_DMI
 	dmi_check_system(thermal_dmi_table);
+#endif /* CONFIG_DMI */
 
 	if (off) {
 		printk(KERN_NOTICE "ACPI: thermal control disabled\n");
-
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/