bcache: Convert to use sysfs_emit()/sysfs_emit_at() APIs
PATCH 1/1 bcache: Convert to use sysfs_emit()/sysfs_emit_at() APIs
From: ye xingchen <[email protected]>
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 <[email protected]>
--- drivers/md/bcache/sysfs.c | 31 +++++++++++++++---------------- 1 file changed, 15 insertions(+), 16 deletions(-)
drivers/md/bcache/sysfs.c +15 -16
@@ -1111,26 +1111,25 @@ SHOW(__bch_cache)vfree (p); - ret =scnprintf (buf,PAGE_SIZE , -"Unused: %zu%%\n" -"Clean: %zu%%\n" -"Dirty: %zu%%\n" -"Metadata: %zu%%\n" -"Average: %llu\n" -"Sectors per Q: %zu\n" -"Quantiles: [" , - unused *100 / (size_t ) ca->sb.nbuckets, - available *100 / (size_t ) ca->sb.nbuckets, - dirty *100 / (size_t ) ca->sb.nbuckets, - meta *100 / (size_t ) ca->sb.nbuckets, sum, - n * ca->sb.bucket_size / (ARRAY_SIZE (q) +1 )); + ret =sysfs_emit (buf, +"Unused: %zu%%\n" +"Clean: %zu%%\n" +"Dirty: %zu%%\n" +"Metadata: %zu%%\n" +"Average: %llu\n" +"Sectors per Q: %zu\n" +"Quantiles: [" , + unused *100 / (size_t ) ca->sb.nbuckets, + available *100 / (size_t ) ca->sb.nbuckets, + dirty *100 / (size_t ) ca->sb.nbuckets, + meta *100 / (size_t ) ca->sb.nbuckets, sum, + n * ca->sb.bucket_size / (ARRAY_SIZE (q) +1 ));for (i =0 ; i <ARRAY_SIZE (q); i++) - ret +=scnprintf (buf + ret,PAGE_SIZE - ret, -"%u " , q[i]); + ret +=sysfs_emit_at (buf, ret,"%u " , q[i]); ret--; - ret +=scnprintf (buf + ret,PAGE_SIZE - ret,"]\n" ); + ret +=sysfs_emit_at (buf, ret,"]\n" );return ret; }
Signature · 1 line
2.25.1
Quoted 11 lines
2022年12月7日 17:02,<[email protected]> <[email protected]> 写道:
From: ye xingchen <[email protected]>
Follow the advice of the
Documentation/filesystems/sysfs.rstandshow()should only usesysfs_emit()orsysfs_emit_at()when formatting the value to be returned to user space.Signed-off-by ye xingchen <[email protected]>
Added to my for-next queue for Linux v6.3 merge window.
Thanks.
Coly Li
Quoted 52 lines
---
drivers/md/bcache/sysfs.c| 31 +++++++++++++++----------------1 file changed, 15 insertions(+), 16 deletions(-)drivers/md/bcache/sysfs.c +15
-16@@ -1111,26 +1111,25 @@ SHOW(__bch_cache)vfree (p); - ret =scnprintf (buf,PAGE_SIZE , -"Unused: %zu%%\n" -"Clean: %zu%%\n" -"Dirty: %zu%%\n" -"Metadata: %zu%%\n" -"Average: %llu\n" -"Sectors per Q: %zu\n" -"Quantiles: [" , - unused *100 / (size_t ) ca->sb.nbuckets, - available *100 / (size_t ) ca->sb.nbuckets, - dirty *100 / (size_t ) ca->sb.nbuckets, - meta *100 / (size_t ) ca->sb.nbuckets, sum, - n * ca->sb.bucket_size / (ARRAY_SIZE (q) +1 )); + ret =sysfs_emit (buf, +"Unused: %zu%%\n" +"Clean: %zu%%\n" +"Dirty: %zu%%\n" +"Metadata: %zu%%\n" +"Average: %llu\n" +"Sectors per Q: %zu\n" +"Quantiles: [" , + unused *100 / (size_t ) ca->sb.nbuckets, + available *100 / (size_t ) ca->sb.nbuckets, + dirty *100 / (size_t ) ca->sb.nbuckets, + meta *100 / (size_t ) ca->sb.nbuckets, sum, + n * ca->sb.bucket_size / (ARRAY_SIZE (q) +1 ));for (i =0 ; i <ARRAY_SIZE (q); i++) - ret +=scnprintf (buf + ret,PAGE_SIZE - ret, -"%u " , q[i]); + ret +=sysfs_emit_at (buf, ret,"%u " , q[i]); ret--; - ret +=scnprintf (buf + ret,PAGE_SIZE - ret,"]\n" ); + ret +=sysfs_emit_at (buf, ret,"]\n" );return ret; }Signature · 1 line
2.25.1