[EMAIL PROTECTED] wrote on Mon, 28 Apr 2008 08:51 -0700:
>  > Add read only module param to count alignment violations.
> 
> I don't think a module parameter is the way to report statistics from
> the kernel.  Can't you just add a device attribute or something?  Or
> stick a file in debugfs?

This is definitely a worthwhile change though.  By monitoring this
statistic we were able to get good insight to what our apps are
doing to cause these alignment violations.

I have a hacky patch that tries to export it via sysfs, but it
doesn't clean up properly.  The iscsi transport class defines the
sysfs tree and doesn't give hooks to a particular device to
add/change those entries, which is why this approach came out rather
ugly.  Hope Eli is willing to do this the right way; maybe debugfs
is the way to go.

                -- Pete

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To post to this group, send email to open-iscsi@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/open-iscsi
-~----------~----~----~----~------~----~------~--~---

Reply via email to