>> From: TangDongxing <tang.dongx...@zte.com.cn> >> >> 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.dongx...@zte.com.cn> >> --- >> 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-35fab857b...@kernel.org/ > >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
