From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2BBD32C15BC for ; Tue, 16 Sep 2025 05:49:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758001743; cv=none; b=laytawyZv+uCiH9w2YDare0WK458ormKerJzq/vZvSBKSmBbbjhoUY1S3tEFUY/XR4OZuYDWykttowF0WN7PRtKAXsiJCL5OIDcqX7eVytjkOdokWCyIAA05hVf7jDl2QqBSPAu31hT6nkA1eeoKxhwHzRJgtr+17/kqgypjUws= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758001743; c=relaxed/simple; bh=aablpwZgkDY2HF/H36K+OQMk4Kq3szZsC1rhL0md7XU=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=KZALKVN004MFnqwhZsHkP2qM7y6lsfO6ddd5r1GPedJ4GIC+bRx+wlrnp2XHyw3UlorKRa842yjCQO/I4cQNhiKddtnIpCz5L1vBQQolH1CeRcEcRYqMnhz2ZPquvFfTzMsBRwnWQez4UHEEQS2JeyK9WkmF0Ki9AxD2sWR3jYs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=vyE6eyT0; arc=none smtp.client-ip=209.85.128.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="vyE6eyT0" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-45e03730f83so22756535e9.0 for ; Mon, 15 Sep 2025 22:49:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1758001740; x=1758606540; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=2+Ij4gEe2LcS6vPzM/CR57NuWgzn7Jx8huZwedt8Tqc=; b=vyE6eyT0MwN2srMR7la91gruLAHsfDKkC/uqT2y7dnzmpxT4aBrJvFBUXFuRjrlkhg BJMlS1SEf/CJiuAYvoagzi0bw1l+08ehlBDgdoabYAvuN1iRY2J89XjLcEsNGDN9k0ac aOjZLNk1nYz+PjPOearTnT8jGsO9sWDnccM5RosHcGGE4ns9SCCi+jtQ9uqyS6IjwXRQ kXgTpCZ7C8UbjDFqTDMIU4WYwJNH2K4wzKvVh/FMpDu56Bloqs0h6jObhVfoUPcDQBJ0 KI8LDEN3FYm5wE6p822OY7ocAFzSOHDGxrmu7nsNAA2pA6lnpeVond2gJ0vTGVm/MMaI ak/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758001740; x=1758606540; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=2+Ij4gEe2LcS6vPzM/CR57NuWgzn7Jx8huZwedt8Tqc=; b=o48AtyoDUGo9I9PikSWgCeIrbI69xBHT8YgkPScfVXFzgVtF3l8ibR0R3GjDrSC0aX 9Fz3P4eCLzLSw8aBE68KirEFDA+Cb7JQhh6p+uPVpnfywEYpVPtp4RN5m3k5282yIyNu /FgAynQNyQdbUzbCj3VGLCqXRcIUjcaQoZXOJc8W8KBT00L+GRhMaIQesQRwtcPfsRjw zMgX3mX+cBH+M9DKzqozEu9wfZnPPbREv9UvNtfX4Kg1YRH+0cZTqZxNJiVpv6dFrE+D Rzx5UNvukb218WN4acaF/HnqfHFAG7llDLldW7ccYMZW/cxU979KufW1lNyogp/P5z9J C3Cg== X-Forwarded-Encrypted: i=1; AJvYcCUlz+NdURfVgtCn16zqSIDITiabT1FQgdUnCTc0970PKP4G3zMgx1MXqU4Ok3nL6rRx0XrjzRVdrjQ=@vger.kernel.org X-Gm-Message-State: AOJu0Ywk+25l+pejDjDq7v0mOoPyL8DMn1k1I5ABvTncjhstDKuEF1WX Ei5fWSMo56mhrGxii6rvwrDMgHl7u0OmDmglRKG8OjPh55vmIekQUs7bv/fkMl7Gjxw= X-Gm-Gg: ASbGncsLe9W6G+uR4RIrTT8iTUUlBT0/cjb4y5DMC261S8vYKjjRQasKa61/yQWeyq/ +5eFdUWIS//FLFY/g/+1iRB9qomU2/rXM5XKJmxuGe5B2rRyA83DZPCu5AlAgYGwMvpaBbaTstX ik8cVbZKBFhnJmNmwMKiiMXOCWN9DvZbA1kyE0UedEyl83nuGoXRmFU8TUropMGiF3KQSXTbAD7 L2jVPYbEArs+TXDmuM2hvZeImUxH2RRCx1D2e8tIV9a7q97LhJqykzgKRqIoF9dDw9xoN11CDkn ba+06+0dvolm6YfY96gnEKZoehwpl1YQAApZIfp3I5DrCICQek7HAkCPG+iUCAJyDivU1QYer6c lDZjcocgImVOP7bNZEYRx92h0ImmSirrEy90zwuwEwKR+2SOIMZKOG7ZqFKww2UjLIzZK X-Google-Smtp-Source: AGHT+IF7x+pGpXf85Vy38ChqF7Pkim2OKEyjFcqs8YYGtgVmrGA1ZIZ90uccF0N4NZf54XR53W2Dfw== X-Received: by 2002:a05:6000:3113:b0:3e2:ac0:8c55 with SMTP id ffacd0b85a97d-3e765a3e423mr13059397f8f.55.1758001740531; Mon, 15 Sep 2025 22:49:00 -0700 (PDT) Received: from localhost (054722ac.skybroadband.com. [5.71.34.172]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3e760786ceasm20896593f8f.16.2025.09.15.22.48.59 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 15 Sep 2025 22:48:59 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 16 Sep 2025 06:48:59 +0100 Message-Id: Cc: , , , , , , , , , , , , , , , , , Subject: Re: [RFC][PATCH v3 15/16] kmemdump: Add Kinfo backend driver From: "Alexey Klimov" To: "Eugen Hristev" X-Mailer: aerc 0.20.0 References: <20250912150855.2901211-1-eugen.hristev@linaro.org> <20250912150855.2901211-16-eugen.hristev@linaro.org> In-Reply-To: <20250912150855.2901211-16-eugen.hristev@linaro.org> On Fri Sep 12, 2025 at 4:08 PM BST, Eugen Hristev wrote: [..] > --- /dev/null > +++ b/mm/kmemdump/kinfo.c > @@ -0,0 +1,293 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * > + * Copyright 2002 Rusty Russell IBM Corporation > + * Copyright 2021 Google LLC > + * Copyright 2025 Linaro Ltd. Eugen Hristev > + */ > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include Could you please check if the headers are sorted here and in all other patches in this series? Also module.h is duplicated. [..] > +static int build_info_set(const char *str, const struct kernel_param *kp= ) > +{ > + struct kernel_all_info *all_info =3D kinfo->all_info_addr; here ^^ > + size_t build_info_size; > + > + if (kinfo->all_info_addr =3D=3D 0 || kinfo->all_info_size =3D=3D 0) > + return -ENAVAIL; > + > + all_info =3D (struct kernel_all_info *)kinfo->all_info_addr; Maybe assignment of all_info on declaration in the beginning of this functi= on is not needed then? > + build_info_size =3D sizeof(all_info->info.build_info); > + > + memcpy(&all_info->info.build_info, str, min(build_info_size - 1, > + strlen(str))); > + update_kernel_all_info(all_info); > + > + if (strlen(str) > build_info_size) { > + pr_warn("%s: Build info buffer (len: %zd) can't hold entire string '%s= '\n", > + __func__, build_info_size, str); > + return -ENOMEM; > + } > + > + return 0; > +} [...] Best regards, Alexey