[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



>> 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



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.