Received: by 2002:a05:7412:2a8c:b0:e2:908c:2ebd with SMTP id u12csp915686rdh; Sun, 24 Sep 2023 18:37:20 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHHrDuaWWPT7yWZXBnwPgld3naN36zVy45ogHayGNO5cCot9OQ1HsgePgIA0lrqBo4m30Qr X-Received: by 2002:a05:6102:34c3:b0:452:8ad4:29d9 with SMTP id a3-20020a05610234c300b004528ad429d9mr2838408vst.17.1695605840042; Sun, 24 Sep 2023 18:37:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1695605839; cv=none; d=google.com; s=arc-20160816; b=oCk6bed2xiS3qnSiP4qiqAnXTr/t+FHUgN9JIbVscvgQrYfqhW15HmY7lYXiCIC+kr k/g8vAcsc0pnziQP1v580K3uDF4/6N5Y8kQzf4B55XRZJIZ91XPt2O39WoXTFYE6KovR P7sLiyT2fVYpdNqBML9Y4J02wsYIIMnPy2pAthH3rGrX7wrQFvAAOfGtIMti7h6M+iBW 85muElsil30olUuKrTFs12oqaOV4zkyiSujKLLOoV2s1Ng2ZfNRjSLrboSDg/aIQTabk G9H81KPqLvz89oKE6ow+N8waZv/ZvZ3mrslI+tRid21wnOgngasFkzRbUAyV64jZOuAx rxgQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=TnVgiGgy2SXRnj7Ad/Tf1AV8uhnk5EUiaHEh6VTehqE=; fh=xHqVk3P6hqRLIrZ4OmIjG4gMgwZ8hRESlv486bH+r5s=; b=fJuNGCyMjkUz2ZWlZLyjEKwNpoZbndU+32R2AIrozuhbPCa7TKKq7FQBqa1qeN6fWy oLaufhMGM3DZQ034+ByttYRmIw/CCW4oo57++ffUbH3ytmn90/4S21UMl6S6HHuDdy6a Dao5YsDtq/IIb5gGguYMRLqZQVrRjwh5A7txHY8W7WXfOxymYC0GhYzUjAgrQvEajWv8 8xtZzoOSmizzFYA6zbSFiSTLECZDX/cYEy77g/YByti7eft4wXx0Da9KsDEbBO396IFL hGGD3s3uxZ//rd3aViDPsH5iE/WSc3cQA2dp2pTUBSrH0CIfSki00jzmqavMz+IWzDjD 933w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linux-foundation.org header.s=korg header.b=Fg5OGepy; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from morse.vger.email (morse.vger.email. [23.128.96.31]) by mx.google.com with ESMTPS id m9-20020a654389000000b0057828b85afdsi8692990pgp.795.2023.09.24.18.37.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 24 Sep 2023 18:37:19 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31; Authentication-Results: mx.google.com; dkim=pass header.i=@linux-foundation.org header.s=korg header.b=Fg5OGepy; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by morse.vger.email (Postfix) with ESMTP id 4ABF48022092; Sun, 24 Sep 2023 16:42:25 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at morse.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230070AbjIXXmK (ORCPT + 99 others); Sun, 24 Sep 2023 19:42:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33926 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229480AbjIXXmJ (ORCPT ); Sun, 24 Sep 2023 19:42:09 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 41534DA; Sun, 24 Sep 2023 16:42:03 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8C93BC433C7; Sun, 24 Sep 2023 23:42:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linux-foundation.org; s=korg; t=1695598922; bh=9ZF3IoLRXqGI/hpsI7yQNEugIBPhe+O8QyOuSML+8nw=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Fg5OGepykvga+q20Fne4dweBz0cYFIcjkDhMPdc4KzRZblBK7bdUJmMo+23HO5EyS 7qJSngXA2x0axtpqHRtlqkexCs40yhQRMWK32Fq6y+uOrZtbqEFABslR/ckWA3giXx lYJglvDcqZPQ4O4Ll/LpwmAYyfgtkuqUCoflLC9M= Date: Sun, 24 Sep 2023 16:42:01 -0700 From: Andrew Morton To: Stephen Rothwell Cc: Linux Kernel Mailing List , Linux Next Mailing List , Ryan Roberts Subject: Re: linux-next: Signed-off-bys missing for commit in the mm-hotfixes tree Message-Id: <20230924164201.863ad5e63a2d555f6d33254c@linux-foundation.org> In-Reply-To: <20230925075026.2a6b4e65@canb.auug.org.au> References: <20230925075026.2a6b4e65@canb.auug.org.au> X-Mailer: Sylpheed 3.8.0beta1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-2.3 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Sun, 24 Sep 2023 16:42:25 -0700 (PDT) On Mon, 25 Sep 2023 07:50:26 +1000 Stephen Rothwell wrote: > Hi all, > > Commit > > 48248e468270 ("mm: hugetlb: add huge page size param to set_huge_pte_at()") > > is missing Signed-off-bys from its author and committer. > > It looks like the commit message has been truncated. Ah, thanks. The changelog has a stray ^---$: : If CONFIG_DEBUG_VM is enabled, we do at least get a BUG(), but otherwise, : it will dereference a bad pointer in page_folio(): : : static inline struct folio *hugetlb_swap_entry_to_folio(swp_entry_t entry) : { : VM_BUG_ON(!is_migration_entry(entry) && !is_hwpoison_entry(entry)); : : return page_folio(pfn_to_page(swp_offset_pfn(entry))); : } : : : Fix : --- : : The simplest fix would have been to revert the dodgy cleanup commit : 18f3962953e4 ("mm: hugetlb: kill set_huge_swap_pte_at()"), but since : things have moved on, this would have required an audit of all the new : set_huge_pte_at() call sites to see if they should be converted to : set_huge_swap_pte_at(). As per the original intent of the change, it : would also leave us open to future bugs when people invariably get it : wrong and call the wrong helper. That's actually from the [0/n] intro, which I pasted into the [1/n] changelog. It gets through my scripts OK (https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/mm-hugetlb-add-huge-page-size-param-to-set_huge_pte_at.patch), but it looks like git-quiltimport did the deed. Let me add a check for that...