From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965844AbcIHJrz (ORCPT ); Thu, 8 Sep 2016 05:47:55 -0400 Received: from ozlabs.org ([103.22.144.67]:55992 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965818AbcIHJrx (ORCPT ); Thu, 8 Sep 2016 05:47:53 -0400 X-powerpc-patch-notification: thanks X-powerpc-patch-commit: 8540571e01f973d321b0821f4f32ed6e9ae8263c In-Reply-To: <20160826144513.A71561A245B@localhost.localdomain> To: Christophe Leroy , Benjamin Herrenschmidt , Paul Mackerras , Scott Wood From: Michael Ellerman Cc: Antoine Blangy , Alessio Igor Bogani , linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: powerpc/32: fix again csum_partial_copy_generic() Message-Id: <3sVFr73rMsz9s3v@ozlabs.org> Date: Thu, 8 Sep 2016 19:47:51 +1000 (AEST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2016-26-08 at 14:45:13 UTC, Christophe Leroy wrote: > commit 7aef4136566b0 ("powerpc32: rewrite csum_partial_copy_generic() > based on copy_tofrom_user()") introduced a bug when destination > address is odd and len is lower than cacheline size. > > In that case the resulting csum value doesn't have to be rotated one > byte because the cache-aligned copy part is skipped so no alignment > is performed. > > Fixes: 7aef4136566b0 ("powerpc32: rewrite csum_partial_copy_generic() based on copy_tofrom_user()") > Cc: stable@vger.kernel.org > Reported-by: Alessio Igor Bogani > Signed-off-by: Christophe Leroy > Tested-by: Alessio Igor Bogani Applied to powerpc fixes, thanks. https://git.kernel.org/powerpc/c/8540571e01f973d321b0821f4f cheers