Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Date:   Mon, 9 Oct 2023 08:04:01 -0700
From:   Mike Kravetz <mike.kravetz@oracle.com>
To:     Konrad Dybcio <konrad.dybcio@linaro.org>
Cc:     Anshuman Khandual <anshuman.khandual@arm.com>,
        Andrew Morton <akpm@linux-foundation.org>,
        Xiongchun Duan <duanxiongchun@bytedance.com>,
        Barry Song <21cnbao@gmail.com>,
        David Rientjes <rientjes@google.com>,
        Miaohe Lin <linmiaohe@huawei.com>,
        Matthew Wilcox <willy@infradead.org>, linux-mm@kvack.org,
        Naoya Horiguchi <naoya.horiguchi@linux.dev>,
        Joao Martins <joao.m.martins@oracle.com>,
        David Hildenbrand <david@redhat.com>,
        Michal Hocko <mhocko@suse.com>,
        Oscar Salvador <osalvador@suse.de>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 2/8] hugetlb: restructure pool allocations
Message-ID: <20231009150401.GA3846@monkey>
References: <20230925234837.86786-3-mike.kravetz@oracle.com>
 <ddc10c41-515f-48c9-966d-4ae7756d208c@kernel.org>
 <20230929205731.GA10357@monkey>
 <ba5a528d-7e85-77a2-5096-5a143a3b1606@linaro.org>
 <20231006030836.GC86415@monkey>
 <5d14b681-a2c9-4210-a0c1-bf38708b7998@linaro.org>
 <20231006223512.GC3861@monkey>
 <20231009032926.GA3376@monkey>
 <070bd916-d4d6-41c2-9f51-af35e80c96b9@linaro.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <070bd916-d4d6-41c2-9f51-af35e80c96b9@linaro.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?pS4FrU2FCCgu+c1WlDdib45Slz4IDS7IcrsBKRi2RT1o0unrtIK9xr+Pn6WD?=
 =?us-ascii?Q?RRSfVGtNcxW3hFDfJVhT2jrR77H7O6kITTV4Cn9H9JWR5xR1nJG52wn3kwux?=
 =?us-ascii?Q?33vdXT3MiCExz9FIzsYBZGagNPvqOqPaEK9VULwoPiGZ7WgDYxGM9n7TKk2s?=
 =?us-ascii?Q?AJNqVN+6BeBoc/xY0OfGD9IVpxXR8fdb4DFl9mE+JBA9QeonyJdVQGdCkKO3?=
 =?us-ascii?Q?G53vk0nxfzV9srcnqSuaXv6SnjX+glpvIcKGd+08bMJmhqCmkdgfjTuZSIek?=
 =?us-ascii?Q?pOpYowhRwO2df7l8/g9O4Jr5AiPF5GsbKgaEB+4hBHV2ltyinc3gelgrMe2s?=
 =?us-ascii?Q?LPKLx/rz3LVZmX/mJ4feF3Y5Uwn4ksSp0FdOv8UqlNihhnFAROoGQPBgRGVy?=
 =?us-ascii?Q?pmPPbjiQwKRXLxUf89tXuap8f60w6q5jRDEJzM7f6VuCaQPCd46g87+sfWcd?=
 =?us-ascii?Q?Py6RW08IjMO8blvj5YQaJPLmusvp2UVicCPNg65Tm+P+KuMjJZIeMNGhTkNN?=
 =?us-ascii?Q?ds7hl5Vak0ZMsfw8iOLEJvY+P1xMFkMGFvJvPpJVhGGKmGAkvtcvKG2E04TP?=
 =?us-ascii?Q?E3mRn6SBvIJYbWVaXq4wZTUCVKtL/eVQYFKk2UaZ2lx0I51F30t4fiPvN4a8?=
 =?us-ascii?Q?sGvByWL7/latQxWRmf1GOrkkY4hHLoTCYq5zOVF9N6+qjPdzLcSxnkvwMl6F?=
 =?us-ascii?Q?yJfOGYI36dyn+XPoZK1nVs/Lk88kRI0R4AW2slXGEePEh0iNk0jD+cxQJ1+D?=
 =?us-ascii?Q?a/qANae9O6cjJYUuMkBDfcSZKfAsInrWGQ5frTzClN2FW+jauT8Qi+YVxaKO?=
 =?us-ascii?Q?Es5fHRDmx+A4NP4sJKs8N0wz9pgueAR1cCPrrpb+/FCSgWL41tvRta+IDzHT?=
 =?us-ascii?Q?JDKGBEoj9OcplQQcBenVrTJM8q7EyfwFIqcqmr4ClT0Q73JwRpGPOmvBGVYQ?=
 =?us-ascii?Q?fLbZyXY9kntJP8x6n8vWc+I2Y+qjMeWoipE+3ekilK8lZzHuQdjaLD633FqS?=
 =?us-ascii?Q?FqYAwndHT6IW0Kvk8hy6+AE7qfFtKfqOyDBGqhFyml705Wh/SKaGzceH6Bx6?=
 =?us-ascii?Q?GskkPgXi8NzIpjNA6aGNJ9aEKL5chaFIIRFLhQRaS4hXBBWzZ6WPoV3xJBaQ?=
 =?us-ascii?Q?Y9WjO4hvpRvj3aWQDgCV71UhCJNNt+iyzk6DumV5wy84Dq1LBnAaRfo1F9TK?=
 =?us-ascii?Q?U8Ob7UcLpwhVIAkkYuFLzT4j1SROwx4GQNBDH3ENuEpTlQhrtH9W39bwNXtw?=
 =?us-ascii?Q?EK9hGnkYcz7YYJsx9GT8/elwPzDV6Qh5fXjzVZooqXsx8Q5pxhZohjGAPx03?=
 =?us-ascii?Q?xgZWi/c2FVRHm44AMDzUsyEVJqtuEYFS+RYuoEz63dbMj4saTTHpub0GRegS?=
 =?us-ascii?Q?qo1fko3zAsplCok2nBB9IzDOTkwCPqnq9pXLiDIZd8ip0XJJO6QQ9Amjvu5s?=
 =?us-ascii?Q?t+QEO5uYzzb+Gm9qNxnHma/Kx3NzOEMMf/lQ7CE2vE+4o8erkjjgOtiFaAT9?=
 =?us-ascii?Q?34XGaUuPYp7qJm7Xi670KNNQeC8KEAftMrvydm4rFp9DWzoFIf1tZZAsaCP2?=
 =?us-ascii?Q?gd7X6QJJeT3FhKHBi9TANoIcR9ODZRaI4yLnK8rCwUVi9CpcxtGPx7KSfRLi?=
 =?us-ascii?Q?7Q=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?D6TECCR/G5sjsnDvnka7oSd85UcC2DW2TpdpdOHYVNm7/uOtTOrB73gV20Fy?=
 =?us-ascii?Q?tEGy+sEYEMPS9flzXe+daGBy73f5dlZ6luWTVEsASWtbDLCRMfrZS1wzO1xV?=
 =?us-ascii?Q?dVbk4xb4Uq2Glu5mj45SLTGjdh9DBCzs3RGXNtwRjOnbvMev+SGbLlxz4UIX?=
 =?us-ascii?Q?LPObTR8jOoXX8HEYOzXOvRnEQYVSnGUhKeWPP12uAWU7O5ICFFF6xvmi9l3/?=
 =?us-ascii?Q?lv/VLGmrg6WN7oqjaP8ONZzUce067h5rRdphVfk+dfI7EGau+US+ExAUE98J?=
 =?us-ascii?Q?l4yMueLrtZ081pEjWAYx0mJx9hkmkVjhTDG1FTQkbr1ZJSYOWgZIzntQHpY2?=
 =?us-ascii?Q?JkX01TYBlzpD7LPmfATa5T8u0fK/vV3Vl/irZptOryUrSeRhn6prwY7BpKxj?=
 =?us-ascii?Q?wuAszfl40cr4T40Rk9W8ycNz1BWuFAudusFhjofBBuLZZBJlH3+oh/hsZvBb?=
 =?us-ascii?Q?BFUk8NgoQWhvjwCN8wMb1MkxwVH5YpP0ESi5ReeC15FYi0CekiqkGKbVNdsl?=
 =?us-ascii?Q?hDCBFWsokZ+paJxx0BX7C0ro1Q2wCLOeVZq9Gsxs53rDR+4TJ31Arsxsd4Wz?=
 =?us-ascii?Q?Ph+vmrTU1wuvCve8oslXfljLJHd/q+biIY1o4m8LuPdxowsixwPdLzffN9+c?=
 =?us-ascii?Q?kGI6zaq9PEjREK8QvKWgCUtv5YLfWtaFtEAzStFY6kkaZh9tPgQuTY6HY8uf?=
 =?us-ascii?Q?/agsqGqCT+/sj+b+DA+iUDF74H8QIY45Hen/fXYcgF1XbdfYHsWTlNGfBt2s?=
 =?us-ascii?Q?Vg3Dgc32Wp4vHnGLGlvnVbCPjODnyENbP3IAQivcf9LOQrlgm7k94pjtd7mG?=
 =?us-ascii?Q?dfAwhPcutlwNGKHPoJjG6L7L7A9cnioGfbTUsLfcyJMI2HyP8KcKXlJ9/Z5q?=
 =?us-ascii?Q?gxisd2V9cUFVVea0DotpFZKVlZbwqDd6azFTSL5qn88GLTiv4CK2VBWUmB6q?=
 =?us-ascii?Q?K/PnfS6hSbAAq7Pn+3YvPQ9wVV7hOSYGmYW+h5D05EWpvHvg3NQKw31MMBBx?=
 =?us-ascii?Q?TvGRI5yM4a61FIwlbSk6AucUaj1IcFK15Rd8ajOVkJ01rr8hGcaaDCQfD8od?=
 =?us-ascii?Q?ErxYvo9bYOfkCfLrSCTixIy/VdcuGmFDmc91lTDwNQe5coaaM9RYMBeKaDvi?=
 =?us-ascii?Q?1uTPI/Z2UkoxjuvIQL3Cph1RsODzj+9g1Q=3D=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f6b00734-8e61-48e3-3416-08dbc8d8fe7b
