Re: [PATCH 2.6] Watchdog timer for Intel IXP4xx CPUs

From: Marc Singer
Date: Tue May 11 2004 - 17:53:49 EST


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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/