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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id ECBE1E77199 for ; Wed, 8 Jan 2025 13:03:12 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 432296B0082; Wed, 8 Jan 2025 08:03:12 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 3E22C6B0083; Wed, 8 Jan 2025 08:03:12 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2AA616B0088; Wed, 8 Jan 2025 08:03:12 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 0B1C96B0082 for ; Wed, 8 Jan 2025 08:03:12 -0500 (EST) Received: from smtpin17.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 87C2CC10E2 for ; Wed, 8 Jan 2025 13:03:11 +0000 (UTC) X-FDA: 82984300182.17.BF45957 Received: from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com [148.163.156.1]) by imf16.hostedemail.com (Postfix) with ESMTP id 341E818001B for ; Wed, 8 Jan 2025 13:03:09 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b="JJ4yAn/x"; spf=pass (imf16.hostedemail.com: domain of hca@linux.ibm.com designates 148.163.156.1 as permitted sender) smtp.mailfrom=hca@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1736341389; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=JeSQhydEbs5uuiQ3e3wD+ox1kLggsR4ZEna9+RgR2rw=; b=dkXDIEuradEpqHJpxei2XR7iLsbEXOGeRiKp9d2mS7JOzoxsR+FoI9wxhg1dvDAM9NtyE1 zmpQnqpktSEfb3DDIHYNFsF/Si18SGTuD5JXthUGAA1CHsgLxcj5KTh4mF3O5eBZ6GDqXA 13du6xFfUJ/nNZR+L2GD1VLOU/mf/8w= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1736341389; a=rsa-sha256; cv=none; b=I+b6j4uWh2nraox+UhEI8BlWiKgekfijqLf6J68zE+kaoRhw/MUjEbkCzM3YAx0zLEu2ig PkQz+nbDr8OiJtFb2qIdagevx+nItkD6TIPnNf+/4WRzKtd/WTh+P0xxovGQuT3y+u/JJ7 a+FnFsRsQ3sx+rZymr7FoegsQSLDFLc= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b="JJ4yAn/x"; spf=pass (imf16.hostedemail.com: domain of hca@linux.ibm.com designates 148.163.156.1 as permitted sender) smtp.mailfrom=hca@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com Received: from pps.filterd (m0360083.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 5083qap1015935; Wed, 8 Jan 2025 13:03:02 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=cc :content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=pp1; bh=JeSQhydEbs5uuiQ3e3wD+ox1kLggsR 4ZEna9+RgR2rw=; b=JJ4yAn/xazBWJarur4447f4wfDgjNUY/2PLsAdCbDh4WPk zil12zd7gAAD/6L/QTdSEtQGtam4a+pkmOLVgh0KKCrHRQ5yIADiz4hN2aYZVOfL PQs/xEXLRAwez6B4kNjDlWRml5Cx2sw8p4UBdCAxe86NDZm/iO/qtz+v/NAFWHNo TElizQcpeAbFfuxRwxwr+jxsaNu5zRN8bS8QSdBX0omHcBW90Jh7FHMf1t7VoX5q 9GDzoRfPhwMi6XmyUeU+dyXMrT9yGrRoDq4K/dyN3njG+ltinIj56Ca43s7z5mZF q3BBSFQbUYtrNbtgRLyYoaUfjzRTgueAGfMYHSyA== Received: from ppma23.wdc07v.mail.ibm.com (5d.69.3da9.ip4.static.sl-reverse.com [169.61.105.93]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 441huc2b7v-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 08 Jan 2025 13:03:01 +0000 (GMT) Received: from pps.filterd (ppma23.wdc07v.mail.ibm.com [127.0.0.1]) by ppma23.wdc07v.mail.ibm.com (8.18.1.2/8.18.1.2) with ESMTP id 5089UH11015798; Wed, 8 Jan 2025 13:03:00 GMT Received: from smtprelay02.fra02v.mail.ibm.com ([9.218.2.226]) by ppma23.wdc07v.mail.ibm.com (PPS) with ESMTPS id 43ygtkypyv-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 08 Jan 2025 13:03:00 +0000 Received: from smtpav06.fra02v.mail.ibm.com (smtpav06.fra02v.mail.ibm.com [10.20.54.105]) by smtprelay02.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 508D2uUe35914126 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 8 Jan 2025 13:02:56 GMT Received: from smtpav06.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 5345020049; Wed, 8 Jan 2025 13:02:56 +0000 (GMT) Received: from smtpav06.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 0C71520040; Wed, 8 Jan 2025 13:02:56 +0000 (GMT) Received: from osiris (unknown [9.152.212.60]) by smtpav06.fra02v.mail.ibm.com (Postfix) with ESMTPS; Wed, 8 Jan 2025 13:02:55 +0000 (GMT) Date: Wed, 8 Jan 2025 14:02:54 +0100 From: Heiko Carstens To: Akihiko Odaki Cc: Dave Martin , Vasily Gorbik , Alexander Gordeev , Christian Borntraeger , Sven Schnelle , Eric Biederman , Kees Cook , Catalin Marinas , Mark Brown , Baoquan He , Vivek Goyal , Dave Young , linux-mm@kvack.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, kexec@lists.infradead.org, binutils@sourceware.org, devel@daynix.com Subject: Re: [PATCH v3 5/6] s390/crash: Use note name macros Message-ID: <20250108130254.7704-K-hca@linux.ibm.com> References: <20250107-elf-v3-0-99cb505b1ab2@daynix.com> <20250107-elf-v3-5-99cb505b1ab2@daynix.com> <965b73e7-d0a3-4fae-b0ec-70b5497cb6c4@daynix.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <965b73e7-d0a3-4fae-b0ec-70b5497cb6c4@daynix.com> X-TM-AS-GCONF: 00 X-Proofpoint-ORIG-GUID: vdH5d42t98bHPmlR73Iz70nttiUQ4qpu X-Proofpoint-GUID: vdH5d42t98bHPmlR73Iz70nttiUQ4qpu X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1051,Hydra:6.0.680,FMLib:17.12.62.30 definitions=2024-10-15_01,2024-10-11_01,2024-09-30_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1011 impostorscore=0 spamscore=0 phishscore=0 suspectscore=0 lowpriorityscore=0 mlxlogscore=799 bulkscore=0 mlxscore=0 malwarescore=0 adultscore=0 priorityscore=1501 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2411120000 definitions=main-2501080108 X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: 341E818001B X-Stat-Signature: rnq9bmron4dtxbhqonwb4jqmxoc4mtfr X-Rspam-User: X-HE-Tag: 1736341388-82075 X-HE-Meta: U2FsdGVkX1+2Re5FKwTrLXRn06M3aA3Iz4qFyvxEbv0QF3KT/euDXU4KoFChJu2EEYZbMy1BMzFcGOC3iNOTPOupbVX6FdYcGhEsxZnALOo5zoyhPc6B5zOz7qI9WDk3O1srLg0OFFSGbDq2gAzUWiRkCA9Eg5ZNDAkTRirW5OWlgxOLvhCYVOa+Q/OZkIMcwotPJ+O4BaifIlR3U+mJNZ9Bx8kFjpbmrUfuZyJGXWb7vJ9ikMBsfPiq7mfW/c7G9qURjZrsf3U+uvF//yQmhk43qNtuUE5mS6nsnhAeFTEtpB3jUlzT5UMdDL+xfStFYyLPVDLSHLZDuHcXboCWaiDmX0q54FV6AGrQkc8TdDkA8WkabQtiJ2PK6kHgusTFMTbsN1PKpsGFHbO450ZLIAro+CS1pv4wQQdoAFitIkMWh2eiNpSC51ONyiZ9wUYI6H4jMKNC+jVW+L7HLwjackO0Bt9qfODjf72c5RwkeHgmXt3hlm2HRt6Ek/C3rdcSIBsO8c/DHOd9Xn5bJFjFD0Gi5ZqFCYaVcCTU6Vy18aNbHpsoRs9g7/Jus7qd0pdasR3PvCBG+lLvsQq2GKDz3yaM2YG1F3p6Duco7XyUxq3ylaZ1mmgazrJXUz539juwakoY9A0hCNP6ra1+a+Is56fD7OIe+qdZbMpYB//guhz6VpqSpxqT3FrJr4dLwXpy1aXMaa6Y2rCK73NR+hNLFp36ExvJzAIOxj++hY3deSgWvwB86fV8BqwsRl6Z2uZnIRaA/U/KbBI+bhganEENEX1jxPuThuiNT52nbE1DIXf05xfQ4oRyYsfhMLfzmZYKOXAbURwzAUY+1LRcDywUujBPdgs1HjSUhvXL9mj7ixijb8fS9wkqm/OWysf8aZ9x8kaYiSLHI9uybveIX0mjHGSYpS2WIgSHcW0bYxl+/JIc3kzyBk3dwEFLn9LUnHwWDukCBpJPFRlhR6u6Haj l+hc1BL6 tHUTfwgMjsxk9HxnB69Rhs4iF2GBnvPjxbmnFiS9w7/wZ87D0yqvyBrt560ZSWncV4UjVOJXmGED8loQsdM3/wVZISDxJhxYN7MOgoandcD2MDq9iH56Ra6AWhMGLGEipQaLxpoDew0N6a2d/qoGb/+SfHIOVKJgX2fHTo1jpdiSH/H8hbgNtCBCo+7wZFDiqVQNuktvMvZvOIbhVifUnvOGsr8MsGbatw69l1AeAIoL+yS7HUjIWBiPZEw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000448, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Wed, Jan 08, 2025 at 01:53:51PM +0900, Akihiko Odaki wrote: > On 2025/01/08 1:17, Dave Martin wrote: > > > +#define NT_INIT(buf, type, desc) \ > > > + (nt_init_name((buf), NT_ ## type, &(desc), sizeof(desc), NN_ ## type)) > > > > Nit: this macro name clashes with the naming scheme in elf.h. > > > > I think that there is a (weak) convention that macros with upper-case > > names don't expand to a C function call; thus, a macro with an upper- > > case name can be invoked in places where a C function call would not be > > allowed. (This convention is not followed everywhere, though -- it's > > up to the maintainer what they prefer here.) > > I wanted to clarify it is a macro as it concatenates tokens with ##, but I > also find there are many macros that are named lower-case and performs token > concatenation. > > S390 maintainers, please tell usr your opinion. Just make the new macros lower case to avoid the naming scheme clashes, please. Otherwise it doesn't matter too much. > > > +#define NT_SIZE(type, desc) (nt_size_name(sizeof(desc), NN_ ## type)) > > > > Nit: name prefix clash (again); possibly redundant parentheses. Same here. > > > - size = nt_size(NT_PRSTATUS, sizeof(struct elf_prstatus)); > > > - size += nt_size(NT_PRFPREG, sizeof(elf_fpregset_t)); > > > - size += nt_size(NT_S390_TIMER, sizeof(sa->timer)); > > > - size += nt_size(NT_S390_TODCMP, sizeof(sa->todcmp)); > > > - size += nt_size(NT_S390_TODPREG, sizeof(sa->todpreg)); > > > - size += nt_size(NT_S390_CTRS, sizeof(sa->ctrs)); > > > - size += nt_size(NT_S390_PREFIX, sizeof(sa->prefix)); > > > + size = NT_SIZE(PRSTATUS, struct elf_prstatus); > > > + size += NT_SIZE(PRFPREG, elf_fpregset_t); > > > + size += NT_SIZE(S390_TIMER, sa->timer); > > > + size += NT_SIZE(S390_TODCMP, sa->todcmp); > > > + size += NT_SIZE(S390_TODPREG, sa->todpreg); > > > + size += NT_SIZE(S390_CTRS, sa->ctrs); > > > + size += NT_SIZE(S390_PREFIX, sa->prefix); > > > > It might be worth fixing the funny spacing on these lines, since all > > the affected lines are being replaced. Yes, please! Besides that this looks good: Acked-by: Heiko Carstens