X-MS-Exchange-CrossTenant-AuthSource: BY5PR10MB4196.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Oct 2023 15:04:11.4428
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: iy8ixeg6IA1lzQZvpa7geckoC01yny0PTHRSgsl8YxVUiVG1LKDwtLLcS4zNzwzjQ1duWhwuGBNBlC1vCiibWQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR10MB4890
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.267,Aquarius:18.0.980,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-10-09_13,2023-10-09_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 spamscore=0 adultscore=0
 suspectscore=0 mlxlogscore=999 phishscore=0 bulkscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2309180000
 definitions=main-2310090125
X-Proofpoint-GUID: wkZvEbQhulMyiCFiCrfvfORcPU2DS9Au
X-Proofpoint-ORIG-GUID: wkZvEbQhulMyiCFiCrfvfORcPU2DS9Au
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,
        RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Mon, 09 Oct 2023 08:05:14 -0700 (PDT)

On 10/09/23 12:11, Konrad Dybcio wrote:
> On 9.10.2023 05:29, Mike Kravetz wrote:
> > On 10/06/23 15:35, Mike Kravetz wrote:
> >> On 10/06/23 23:39, Konrad Dybcio wrote:
> >>> On 6.10.2023 05:08, Mike Kravetz wrote:
> >>>> On 10/02/23 11:57, Konrad Dybcio wrote:
> >>>>> On 9/29/23 22:57, Mike Kravetz wrote:
> >>>>>> On 09/27/23 13:26, Konrad Dybcio wrote:
> >>>>>>> On 26.09.2023 01:48, Mike Kravetz wrote:
>
> I managed to get a boot log:
> 
> https://pastebin.com/GwurpCw9
> 
> This is using arch/arm64/boot/dts/qcom/sm8550-mtp.dts for reference
> 

Early on in boot log before the panic, I see this in the log:

[    0.000000] efi: UEFI not found.
[    0.000000] [Firmware Bug]: Kernel image misaligned at boot, please fix your bootloader!

Isn't that misalignment pretty serious?  Or, is is possible to run with that?

There are no hugetlb pages allocated at boot time:

[    0.000000] Kernel command line: PMOS_NO_OUTPUT_REDIRECT console=ttyMSM0 earlycon clk_ignore_unused pd_ignore_unused androidboot.bootdevice=1d84000.ufshc androidboot.fstab_suffix=default androidboot.boot_devices=soc/1d84000.ufshc androidboot.serialno=ab855d8d androidboot.baseband=msm 

So, the routine where we are panic'ing (gather_bootmem_prealloc) should be a
noop.  The first thing it does is:
list_for_each_entry(m, &huge_boot_pages, list) {
...
}

However, huge_boot_pages should be empty as initialized here:
__initdata LIST_HEAD(huge_boot_pages);

At the end of the routine, we call prep_and_add_bootmem_folios to
process the local list created withing that above loop:

LIST_HEAD(folio_list);

This should also be empty and a noop.

Is it possible that the misaligned kernel image could make these lists
appear as non-empty?
-- 
Mike Kravetz