On Wed, May 21, 2014 at 07:32:29AM -0700, Christoph Hellwig wrote: > On Wed, May 21, 2014 at 02:25:21PM +0200, Heiko Carstens wrote: > > Hi all, > > > > I'm just wondering why /proc/stat is a single_open() seq_file and not a > > regular seq_file with an iterator (say 48 online cpus for each iteration > > or something similar). > > Probably because no one sent a patch for it. I'm pretty sure it used the > even more horrible old proc ops before and was converted in batch with > various other files. Ok, so how about the two patches sent as reply to this mail. (btw. if nobody objects to the modified patch from KAMEZAWA Hiroyuki the first patch could be dropped and/or folded into the second patch) Performance wise there doesn't seem to be too much of a difference, however all measurements have been done a 64 cpu 2nd level guest. It _looks_ like the new code is < 3% slower. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
