[ofa-general] [PATCH] IB/iSER: Add module param to count alignment violations
Mike Christie
michaelc at cs.wisc.edu
Wed Apr 30 09:27:22 PDT 2008
Pete Wyckoff wrote:
> rdreier at cisco.com 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.
>
We have iscsi stats already. I thought Eli sent a patch to put this
there already? If not then put this in the get_stats callout as one of
the iser custom values.
More information about the general
mailing list