On Sat, Feb 07, 2015 at 12:07:43PM +0800, Greg KH wrote: > On Fri, Feb 06, 2015 at 03:41:58PM +0000, Bryn M. Reeves wrote: > > I can't speak for Shane but wouldn't spend too much time looking at the > > current v2 patch: it's the result of a pretty ugly compromise suggested > > on linux-scsi. > > Fair enough, but please feel free to cc: me on the patch that you do > feel is correct to get a sysfs-related review. Will do; I'm back from travels this week & will have some time to look at this. > > Likewise for disk stats: although fluff like maj:min/name etc. has been > > shuffled a few times the basic fields have remained unchanged for a very > > long time and sysfs already removes the need to include an identity > > field. > > We already handle i/o stats just fine, why create a special sysfs > interface for just a tape device interface? What makes them so special? But the iostats use exactly the sort of array file we're talking about: $ cat /sys/block/sda/stat 127644 20869 4320505 2305697 154045 30056 3834036 9065092 0 931842 11371357 And we can't simply extend these to tapes as they are not block devices. > > I understand the fact that you can't change them; I just don't think it's > > a big problem in this specific case (and much less than some of the > > more imaginative sysfs content - 2d int arrays with column headers > > anyone?). > > What sysfs file is a 2d int array? I'll be glad to fix it. $ cat /sys/fs/selinux/avc/cache_stats lookups hits misses allocations reclaims frees 18938916 18921707 17209 17209 17328 22215 38164283 38146514 17769 17769 16800 19049 18078108 18056991 21117 21117 21344 19305 15168204 15150079 18125 18125 17776 13149 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 $ cat /sys/fs/selinux/avc/hash_stats entries: 506 buckets used: 290/512 longest chain: 5 > If you want to measure tens of thousands of tape devices then you > shouldn't be using sysfs in the first place as it is not designed for > "speed" at all. Use the existing i/o rate interfaces instead, don't try > to cram something into sysfs that doesn't belong there. So far as I'm aware there is no other way to obtain performance data for the SCSI tape subsystem (without resorting to ftrace/systemtap). Regards, Bryn. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
