From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261293AbVF1LKT (ORCPT ); Tue, 28 Jun 2005 07:10:19 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261282AbVF1LKT (ORCPT ); Tue, 28 Jun 2005 07:10:19 -0400 Received: from smtp.osdl.org ([65.172.181.4]:63448 "EHLO smtp.osdl.org") by vger.kernel.org with ESMTP id S261293AbVF1LKJ (ORCPT ); Tue, 28 Jun 2005 07:10:09 -0400 Date: Tue, 28 Jun 2005 04:03:13 -0700 From: Andrew Morton To: "d binderman" Cc: linux-kernel@vger.kernel.org Subject: Re: Message-Id: <20050628040313.3c808093.akpm@osdl.org> In-Reply-To: References: X-Mailer: Sylpheed version 1.0.4 (GTK+ 1.2.10; i386-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org "d binderman" wrote: > > > Hello there, > > I just tried to compile the Linux Kernel version 2.6.11.12 > with the gcc 4.0 compiler. The compiler said > > drivers/net/depca.c:1829: warning: operation on 'i' may be undefined > > The source code is > > for (i = entry; i != end; i = (++i) & lp->txRingMask) { > > I agree with the compiler. Better code is > > for (i = entry; i != end; i = (i + 1) & lp->txRingMask) { > Someone already fixed it. /* set up the buffer descriptors */ len = (skb->len < ETH_ZLEN) ? ETH_ZLEN : skb->len; for (i = entry; i != end; i = (i+1) & lp->txRingMask) {