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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id E7EA5C433EF for ; Tue, 15 Feb 2022 13:17:12 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 2C5106B0078; Tue, 15 Feb 2022 08:17:12 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 275A36B007B; Tue, 15 Feb 2022 08:17:12 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 13DD16B007D; Tue, 15 Feb 2022 08:17:12 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0164.hostedemail.com [216.40.44.164]) by kanga.kvack.org (Postfix) with ESMTP id 04F516B0078 for ; Tue, 15 Feb 2022 08:17:12 -0500 (EST) Received: from smtpin23.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id B035F9277A for ; Tue, 15 Feb 2022 13:17:11 +0000 (UTC) X-FDA: 79145065062.23.C4D9CBF Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf05.hostedemail.com (Postfix) with ESMTP id 474A8100005 for ; Tue, 15 Feb 2022 13:17:11 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 4FEA261716 for ; Tue, 15 Feb 2022 13:17:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 883F9C340FE for ; Tue, 15 Feb 2022 13:17:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1644931028; bh=yV4ORJd5b6IZao4Zu1H2u9bAD/0nkvfeGMC8j+boEO0=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=Y1UbQDYKVd/W55uCfVo2WGlgo3dfDZUC0yTFtlZUY0DOBzt83pgV9/7gMxh05e3AH lgukd8DqmxPwxja7mK4vtMYnDC6X4AvagIoktla5uPG7vSSfPBtEQFLrXn9FMcdgHH E1qEJeHLjvJUuBL6dqkrwsbxdH4jW8atiF/scjZBMC9lLF7Ype1WGnPRnqUer24JD8 tu4gGKuwZtD0+Nibza1Ys9PPtyZzsqg7g90GGNebDRKc+NU4HRCxVmDtvM0+9S5mSP qhfZT5HIgJ8aWSTxW8hfUnfgxShZHe3y/AzzhEX87Jyr/ToKwZr4yc1EXWSuI0q6BQ u0/dxkrR5dDfA== Received: by mail-wm1-f49.google.com with SMTP id j9-20020a05600c190900b0037bff8a24ebso1423240wmq.4 for ; Tue, 15 Feb 2022 05:17:08 -0800 (PST) X-Gm-Message-State: AOAM533fcrbA0aLqab16Gq2lZRMBYyy+ql9iI/DU0V4YgMRQTpSZMtk5 788YJyG9D+1qy8A+RsIae6Gx0KPT/qyR7ggNAKs= X-Google-Smtp-Source: ABdhPJzhbiTSvbv50mQWSJ7smaLZnr2q978YJ0F6+MeQ3SrwEp5s0NrkvRE5+WnWUVexqYiFblH5E9BGGqHgxsxWGyI= X-Received: by 2002:a05:600c:4ecb:: with SMTP id g11mr3088436wmq.98.1644931026534; Tue, 15 Feb 2022 05:17:06 -0800 (PST) MIME-Version: 1.0 References: <20220214163452.1568807-1-arnd@kernel.org> <20220214163452.1568807-6-arnd@kernel.org> In-Reply-To: From: Arnd Bergmann Date: Tue, 15 Feb 2022 14:16:50 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 05/14] uaccess: add generic __{get,put}_kernel_nofault To: Al Viro Cc: Linus Torvalds , Christoph Hellwig , linux-arch , Linux-MM , Linux API , Arnd Bergmann , Linux Kernel Mailing List , Russell King - ARM Linux , Will Deacon , Guo Ren , Brian Cain , Geert Uytterhoeven , Michal Simek , Thomas Bogendoerfer , Nick Hu , Greentime Hu , Dinh Nguyen , Stafford Horne , Helge Deller , Michael Ellerman , Peter Zijlstra , Ingo Molnar , Mark Rutland , Heiko Carstens , Rich Felker , David Miller , Richard Weinberger , "the arch/x86 maintainers" , Max Filippov , "Eric W . Biederman" , Andrew Morton , Ard Biesheuvel , alpha , "open list:SYNOPSYS ARC ARCHITECTURE" , Linux ARM , linux-csky@vger.kernel.org, "open list:QUALCOMM HEXAGON..." , linux-ia64@vger.kernel.org, linux-m68k , "open list:BROADCOM NVRAM DRIVER" , Openrisc , Parisc List , linuxppc-dev , linux-riscv , linux-s390 , Linux-sh list , sparclinux , linux-um , "open list:TENSILICA XTENSA PORT (xtensa)" Content-Type: text/plain; charset="UTF-8" X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 474A8100005 X-Rspam-User: Authentication-Results: imf05.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=Y1UbQDYK; spf=pass (imf05.hostedemail.com: domain of arnd@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=arnd@kernel.org; dmarc=pass (policy=none) header.from=kernel.org X-Stat-Signature: k7nc7xo8bd7bjnjqmapx1rwjkcydfjgr X-HE-Tag: 1644931031-223052 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Tue, Feb 15, 2022 at 1:31 AM Al Viro wrote: > > On Mon, Feb 14, 2022 at 05:34:43PM +0100, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > All architectures that don't provide __{get,put}_kernel_nofault() yet > > can implement this on top of __{get,put}_user. > > > > Add a generic version that lets everything use the normal > > copy_{from,to}_kernel_nofault() code based on these, removing the last > > use of get_fs()/set_fs() from architecture-independent code. > > I'd put the list of those architectures (AFAICS, that's alpha, ia64, > microblaze, nds32, nios2, openrisc, sh, sparc32, xtensa) into commit > message - it's not that hard to find out, but... done. > And AFAICS, you've missed nios2 - see > #define __put_user(x, ptr) put_user(x, ptr) > in there. nds32 oddities are dealt with earlier in the series, this > one is not... Ok, fixed my bug in nios2 __put_user() as well now. This one is not nearly as bad as nds32, at least without my patches it should work as expected. Unfortunately I also noticed that __get_user() on microblaze and nios2 is completely broken for 64-bit arguments, where these copy eight bytes into a four byte buffer. I'll try to come up with a fix for this as well then. Arnd