2022-12-05 18:03 GMT+09:00, ye.xingchen@xxxxxxxxxx <ye.xingchen@xxxxxxxxxx>: > From: ye xingchen <ye.xingchen@xxxxxxxxxx> > > Follow the advice of the Documentation/filesystems/sysfs.rst and show() > should only use sysfs_emit() or sysfs_emit_at() when formatting the > value to be returned to user space. > > Signed-off-by: ye xingchen <ye.xingchen@xxxxxxxxxx> > --- > fs/ksmbd/server.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/ksmbd/server.c b/fs/ksmbd/server.c > index a0d635304754..b0da15377709 100644 > --- a/fs/ksmbd/server.c > +++ b/fs/ksmbd/server.c > @@ -433,7 +433,7 @@ static ssize_t stats_show(struct class *class, struct > class_attribute *attr, > "shutdown" > }; > > - ssize_t sz = scnprintf(buf, PAGE_SIZE, "%d %s %d %lu\n", stats_version, > + ssize_t sz = sysfs_emit(buf, "%d %s %d %lu\n", stats_version, > state[server_conf.state], server_conf.tcp_port, > server_conf.ipc_last_active / HZ); It would be great if you remove sz variable and return it directly. Thanks. > return sz; > -- > 2.25.1 >
