Received-SPF: pass (google.com: domain of linux-kernel+bounces-14759-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Tue, 2 Jan 2024 11:05:41 -0800
From: Dan Williams <dan.j.williams@intel.com>
To: Randy Dunlap <rdunlap@infradead.org>, <linux-kernel@vger.kernel.org>
CC: Randy Dunlap <rdunlap@infradead.org>, Fan Ni <fan.ni@samsung.com>,
	Davidlohr Bueso <dave@stgolabs.net>, Jonathan Cameron
	<jonathan.cameron@huawei.com>, Dave Jiang <dave.jiang@intel.com>, "Alison
 Schofield" <alison.schofield@intel.com>, Vishal Verma
	<vishal.l.verma@intel.com>, Ira Weiny <ira.weiny@intel.com>, Dan Williams
	<dan.j.williams@intel.com>, <linux-cxl@vger.kernel.org>
Subject: RE: [PATCH] cxl/region: use %pap format to print resource_size_t
Message-ID: <65945e8583d5e_8dc6829487@dwillia2-xfh.jf.intel.com.notmuch>
References: <20240102173917.19718-1-rdunlap@infradead.org>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240102173917.19718-1-rdunlap@infradead.org>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?8Ffhf91+FOAykFRmspbkKpijs1OKU93uzSopUOPx/Jq/dv5TSU1P/9m9MGQR?=
 =?us-ascii?Q?CGyWVdOohK7rz06gSHfhSv+6lh51hguHqTp2mwBORsESPusnF/uBA1X+voiW?=
 =?us-ascii?Q?F6evhJGuICnItabYRi2rqRK8+dMYxUltLhi/Y4r5+KE8JpGco0JY3MWttzJU?=
 =?us-ascii?Q?UaowIOFpSrQshW3B7C9qmHujBVX1EQS3RoTiRPIXWCB7yBkMtE5A8xyt2HfP?=
 =?us-ascii?Q?IkRe8eu5Qw56lsJ4oYQRMNV/zFVCTrrE0Hj9ge5k2OVAlMHsbqLccPROjCe0?=
 =?us-ascii?Q?cAM7f212Ut59msk3NOQB6zhhbmmjkpo+WRSJLoIxDNFdus8cTMPbt9QHcR87?=
 =?us-ascii?Q?954CYSSn3GW3Ej/C1Ty1Qx/m+6PlN0RmWk46dxpw6Kjy3b2yP/j2NzREbA/p?=
 =?us-ascii?Q?OfrfjIKVl8qivmwSNgKTetTJymwWB8/yamyNXIR21Je79Dx358LrkcqvauO+?=
 =?us-ascii?Q?UK3BKb36ZX4SMpFLzRJT4LyWJYw2DvY9ZNysF5jBpFELzdDLui1hdftWkduv?=
 =?us-ascii?Q?PGmBsEA7Y9cQH4wNrEVXKNaWdnWI+Vzg2mxYJykv5ov9tq8+RP4LzHjJJzhV?=
 =?us-ascii?Q?qO+nfB5AWs589LBsrR3D+op9bngS82J6hzpJFwyEKDBIf0h7hT3KXGytPEiA?=
 =?us-ascii?Q?1D5U6DF79yGCjSFLJcE3vJ8/X6SjRACQXO2iMT8POyBzqb1kxGtdeQDcPvq8?=
 =?us-ascii?Q?d5kgQ97G2hCYO5X0N5E3rhtcf8LbioaE27NWTNLYtMm/UPxFrygjMSfVdBvO?=
 =?us-ascii?Q?1EWF+Cmn2eMukSz4zawinFt8c4KtaH7xzxy5Pk4p6fCpgTS+dEIgNB+7tz5q?=
 =?us-ascii?Q?Vtd6aDq75LuPVMaE4LD7aOURHoJ58e1lSVnwvOMVR9ex4rF7bYNwls9hYQA0?=
 =?us-ascii?Q?Gj+h4/B/b/iDAT5bW8PcBaKisk9uT+LDeAGIeLKBSe+HBOqLLyERMpiIoJmq?=
 =?us-ascii?Q?WxZjxEzjLTrYnC7a2w5lTdpbyqqztrkIt1MPFHUpdfYWwnKiTQma6X1n1dB8?=
 =?us-ascii?Q?ShU39RYn1aKTkymtyzVnYzlGsUnrO0Rsjs3D22XI8CS3W67EvUT+poYWIh3j?=
 =?us-ascii?Q?N5ptEgxxbIL7bDk5ZZArCoM+VOBYww36D4b+d0xZ/AY274al6Q5GYnkQjkQy?=
 =?us-ascii?Q?2N32+Q3rDbyPrsZHY9zx/j4ro1eGsiophOLTxAp3Kt+BAvwLSbWT23JDYH4+?=
 =?us-ascii?Q?+NIEbdG9TtouVfFh4eVLR9LQWuCZ/X2OH8J8/0H2epYe5j/HV3X+aQcyq/SZ?=
 =?us-ascii?Q?Obe+cT2vm9HR4uvDzg/KibOR/8dFHEzPxoyAYHcCFxM5Zc3TDIdQNGvcNlHW?=
 =?us-ascii?Q?ScusJm+et28w3qB+TmfUveaOSXysgLq/hCrv0crZl5oHmcWf5WxdWh8IIsyY?=
 =?us-ascii?Q?yUagqfBsKohraFRQr9sMwAnqsaE9BiFXZIBWuAkGkRgtnkKSBOxERXam/SzS?=
 =?us-ascii?Q?ciuAuFrCqzCd+L0SifcYI3Xdjn/a8m1vqx2lWczR9ONtIvAhvtzM8BnKXNGg?=
 =?us-ascii?Q?qxg0m1Nsl0wCFT0t1uRRBolKrX5hogmeYIr4fI0A3OJiEO/s4LGECcmlP3ej?=
 =?us-ascii?Q?tNHDkF3faIISxKMpWHoRJvSAXNKDec0Pbua9ZL9rbASn4Le5RxhTlaDG7nRp?=
 =?us-ascii?Q?Yg=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 208af6af-f37d-484d-76fd-08dc0bc5d2b7
X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jan 2024 19:05:45.3721
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: rpLqGOXMLrKCzHMo4GNNLwm6ZbU5lX1H5b3xiclxd2lMVtnz1VkRVjyB1E0n+NZP9iUPzlDnX3W3Q2p5UyNpkdx1oFT//1j+es7EreJiHxM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB8687
X-OriginatorOrg: intel.com

Randy Dunlap wrote:
> Use "%pap" to print a resource_size_t (phys_addr_t derived type)
> to prevent build warnings on 32-bit arches (seen on i386 and
> riscv-32).
> 
> ../drivers/cxl/core/region.c: In function 'alloc_hpa':
> ../drivers/cxl/core/region.c:556:25: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 5 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=]
>   556 |                         "HPA allocation error (%ld) for size:%#llx in %s %pr\n",
> 
> Fixes: 7984d22f1315 ("cxl/region: Add dev_dbg() detail on failure to allocate HPA space")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Fan Ni <fan.ni@samsung.com>
> Cc: Davidlohr Bueso <dave@stgolabs.net>
> Cc: Jonathan Cameron <jonathan.cameron@huawei.com>
> Cc: Dave Jiang <dave.jiang@intel.com>
> Cc: Alison Schofield <alison.schofield@intel.com>
> Cc: Vishal Verma <vishal.l.verma@intel.com>
> Cc: Ira Weiny <ira.weiny@intel.com>
> Cc: Dan Williams <dan.j.williams@intel.com>
> Cc: linux-cxl@vger.kernel.org
> ---
>  drivers/cxl/core/region.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied, thanks Randy.