On Wed, Nov 07, 2007 at 09:57:42AM -0800, Stephane Eranian wrote: > Greg, > > On Wed, Nov 07, 2007 at 09:47:47AM -0800, Greg KH wrote: > > On Wed, Nov 07, 2007 at 05:42:55AM -0800, Stephane Eranian wrote: > > > Greg, > > > > > > Perfmon sysfs document has been updated following your adivce. > > > you can check out in my perfmon tree the following commit: > > > > > > e83278f879e52ecee025effe9ad509fd51e4a516 > > > > Thanks, that looks a lot better. > > > > Do you want me to send you patches based on this tree to help clean up > > the sysfs usage now that it's documented? > > > Yes, send me the patches. But from what you were saying earlier it seems > I would need an extra sysfs patches to make this compile. Is that particular > patch already in Linus's tree?
No, it's in my tree, and will be in the next -mm. You will need a few patches to get this to work, not just a single patch. > > Also, a lot of your per-cpu sysfs files should probably move to debugfs > > as they are for debugging only, right? No need to clutter up sysfs with > > them when only the very few perfmon developers would be needing access > > to them. > > > Yes, this is mostly debugging. If debugfs is meant for this, then I'll > be happy to move this stuff over there. Is there some good example of how > I could do that based on my current sysfs code? There is documentation for debugfs in the kernel api document :) And, there are many in-kernel users of debugfs, a grep for "debugfs_create_" should show you some examples of how to use this. If you have any questions, please let me know. thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/