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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6422E109E536 for ; Thu, 26 Mar 2026 05:19:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; 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=eUrdb2uU7dK3q8W5XxEPZpQ5Qgo+Gp1FR2OuwX7R0uI=; b=XPvGPA/EDDcJFd boQPJWENj27kLpLK2r74g/felcT25Oq9MNYV/+WK589bKe6As0KMu5MV44emBHALtiQglQebhuv+R CbynkbwnC15qRUKm91A5DT4gbqHiP4KwRfURNVbAGCkLDEdn8pvWqUdDlL4BCROvPUQJDu7qFI6Uo iiSa53xmAuPTZY+ISB/w3Ffc9S3jgUuXbEDcAtPoZZhPX3fQUYxBmG0KXYRMQ6/Au4FLr7EHvPsi5 oQzKjGLbUCwfGxRRlD8vBiRaII5rTlc7XFJLdsIxeQvvTHaDYYoS1PPB9Y8XhGeuD+HK+t9xTllyK mrM0vfL8KWDtzbJziOKA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1w5d7M-00000004jSF-0dZS; Thu, 26 Mar 2026 05:18:52 +0000 Received: from verein.lst.de ([213.95.11.211]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1w5d7J-00000004jQt-0rPP; Thu, 26 Mar 2026 05:18:51 +0000 Received: by verein.lst.de (Postfix, from userid 2407) id 7351F68B05; Thu, 26 Mar 2026 06:18:37 +0100 (CET) Date: Thu, 26 Mar 2026 06:18:37 +0100 From: Christoph Hellwig To: Eric Biggers Cc: Christoph Hellwig , Andrew Morton , Richard Henderson , Matt Turner , Magnus Lindholm , Russell King , Catalin Marinas , Will Deacon , Ard Biesheuvel , Huacai Chen , WANG Xuerui , Madhavan Srinivasan , Michael Ellerman , Nicholas Piggin , "Christophe Leroy (CS GROUP)" , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Ghiti , Heiko Carstens , Vasily Gorbik , Alexander Gordeev , Christian Borntraeger , Sven Schnelle , "David S. Miller" , Andreas Larsson , Richard Weinberger , Anton Ivanov , Johannes Berg , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Herbert Xu , Dan Williams , Chris Mason , David Sterba , Arnd Bergmann , Song Liu , Yu Kuai , Li Nan , Theodore Ts'o , "Jason A. Donenfeld" , linux-alpha@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, loongarch@lists.linux.dev, linuxppc-dev@lists.ozlabs.org, linux-riscv@lists.infradead.org, linux-s390@vger.kernel.org, sparclinux@vger.kernel.org, linux-um@lists.infradead.org, linux-crypto@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-arch@vger.kernel.org, linux-raid@vger.kernel.org Subject: Re: cleanup the RAID5 XOR library v3 Message-ID: <20260326051837.GA22847@lst.de> References: <20260324062211.3216301-1-hch@lst.de> <20260325193954.GC2305@quark> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20260325193954.GC2305@quark> User-Agent: Mutt/1.5.17 (2007-11-01) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260325_221849_395117_87A6AD10 X-CRM114-Status: GOOD ( 22.28 ) X-BeenThere: linux-riscv@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-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On Wed, Mar 25, 2026 at 12:39:54PM -0700, Eric Biggers wrote: > This generally looks good, but yes, please check the comments from > https://sashiko.dev/#/patchset/20260324062211.3216301-1-hch@lst.de, as > Andrew mentioned. Yes, I've looked into them and fixed the, the current version in the git branch already has the changes. > looks real as well, though I haven't tested it. If preemption is indeed > not the right thing to check, then I guess (following up from > https://lore.kernel.org/linux-crypto/20260303195517.GC2846@sol/) it > would need to be something like: > > WARN_ON_ONCE(!in_task() || irqs_disabled() || softirq_count() != 0); > > Ugly, but we're running out of options. So far I've just reverted back to the in_interrupted() check we had before. I can switch to the above, though. > (This sort of thing is why the functions in lib/crypto/ and lib/crc/ are > just supported in all contexts instead. If FPU/vector/SIMD registers > cannot be used in the current context, then a scalar fallback is used.) We could do this fairly easily, but I'm not sure it is a good idea. The callers of these routines are extremely limited, so we'd have to add code for this which will then only be used by the new extensive test code we'd have to add for it. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv