From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH] misplaced parenthesis? Date: Thu, 25 Jun 2009 13:29:25 -0700 Message-ID: <20090625132925.982bc72d.akpm@linux-foundation.org> References: <4A0DC131.8020606@gmail.com> <4A0E3D95.5090404@tausq.org> <4A10BD1A.3060208@tausq.org> <1e3eba760906230753l4e14a025y6bc07a753999483e@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Cc: roel.kluin@gmail.com, kyle@mcmartin.ca, linux-parisc@vger.kernel.org To: Randolph Chung Return-path: In-Reply-To: <1e3eba760906230753l4e14a025y6bc07a753999483e@mail.gmail.com> List-ID: List-Id: linux-parisc.vger.kernel.org On Tue, 23 Jun 2009 22:53:26 +0800 Randolph Chung wrote: > Here's an updated patch: > > Reported-by: Roel Kluin > Signed-off-by: Randolph Chung > > diff --git a/arch/parisc/lib/memcpy.c b/arch/parisc/lib/memcpy.c > index 2d68431..827e535 100644 > --- a/arch/parisc/lib/memcpy.c > +++ b/arch/parisc/lib/memcpy.c > @@ -405,7 +405,7 @@ byte_copy: > > unaligned_copy: > /* possibly we are aligned on a word, but not on a double... */ > - if (likely(t1 & (sizeof(unsigned int)-1)) == 0) { > + if (likely((t1 & (sizeof(unsigned int)-1)) == 0)) { > t2 = src & (sizeof(unsigned int) - 1); > > if (unlikely(t2 != 0)) { > I think I'll delete this email. If someone has a patch which they think should be applied, please resend it with a changelog.