* Re: [PATCH] cxl/region: fix format string for resource_size_t
2025-12-04 9:52 [PATCH] cxl/region: fix format string for resource_size_t Arnd Bergmann
@ 2025-12-04 15:22 ` Dave Jiang
2026-01-05 22:17 ` Ira Weiny
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Dave Jiang @ 2025-12-04 15:22 UTC (permalink / raw)
To: Arnd Bergmann, Davidlohr Bueso, Jonathan Cameron,
Alison Schofield, Vishal Verma, Ira Weiny, Dan Williams,
Ben Cheatham
Cc: Arnd Bergmann, Gregory Price, Robert Richter, Li Ming, linux-cxl,
linux-kernel
On 12/4/25 2:52 AM, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> The size of this type is architecture specific, and the recommended
> way to print it portably is through the custom %pap format string.
>
> Fixes: d6602e25819d ("cxl/region: Add support to indicate region has extended linear cache")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>> ---
> drivers/cxl/core/region.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cxl/core/region.c b/drivers/cxl/core/region.c
> index ae899f68551f..fc36a5413d3f 100644
> --- a/drivers/cxl/core/region.c
> +++ b/drivers/cxl/core/region.c
> @@ -759,7 +759,7 @@ static ssize_t extended_linear_cache_size_show(struct device *dev,
> ACQUIRE(rwsem_read_intr, rwsem)(&cxl_rwsem.region);
> if ((rc = ACQUIRE_ERR(rwsem_read_intr, &rwsem)))
> return rc;
> - return sysfs_emit(buf, "%#llx\n", p->cache_size);
> + return sysfs_emit(buf, "%pap\n", &p->cache_size);
> }
> static DEVICE_ATTR_RO(extended_linear_cache_size);
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] cxl/region: fix format string for resource_size_t
2025-12-04 9:52 [PATCH] cxl/region: fix format string for resource_size_t Arnd Bergmann
2025-12-04 15:22 ` Dave Jiang
@ 2026-01-05 22:17 ` Ira Weiny
2026-01-05 23:06 ` Alison Schofield
2026-01-06 1:13 ` Dave Jiang
3 siblings, 0 replies; 5+ messages in thread
From: Ira Weiny @ 2026-01-05 22:17 UTC (permalink / raw)
To: Arnd Bergmann, Davidlohr Bueso, Jonathan Cameron, Dave Jiang,
Alison Schofield, Vishal Verma, Ira Weiny, Dan Williams,
Ben Cheatham
Cc: Arnd Bergmann, Gregory Price, Robert Richter, Li Ming, linux-cxl,
linux-kernel
Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> The size of this type is architecture specific, and the recommended
> way to print it portably is through the custom %pap format string.
>
> Fixes: d6602e25819d ("cxl/region: Add support to indicate region has extended linear cache")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
[snip]
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] cxl/region: fix format string for resource_size_t
2025-12-04 9:52 [PATCH] cxl/region: fix format string for resource_size_t Arnd Bergmann
2025-12-04 15:22 ` Dave Jiang
2026-01-05 22:17 ` Ira Weiny
@ 2026-01-05 23:06 ` Alison Schofield
2026-01-06 1:13 ` Dave Jiang
3 siblings, 0 replies; 5+ messages in thread
From: Alison Schofield @ 2026-01-05 23:06 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Davidlohr Bueso, Jonathan Cameron, Dave Jiang, Vishal Verma,
Ira Weiny, Dan Williams, Ben Cheatham, Arnd Bergmann,
Gregory Price, Robert Richter, Li Ming, linux-cxl, linux-kernel
On Thu, Dec 04, 2025 at 10:52:26AM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> The size of this type is architecture specific, and the recommended
> way to print it portably is through the custom %pap format string.
>
> Fixes: d6602e25819d ("cxl/region: Add support to indicate region has extended linear cache")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Alison Schofield <alison.schofield@intel.com>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] cxl/region: fix format string for resource_size_t
2025-12-04 9:52 [PATCH] cxl/region: fix format string for resource_size_t Arnd Bergmann
` (2 preceding siblings ...)
2026-01-05 23:06 ` Alison Schofield
@ 2026-01-06 1:13 ` Dave Jiang
3 siblings, 0 replies; 5+ messages in thread
From: Dave Jiang @ 2026-01-06 1:13 UTC (permalink / raw)
To: Arnd Bergmann, Davidlohr Bueso, Jonathan Cameron,
Alison Schofield, Vishal Verma, Ira Weiny, Dan Williams,
Ben Cheatham
Cc: Arnd Bergmann, Gregory Price, Robert Richter, Li Ming, linux-cxl,
linux-kernel
On 12/4/25 2:52 AM, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> The size of this type is architecture specific, and the recommended
> way to print it portably is through the custom %pap format string.
>
> Fixes: d6602e25819d ("cxl/region: Add support to indicate region has extended linear cache")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Applied to cxl/fixes
88c72bab77aaf389beccf762e112828253ca0564
DJ
> ---
> drivers/cxl/core/region.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cxl/core/region.c b/drivers/cxl/core/region.c
> index ae899f68551f..fc36a5413d3f 100644
> --- a/drivers/cxl/core/region.c
> +++ b/drivers/cxl/core/region.c
> @@ -759,7 +759,7 @@ static ssize_t extended_linear_cache_size_show(struct device *dev,
> ACQUIRE(rwsem_read_intr, rwsem)(&cxl_rwsem.region);
> if ((rc = ACQUIRE_ERR(rwsem_read_intr, &rwsem)))
> return rc;
> - return sysfs_emit(buf, "%#llx\n", p->cache_size);
> + return sysfs_emit(buf, "%pap\n", &p->cache_size);
> }
> static DEVICE_ATTR_RO(extended_linear_cache_size);
>
^ permalink raw reply [flat|nested] 5+ messages in thread