From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Message-ID: <17545.16696.195276.334774@cargo.ozlabs.ibm.com> Date: Fri, 9 Jun 2006 19:36:56 +1000 From: Paul Mackerras To: mel@csn.ul.ie (Mel Gorman) Subject: Re: [PATCH] Compile failure fix for ppc on 2.6.17-rc4-mm3 (2nd attempt) In-Reply-To: <20060529190515.GA17608@skynet.ie> References: <20060526151214.GA5190@skynet.ie> <20060526094924.10efc515.akpm@osdl.org> <20060529154923.GA9025@skynet.ie> <2ebd96e4a7ea753273b2c5f856ba8c7a@kernel.crashing.org> <20060529190515.GA17608@skynet.ie> Cc: Andrew Morton , linuxppc-dev@ozlabs.org, vgoyal@in.ibm.com, linux-kernel@vger.kernel.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Mel Gorman writes: > + res->end = -(-res->end & ~(unsigned long)mask); \ > + res->end += mask; \ I think this is equivalent to res->end = (res->end + mask) | mask; and I have to say the latter seems more understandable to me (and doesn't need a cast) ... Regards, Paul.