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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 114BAC433FE for ; Mon, 3 Oct 2022 23:37:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229526AbiJCXh0 (ORCPT ); Mon, 3 Oct 2022 19:37:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38574 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229596AbiJCXhY (ORCPT ); Mon, 3 Oct 2022 19:37:24 -0400 Received: from zeniv.linux.org.uk (zeniv.linux.org.uk [IPv6:2a03:a000:7:0:5054:ff:fe1c:15ff]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 39BB43AB16; Mon, 3 Oct 2022 16:37:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=linux.org.uk; s=zeniv-20220401; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=QaXhKfiYb2wJS1cY8/i43OfjDz29G/pG08gdTRtzOvY=; b=Ya1R14PzsJf4UAzejQPot5jf68 Guea09dRURyQZwSWLqVaBddRpqh+b0OYZADH+u9t91s21U6uzjdsG9hIusf6+yKM+LsFHMj7vs1YD qPhimBkoqG/udxerNZl/gWoyF9yNNKkQgiskgQrhLfzXCvIR/N55sfunr9NZlJ98O6VyBLoeTcHz3 C/ZDOJqu6+d7V+P2DWDOJ2kurAuo0EgM9EsMShYLsR680uPBv/pGNmte9Wmkwww8+2k8znJai+sqq Ugrhh4AJ9U0Mr3eA81mxbtO+dYYsnqtxEBkAHljAdNKf/FGcYjLLW6KTh3SZrMUrc94rCXfHqee0p oW3ozh9w==; Received: from viro by zeniv.linux.org.uk with local (Exim 4.96 #2 (Red Hat Linux)) id 1ofUzs-006ePP-31; Mon, 03 Oct 2022 23:37:17 +0000 Date: Tue, 4 Oct 2022 00:37:16 +0100 From: Al Viro To: "J. R. Okajima" Cc: Ira Weiny , Linus Torvalds , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH][CFT] [coredump] don't use __kernel_write() on kmap_local_page() Message-ID: References: <7714.1664794108@jrobl> <4011.1664837894@jrobl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4011.1664837894@jrobl> Sender: Al Viro Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Tue, Oct 04, 2022 at 07:58:14AM +0900, J. R. Okajima wrote: > Al Viro: > > Argh.... Try this: > > > > fix coredump breakage caused by badly tested "[coredump] don't use __kernel_write() on kmap_local_page()" > > Thanx, it passed my local test. > > > > * fix for problem that occurs on rather uncommon setups (and hadn't > > been observed in the wild) sent very late in the cycle. > > If the commit was merged in RC versions, I guess someone found the > problem earlier. Most likely - the breakage is really not hard to trigger ;-/ Linus, which way would you prefer to handle that? It's a brown paperbag stuff - the worst I had in quite a while. Mea maxima culpa ;-/ One variant would be to revert the original patch, put its (hopefully) fixed variant into -next and let it sit there for a while. Another is to put this incremental into -next and merge it into mainline once it gets a sane amount of testing. Up to you...