From: Mike Christie <[email protected]<mailto:[email protected]>> Date: Friday, 4 March 2016 6:11 am To: "[email protected]<mailto:[email protected]>" <[email protected]<mailto:[email protected]>> Cc: Shyam Sundar <[email protected]<mailto:[email protected]>>, Saurav Kashyap <[email protected]<mailto:[email protected]>>, Adheer Chandravanshi <[email protected]<mailto:[email protected]>> Subject: Re: [PATCH] iscsi tools: Add support for some host statistics
On 03/02/2016 02:04 AM, [email protected]<mailto:[email protected]> wrote: From: Adheer Chandravanshi <[email protected]<mailto:[email protected]>> Add support to maintain and show some host statistics in iscsid. This provides following host specific stats: iscsi_login_accept_rsps iscsi_login_other_fail_rsps iscsi_login_negotiate_fails iscsi_login_authenticate_fails iscsi_login_auth_fail_rsps iscsi_login_redirect_rsps iscsi_logout_normals iscsi_logout_others iscsi_connection_timeout_errors iscsi_session_failures What happened? I actually liked your guys's originally attempt where it was kernel based better. We think that putting the host statistics in userspace looks cleaner and is also inline with current design of partial offload solutions (bnx2i and likes) and software initiator. The stats to be supported currently can be easily managed with this patch as they are updated with the current PDU parsing flow in userspace. How are you going to support qla4xxx session with this approach? Were you going to do this and also a kernel interface for that type of driver? For qla4xxx, it already has support for host stats through current offload host stats interface. And in case its needed, we can support extra stats using iscsi_host_stats_custom field of struct iscsi_offload_host_stats. For this userspace based patch, what was the last issue with it? Were you guys having some issue about when to add the host or delete it or something and did you solve the issue? Yes, it was related to host deletion and dropping the reference. It has been solved by listening to udev events for iscsi_host removal. As correctly pointed out by Chris, following points should have been added to description. 1) adds a refcounted iscsi_host structure 2) adds a uevent listener to iscsid Thanks, Adheer -- You received this message because you are subscribed to the Google Groups "open-iscsi" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/open-iscsi. For more options, visit https://groups.google.com/d/optout.
