lkml.org 
[lkml]   [2002]   [Aug]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [patch] reduce stack usage of sanitize_e820_map
On Sat, Aug 17, 2002 at 03:17:04PM -0400, Benjamin LaHaise wrote:
> Nope. static conflicts with __initdata. If namespace pollution is a
> concern, just prefix them with e820_.

Erm, foot-in-mouth. I'd tried adding __initdata within the function,
which conflicts with the auto attribute. Making it static within the
function and adding __initdata works here and has the same effect on
stack usage. Linus, could you apply this? Thanks.

-ben

--- foo-v2.5.31/arch/i386/kernel/setup.c Sat Aug 17 15:31:33 2002
+++ test-v2.5.31/arch/i386/kernel/setup.c Sat Aug 17 15:28:33 2002
@@ -275,17 +275,16 @@
* replaces the original e820 map with a new one, removing overlaps.
*
*/
-struct change_member {
- struct e820entry *pbios; /* pointer to original bios entry */
- unsigned long long addr; /* address for this change point */
-};
-struct change_member change_point_list[2*E820MAX] __initdata;
-struct change_member *change_point[2*E820MAX] __initdata;
-struct e820entry *overlap_list[E820MAX] __initdata;
-struct e820entry new_bios[E820MAX] __initdata;
-
static int __init sanitize_e820_map(struct e820entry * biosmap, char * pnr_map)
{
+ struct change_member {
+ struct e820entry *pbios; /* pointer to original bios entry */
+ unsigned long long addr; /* address for this change point */
+ };
+ static struct change_member change_point_list[2*E820MAX] __initdata;
+ static struct change_member *change_point[2*E820MAX] __initdata;
+ static struct e820entry *overlap_list[E820MAX] __initdata;
+ static struct e820entry new_bios[E820MAX] __initdata;
struct change_member *change_tmp;
unsigned long current_type, last_type;
unsigned long long last_addr;
-
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/
\
 
 \ /
  Last update: 2005-03-22 13:28    [W:0.043 / U:1.512 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site