Re: [PATCH] xen/pciback: use sysfs_emit_at() instead of scnprintf()

From: tang.dongxing
Date: Mon Mar 17 2025 - 05:19:58 EST


>> From: TangDongxing <tang.dongxing@xxxxxxxxxx>

>> 

>> Follow the advice in Documentation/filesystems/sysfs.rst:

>> show() should only use sysfs_emit() or sysfs_emit_at() when formatting

>> the value to be returned to user space.

>> 

>> Signed-off-by: TangDongxing <tang.dongxing@xxxxxxxxxx>

>> ---

>>   drivers/xen/xen-pciback/pci_stub.c | 2 +-

>>   1 file changed, 1 insertion(+), 1 deletion(-)

>> 

>> diff --git a/drivers/xen/xen-pciback/pci_stub.c b/drivers/xen/xen-pciback/pci_stub.c

>> index b616b7768c3b..a0782a74ed34 100644

>> --- a/drivers/xen/xen-pciback/pci_stub.c

>> +++ b/drivers/xen/xen-pciback/pci_stub.c

>> @@ -1281,7 +1281,7 @@ static ssize_t slots_show(struct device_driver *drv, char *buf)

>>           if (count >= PAGE_SIZE)

>>               break;

>> 

>> -        count += scnprintf(buf + count, PAGE_SIZE - count,

>> +        count += sysfs_emit_at(buf, count,

>

>This is wrong too [1].

>

>[1] 

>https://lore.kernel.org/all/9c552d9a-2d46-4069-a9c4-35fab857bfc3@xxxxxxxxxx/

>

>regards,

>-- 

>js

>suse labs


Dear JiriSlaby,


Thank you for your feedback,I am sorry for my previous submissions.

I will check my work before sending any further updates.


Thank you for your guidance.


Best regards,

Tang Dongxing