From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 47BE2C433ED for ; Mon, 17 May 2021 21:55:24 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id CAEE8610CC for ; Mon, 17 May 2021 21:55:23 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CAEE8610CC Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=8ZSja9pSYK3B84mgKBhHwCcgbl/BrAMelLViynpSMDQ=; b=RT7EeDr7yRpieO/ayrmach9UT UCUGWJvM3nHdDmVJ6DMUeSF2+tQslJz8NOfS2yKazAdaXtAt1G8upYXhHC+jiX1FuDx7ecwJJFiEb TNBULrT/kWWkogsfED23mGgi/LqfbOjndnlGKZ+QFaJSDy015mtUAGFGXPSePEUNH7i+Jrh4ArEw8 0XJdnznh6q/A6/U7+SyDfXoosP7s8dKhWumKjbLStHY/nRNP0hKdDEEV7EunjSGHCZGlxDIkenef+ mryhZIxTrr/f4mb+dFfWAHertUp2RoRI9MS0+4/MBCP9c0rJhSfz+63FoTrPKdBwe5ePeqqJxIzQG Rfbr4UlRQ==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lilBA-00GFrX-4O; Mon, 17 May 2021 21:53:36 +0000 Received: from bombadil.infradead.org ([2607:7c80:54:e::133]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lilAs-00GFqD-OG for linux-arm-kernel@desiato.infradead.org; Mon, 17 May 2021 21:53:21 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=yrZZ6HjC2sXusjwyEaw1ab5WRux9vonr+ihAf85Qlxo=; b=B7GXQFYufZROn4Nvp2dqMZPl2B R/w8wnJaM37/5AssXDajAK4MdRFNic1/Y0C8TSIybDUtFPaDoC5nzKQN7YAEDzks7S2ars/V1OyAd 5KUpDmFns9MM8m8OvRceZZ2CMpESheyapIe1IIojB5XkfN/ZTXbYqK6dLBARMMxtAaNk7W6++3/Ib VraNPWFZCf3bIs6q0u0VOp4Qzwllm6Gdxef7rI1zo3+WUiHtYTin9oaZYmxOXT6k4/6Qz8Q6H5jCl XASrXsRQpW7/Rv0SyWtZ5tilEYczDjwhZ62CrehrxGOX62bQR4YwBYqfksH+PEAtngtoKdcaa+p0f TVUqaN7w==; Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lilAq-00EAiT-1t for linux-arm-kernel@lists.infradead.org; Mon, 17 May 2021 21:53:17 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 4A091610CB; Mon, 17 May 2021 21:53:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1621288395; bh=U9deKdKok+VzFOKlTKX/QNSM6gu2QbQroTH/5xZ/7fQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=aP84ny8Fs/au6FOc58Vbu8tzwXecGsPkJXIznd2YNXehsIGFUzIDud/9eX62S28hT XcBwDm/MTKgFYwXN4XpIRlVr9PCez/oLptRcnWPJyfMB1cwHni+LUs6lfDGY+8IuIj RStmlVeD0jwN2Tmwg3977fVIZjPXm8be2N4pWHHt6DLAF7XmDj2KrZqSdJ1Qr1abK/ WdQ65OqPMXbw9jvdxjlLcAlU2N5ulOt9R6HyODKNsMR8dRU2y7UV7Pbj2sWsQLG7Pk FhyDKPr3JQTro1a4HjiJD/zrYGToQg/ecdSArZN/J1BBsrYYEDrYE9dQb/VBcNviM3 osTbC5axoSHlA== Date: Mon, 17 May 2021 14:53:13 -0700 From: Eric Biggers To: Arnd Bergmann Cc: linux-arch@vger.kernel.org, Linus Torvalds , Vineet Gupta , Arnd Bergmann , Russell King , Herbert Xu , "David S. Miller" , Thomas Bogendoerfer , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-crypto@vger.kernel.org, linux-mips@vger.kernel.org Subject: Re: [PATCH v2 07/13] asm-generic: unaligned always use struct helpers Message-ID: References: <20210514100106.3404011-1-arnd@kernel.org> <20210514100106.3404011-8-arnd@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210514100106.3404011-8-arnd@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210517_145316_138615_AB3A1E34 X-CRM114-Status: GOOD ( 20.15 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, May 14, 2021 at 12:00:55PM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann > > As found by Vineet Gupta and Linus Torvalds, gcc has somewhat unexpected > behavior when faced with overlapping unaligned pointers. The kernel's > unaligned/access-ok.h header technically invokes undefined behavior > that happens to usually work on the architectures using it, but if the > compiler optimizes code based on the assumption that undefined behavior > doesn't happen, it can create output that actually causes data corruption. > > A related problem was previously found on 32-bit ARMv7, where most > instructions can be used on unaligned data, but 64-bit ldrd/strd causes > an exception. The workaround was to always use the unaligned/le_struct.h > helper instead of unaligned/access-ok.h, in commit 1cce91dfc8f7 ("ARM: > 8715/1: add a private asm/unaligned.h"). > > The same solution should work on all other architectures as well, so > remove the access-ok.h variant and use the other one unconditionally on > all architectures, picking either the big-endian or little-endian version. FYI, gcc 10 had a bug where it miscompiled code that uses "packed structs" to copy between overlapping unaligned pointers (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94994). I'm not sure whether the kernel will run into that or not, and gcc has since fixed it. But it's worth mentioning, especially since the issue mentioned in this commit sounds very similar (overlapping unaligned pointers), and both involved implementations of DEFLATE decompression. Anyway, partly due to the above, in userspace I now only use memcpy() to implement {get,put}_unaligned_*, since these days it seems to be compiled optimally and have the least amount of problems. I wonder if the kernel should do the same, or whether there are still cases where memcpy() isn't compiled optimally. armv6/7 used to be one such case, but it was fixed in gcc 6. - Eric _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel