Re: [ofa-general] [PATCH] IB/iSER: Add module param to count alignment violations
Mike Christie wrote: > Pete Wyckoff wrote: >> [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. >> > > 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. > Nevermind I see the other mail. --~--~-~--~~~---~--~~ 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 -~--~~~~--~~--~--~---
Re: [ofa-general] [PATCH] IB/iSER: Add module param to count alignment violations
Pete Wyckoff wrote: > [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. > 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. --~--~-~--~~~---~--~~ 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 -~--~~~~--~~--~--~---
Re: [ofa-general] [PATCH] IB/iSER: Add module param to count alignment violations
[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 -~--~~~~--~~--~--~---