Date: Sun, 2 Nov 2003 03:59:46 -0500 (EST)
From: Zwane Mwaikambo <zwane@arm.linux.org.uk>
To: Willy Tarreau <willy@w.ods.org>
cc: Geoffrey Lee <glee@gnupilgrims.org>, linux-kernel@vger.kernel.org,
       davej@redhat.com
Subject: Re: [patch] reproducible athlon mce fix
In-Reply-To: <20031102072519.GD530@alpha.home.local>
Message-ID: <Pine.LNX.4.53.0311020358080.32567@montezuma.fsmlabs.com>
References: <20031102055748.GA1218@anakin.wychk.org> <20031102072519.GD530@alpha.home.local>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 815
Lines: 37

On Sun, 2 Nov 2003, Willy Tarreau wrote:

> 1) a still readable one
> #if CONFIG_MK7
> 	i=1;
> #else
>  	i=0;
> #endif
>  	for (; i<nr_mce_banks; i++) {
> 
> 2) when there's absolutely no choice (eg changing part of an
>    expression...) something unreadable like this.
> 
> 	for (
> #if CONFIG_MK7
> 	i=1;
> #else
>  	i=0;
> #endif
>  	i<nr_mce_banks; i++) {
> 
> I'm sure other people have other solution that don't come to mind.

Perhaps..

#ifdef CONFIG_MK7
#define MCE_BANK_START	1
#else
#define MCE_BANK_START	0
#endif

for (i = MCE_BANK_START; nr_mce_banks; i++)
-
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/