Messages in this thread |  | | | Date | Tue, 11 May 2004 15:48:11 -0700 | | From | Marc Singer <> | | Subject | Re: [PATCH 2.6] Watchdog timer for Intel IXP4xx CPUs |
| |
On Tue, May 11, 2004 at 06:37:32PM -0400, Jeff Garzik wrote: > Deepak Saxena wrote: > >On May 11 2004, at 14:33, Andrew Morton was caught saying: > >+#ifdef CONFIG_WATCHDOG_NOWAYOUT > >+static int nowayout = 1; > >+#else > >+static int nowayout = 0; > >+#endif > >+static int heartbeat = 60; /* (secs) Default is 1 minute */ > >+static unsigned long wdt_status = 0; > >+static unsigned long boot_status = 0; > > Wastes bss(?) space to explicitly zero statics.
BTW, BSS is the stuff that is zeroed automatically. Explicitly zeroing a variable is a waste of DATA and requires space in the executable. - 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/
|  |