* Patch - To display FCoE Host information and statistics in sysfs
@ 2009-04-18 1:01 Giridhar Malavali
[not found] ` <0DC3E4D6-2BD9-4A1B-9A70-5395FEAC1F04@qlogic.com>
0 siblings, 1 reply; 6+ messages in thread
From: Giridhar Malavali @ 2009-04-18 1:01 UTC (permalink / raw)
To: James Smart; +Cc: linux-scsi
Hi All,
Attached is the patch for displaying FCoE Host bus adapter information
in sysfs.
This patch adds fcoe directory (/sys/class/fc_host/hostX/fcoe) and a
statistics sub-directory (/sys/class/fc_host/hostX/fcoe/statistics) to
display FCoE host specific and it statistics respectively.
A new port type definition called VF_PORT (Virtual Fabric Port) is
added to identify the virtual link between ENode and the FCF switch.
The contents in the fcoe and its sub-directory will be valid, if the
FC port type is a VF_PORT.
Following are the members added to display host specific information
under /sys/class/fc_host/hostX/fcoe directory
1) enode_mac_address - The burnt in MAC address used by the FCoE
adapter
2) vn_port_mac_address - The current MAC address used by the FCoE
adapter. This can be same as ENode MAC address or different based on
the MAC addressing mode.
3) vf_port_mac_address - MAC address of the FCF.
4) mac_addressing_mode - The type of FCoE MAC addressing mode
FCF_SELECTED - The FCF selects either SPMA or FPMA addressing mode
SPMA_ONLY - Only SPMA addressing mode is supported by ENode
FPMA_ONLY - Only FPMA addressing mode is supported by ENode
SPMA_PREFERRED - SPMA addressing mode is preferred by ENode
FPMA_PREFERRED - FPMA addressing mode is preferred by ENode
5) vlan_id - The Vlan ID used
Following counters are added to display host statistics information
under /sys/class/fc_host/hostX/fcoe/statistics directory
The statistics information displayed is similar to FC host
statistics. The FCoE statistics carries the same naming as FC host
statistics but the information displayed are mapped to related MAC
statistics.
This information is based on the proposal made to T11 for mapping FC
LESB ( Link Error Status Block) to FC_BB_E. Please refer to FCoE Link
Error Status Block (09-204v1.pdf) document at www.t11.org/index.html.
The various counters are
1) Invalid_crc_count
2) Invalid_tx_word_count
3) link_failure_count
4) loss_of_signal_count
5) loss_of_sync_count
6) prim_seq_protocol_error.
Thanks,
Giridhar.M.B
From: Giridhar Malavali <giridhar.malavali@qlogic.com>
Date: Fri, 17 Apr 2009 15:51:42 -0700
Subject: [PATCH 1/2] Addition of FCoE information to sysfs.
The information contains both FCoE host specific as well as its
statistics.
The FCoE specific information are displayed under /sys/class/fc_host/
hostX/fcoe
and /sys/class/fc_host/hostX/fcoe/statistics directories respectively.
Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
---
drivers/scsi/scsi_transport_fc.c | 292 +++++++++++++++++++++++++++++
++++-----
include/scsi/scsi_transport_fc.h | 53 +++++++
2 files changed, 311 insertions(+), 34 deletions(-)
diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/
scsi_transport_fc.c
index a152f89..55e3757 100644
--- a/drivers/scsi/scsi_transport_fc.c
+++ b/drivers/scsi/scsi_transport_fc.c
@@ -44,6 +44,18 @@ static int fc_vport_setup(struct Scsi_Host *shost,
int channel,
struct device *pdev, struct fc_vport_identifiers *ids,
struct fc_vport **vport);
+/* Kobjects for /sys/class/fc_host/hostx/fcoe
+ * and /sys/class/fc_host/hostx/fcoe/statistics objects
+ */
+struct kobject *private_fcoe_kobj;
+struct kobject *private_fcoe_stat_kobj;
+
+/* Kobject attributes defined for /sys/class/fc_host/hostx/fcoe
+ * and /sys/class/fc_host/hostx/fcoe/statistics objects
+ */
+#define FCOE_KOBJECT_ATTR(_prefix,_name,_mode,_show,_store) \
+struct kobj_attribute kobject_attr_##_prefix##_##_name = \
+ __ATTR(_name,_mode,_show,_store)
/*
* Redefine so that we can have same named attributes in the
* sdev/starget/host objects.
@@ -97,6 +109,7 @@ static struct {
{ FC_PORTTYPE_LPORT, "LPort (private loop)" },
{ FC_PORTTYPE_PTP, "Point-To-Point (direct nport connection)" },
{ FC_PORTTYPE_NPIV, "NPIV VPORT" },
+ { FC_PORTTYPE_VFPORT, "VFPort (Virtual fabric via point-to-point)" },
};
fc_enum_name_search(port_type, fc_port_type, fc_port_type_names)
#define FC_PORTTYPE_MAX_NAMELEN 50
@@ -414,12 +427,6 @@ static int fc_host_setup(struct
transport_container *tc, struct device *dev,
return 0;
}
-static DECLARE_TRANSPORT_CLASS(fc_host_class,
- "fc_host",
- fc_host_setup,
- NULL,
- NULL);
-
/*
* Setup and Remove actions for remote ports are handled
* in the service functions below.
@@ -617,34 +624,6 @@ send_vendor_fail:
}
EXPORT_SYMBOL(fc_host_post_vendor_event);
-
-
-static __init int fc_transport_init(void)
-{
- int error;
-
- atomic_set(&fc_event_seq, 0);
-
- error = transport_class_register(&fc_host_class);
- if (error)
- return error;
- error = transport_class_register(&fc_vport_class);
- if (error)
- return error;
- error = transport_class_register(&fc_rport_class);
- if (error)
- return error;
- return transport_class_register(&fc_transport_class);
-}
-
-static void __exit fc_transport_exit(void)
-{
- transport_class_unregister(&fc_transport_class);
- transport_class_unregister(&fc_rport_class);
- transport_class_unregister(&fc_host_class);
- transport_class_unregister(&fc_vport_class);
-}
-
/*
* FC Remote Port Attribute Management
*/
@@ -1655,6 +1634,224 @@ static struct attribute_group
fc_statistics_group = {
.attrs = fc_statistics_attrs,
};
+/*
+ * FCoE Host attributes
+ */
+
+/* Convert mac_addressing_mode values to ascii string name */
+static struct {
+ enum fcoe_mac_addressing_mode value;
+ char *name;
+} mac_addressing_mode_names [] = {
+ { FCF_SELECTED, "FCF selcted - Either FPMA or SPMA" },
+ { SPMA_ONLY, "SPMA only - ENode Solicites SPMA only support" },
+ { FPMA_ONLY, "FPMA only - ENode Solicites SPMA only support" },
+ { SPMA_PREFERRED, "SPMA preferred - ENode Solicites SPMA preferred
support" },
+ { FPMA_PREFERRED, "FPMA preferred - ENode Solicites FPMA preferred
support" },
+};
+fc_enum_name_search(mac_addressing_mode, fcoe_mac_addressing_mode,
mac_addressing_mode_names)
+#define FCOE_MAC_ADDR_MODE_MAX_NAMELEN 80
+
+/* Show MAC address members of the FCoE host attributes */
+static ssize_t
+fcoe_host_attrs_show(const struct kobject *kobj, char *buf, unsigned
long offset)
+{
+ struct device *dev = kobj_to_dev(kobj->parent);
+ struct Scsi_Host *shost = transport_class_to_shost(dev);
+ struct fc_internal *i = to_fc_internal(shost->transportt);
+ struct fcoe_host_attrs *stats;
+ ssize_t ret = -ENOENT;
+
+ if (i->f->get_fcoe_host) {
+ stats = (i->f->get_fcoe_host)(shost);
+ if (stats)
+ ret = snprintf(buf, 20, "0x%x\n",
+ *(u16 *)(((u8 *) stats) + offset));
+ }
+ return ret;
+}
+
+/* generate a read-only FCoE Host attribute */
+#define fcoe_host_attrs(name) \
+static ssize_t show_fcoe_host_attrs_##name(struct kobject *cd, \
+ struct kobj_attribute *attr, \
+ char *buf) \
+{ \
+ return fcoe_host_attrs_show(cd, buf, \
+ offsetof(struct fcoe_host_attrs, name)); \
+} \
+static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
show_fcoe_host_attrs_##name, NULL)
+
+fcoe_host_attrs(vlan_id);
+
+#define fcoe_host_attrs_enum_attr(title, maxlen) \
+static ssize_t show_fcoe_host_attrs_##title(struct kobject *kobj, \
+ struct kobj_attribute *attr, \
+ char *buf) \
+{ \
+ struct device *dev = kobj_to_dev(kobj->parent); \
+ struct Scsi_Host *shost = transport_class_to_shost(dev); \
+ struct fc_internal *i = to_fc_internal(shost->transportt); \
+ struct fcoe_host_attrs *stats; \
+ ssize_t ret = -EINVAL; \
+ const char *name; \
+ \
+ if (i->f->get_fcoe_host) { \
+ stats = (i->f->get_fcoe_host)(shost); \
+ if(stats) { \
+ name = get_fc_##title##_name(stats->mac_addressing_mode); \
+ if (name) \
+ ret= snprintf(buf, maxlen, "%s\n", name); \
+ } \
+ } \
+ return ret; \
+} \
+static FCOE_KOBJECT_ATTR(fcoe, title, S_IRUGO,
show_fcoe_host_attrs_##title, NULL)
+
+fcoe_host_attrs_enum_attr(mac_addressing_mode,
FCOE_MAC_ADDR_MODE_MAX_NAMELEN);
+
+static ssize_t
+fcoe_mac_addr_show(const struct kobject *kobj, char *buf, unsigned
long offset)
+{
+ struct device *dev = kobj_to_dev(kobj->parent);
+ struct Scsi_Host *shost = transport_class_to_shost(dev);
+ struct fc_internal *i = to_fc_internal(shost->transportt);
+ struct fcoe_host_attrs *stats;
+ ssize_t ret = -ENOENT;
+
+ if (offset > sizeof(struct fcoe_host_attrs))
+ WARN_ON(1);
+
+ if (i->f->get_fcoe_host) {
+ stats = (i->f->get_fcoe_host)(shost);
+ if (stats) {
+ u8 *macaddr = (u8*)(((u8 *) stats) + offset);
+ ret = sysfs_format_mac(buf, macaddr, 6);
+ }
+ }
+ return ret;
+}
+
+/* generate a read-only FCoE MAC address host attribute */
+#define fcoe_mac_addr(name) \
+static ssize_t show_fcoe_mac_addr_##name(struct kobject *cd, \
+ struct kobj_attribute *attr, \
+ char *buf) \
+{ \
+ return fcoe_mac_addr_show(cd, buf, \
+ offsetof(struct fcoe_host_attrs, name)); \
+} \
+static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
show_fcoe_mac_addr_##name, NULL)
+
+fcoe_mac_addr(enode_mac_address);
+fcoe_mac_addr(vn_port_mac_address);
+fcoe_mac_addr(vf_port_mac_address);
+
+static struct attribute *fcoe_host_attrs[] = {
+ &kobject_attr_fcoe_enode_mac_address.attr,
+ &kobject_attr_fcoe_vn_port_mac_address.attr,
+ &kobject_attr_fcoe_vf_port_mac_address.attr,
+ &kobject_attr_fcoe_vlan_id.attr,
+ &kobject_attr_fcoe_mac_addressing_mode.attr,
+ NULL
+};
+
+static struct attribute_group fcoe_host_attrs_group = {
+ .attrs = fcoe_host_attrs,
+};
+
+/* Read FCoE host statistics */
+static ssize_t
+fcoe_host_statistics_show(const struct kobject *kobj, char *buf,
unsigned long offset)
+{
+ struct device *dev = kobj_to_dev(kobj->parent->parent);
+ struct Scsi_Host *shost = transport_class_to_shost(dev);
+ struct fc_internal *i = to_fc_internal(shost->transportt);
+ struct fcoe_host_statistics *stats;
+ ssize_t ret = -ENOENT;
+
+ if (offset > sizeof(struct fcoe_host_statistics) ||
+ offset % sizeof(u64) != 0)
+ WARN_ON(1);
+
+ if (i->f->get_fcoe_host_stats) {
+ stats = (i->f->get_fcoe_host_stats)(shost);
+ if (stats)
+ ret = snprintf(buf, 20, "0x%llx\n",
+ (unsigned long long)*(u64 *)(((u8 *) stats) + offset));
+ }
+ return ret;
+}
+
+/* generate a read-only FCoE host staistics attribute */
+#define fcoe_host_statistics(name) \
+static ssize_t show_fcoe_host_statistics_##name(struct kobject *cd, \
+ struct kobj_attribute *attr, \
+ char *buf) \
+{ \
+ return fcoe_host_statistics_show(cd, buf, \
+ offsetof(struct fcoe_host_statistics, name)); \
+} \
+static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
show_fcoe_host_statistics_##name, NULL)
+
+fcoe_host_statistics(link_failure_count);
+fcoe_host_statistics(loss_of_sync_count);
+fcoe_host_statistics(loss_of_signal_count);
+fcoe_host_statistics(prim_seq_protocol_err_count);
+fcoe_host_statistics(invalid_tx_word_count);
+fcoe_host_statistics(invalid_crc_count);
+
+static struct attribute *fcoe_host_statistics[] = {
+ &kobject_attr_fcoe_link_failure_count.attr,
+ &kobject_attr_fcoe_loss_of_sync_count.attr,
+ &kobject_attr_fcoe_loss_of_signal_count.attr,
+ &kobject_attr_fcoe_prim_seq_protocol_err_count.attr,
+ &kobject_attr_fcoe_invalid_tx_word_count.attr,
+ &kobject_attr_fcoe_invalid_crc_count.attr,
+ NULL
+};
+
+static struct attribute_group fcoe_host_statistics_group = {
+ .attrs = fcoe_host_statistics,
+};
+
+static int fc_host_configure(struct transport_container *tc, struct
device *dev,
+ struct device *cdev)
+{
+ int error;
+
+ /* Create a fcoe sub-directory under /sys/class/fc_host/hostX/ */
+ private_fcoe_kobj = kobject_create_and_add("fcoe", &cdev->kobj);
+ if (private_fcoe_kobj) {
+ /* Create the files associated with this kobject */
+ error = sysfs_create_group(private_fcoe_kobj,
&fcoe_host_attrs_group);
+ if (error) {
+ kobject_put(private_fcoe_kobj);
+ goto error;
+ }
+ }
+
+ /* Create a statistics sub-directory under /sys/class/fc_host/hostX/
fcoe/ */
+ private_fcoe_stat_kobj = kobject_create_and_add("statistics",
private_fcoe_kobj);
+ if (private_fcoe_stat_kobj) {
+ /* Create the files associated with this kobject */
+ error = sysfs_create_group(private_fcoe_stat_kobj,
&fcoe_host_statistics_group);
+ if (error) {
+ kobject_put(private_fcoe_stat_kobj);
+ goto error;
+ }
+ }
+
+error:
+ return 0;
+}
+
+static DECLARE_TRANSPORT_CLASS(fc_host_class,
+ "fc_host",
+ fc_host_setup,
+ NULL,
+ fc_host_configure);
+
/* Host Vport Attributes */
@@ -3342,6 +3539,33 @@ fc_vport_sched_delete(struct work_struct *work)
vport->channel, stat);
}
+static __init int fc_transport_init(void)
+{
+ int error;
+
+ atomic_set(&fc_event_seq, 0);
+
+ error = transport_class_register(&fc_host_class);
+ if (error)
+ return error;
+ error = transport_class_register(&fc_vport_class);
+ if (error)
+ return error;
+ error = transport_class_register(&fc_rport_class);
+ if (error)
+ return error;
+ return transport_class_register(&fc_transport_class);
+}
+
+static void __exit fc_transport_exit(void)
+{
+ transport_class_unregister(&fc_transport_class);
+ transport_class_unregister(&fc_rport_class);
+ transport_class_unregister(&fc_host_class);
+ transport_class_unregister(&fc_vport_class);
+ kobject_put(private_fcoe_kobj);
+ kobject_put(private_fcoe_stat_kobj);
+}
/* Original Author: Martin Hicks */
MODULE_AUTHOR("James Smart");
diff --git a/include/scsi/scsi_transport_fc.h b/include/scsi/
scsi_transport_fc.h
index c9184f7..def2e23 100644
--- a/include/scsi/scsi_transport_fc.h
+++ b/include/scsi/scsi_transport_fc.h
@@ -30,6 +30,8 @@
#include <linux/sched.h>
#include <scsi/scsi.h>
#include <scsi/scsi_netlink.h>
+#include <scsi/scsi_netlink.h>
+#include <linux/etherdevice.h>
struct scsi_transport_template;
@@ -63,6 +65,7 @@ enum fc_port_type {
FC_PORTTYPE_LPORT, /* (Private) Loop w/o FLPort */
FC_PORTTYPE_PTP, /* Point to Point w/ another NPort */
FC_PORTTYPE_NPIV, /* VPORT based on NPIV */
+ FC_PORTTYPE_VFPORT, /* Virtual Fabric PORT */
};
@@ -428,6 +431,53 @@ struct fc_host_statistics {
u64 fcp_output_megabytes;
};
+/*
+ * FCoE Local Port (Host) Attributes
+ *
+ * Fixed attributes are not expected to change. The driver is
+ * expected to set these values after successfully calling
scsi_add_host().
+ * The transport fully manages all get functions w/o driver
interaction.
+ *
+ * Dynamic attributes are expected to change. The driver participates
+ * in all get/set operations via functions provided by the driver.
+ *
+ */
+struct fcoe_host_attrs {
+ /* Fixed attributes */
+ uint8_t enode_mac_address[ETH_ALEN];
+
+ /* Dynamic attributes */
+ uint8_t vn_port_mac_address[ETH_ALEN];
+ uint8_t vf_port_mac_address[ETH_ALEN];
+ u16 vlan_id;
+ u16 mac_addressing_mode;
+};
+
+/* FCoE Statistics - Following proposal for FC_BB_E FC Link error
status block representation
+ * from T11/09-204v0 guidelines
+ */
+struct fcoe_host_statistics {
+ /* Port statistics */
+ u64 link_failure_count;
+ u64 loss_of_sync_count;
+ u64 loss_of_signal_count;
+ u64 prim_seq_protocol_err_count;
+ u64 invalid_tx_word_count;
+ u64 invalid_crc_count;
+};
+
+
+/*
+ * mac_addressing_mode: If you alter this, you also need to alter
+ * scsi_transport_fc.c (for the ascii descriptions).
+ */
+enum fcoe_mac_addressing_mode {
+ FCF_SELECTED,
+ SPMA_ONLY,
+ FPMA_ONLY,
+ SPMA_PREFERRED,
+ FPMA_PREFERRED,
+};
/*
* FC Event Codes - Polled and Async, following FC HBAAPI v2.0
guidelines
@@ -600,6 +650,9 @@ struct fc_function_template {
struct fc_host_statistics * (*get_fc_host_stats)(struct Scsi_Host *);
void (*reset_fc_host_stats)(struct Scsi_Host *);
+ struct fcoe_host_attrs* (*get_fcoe_host)(struct Scsi_Host *);
+ struct fcoe_host_statistics* (*get_fcoe_host_stats)(struct Scsi_Host
*);
+
int (*issue_fc_host_lip)(struct Scsi_Host *);
void (*dev_loss_tmo_callbk)(struct fc_rport *);
--
1.6.2.rc1.30.gd43c
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: Patch - To display FCoE Host information and statistics in sysfs
[not found] ` <49F9D93A.9070906@emulex.com>
@ 2009-05-05 7:07 ` Giridhar Malavali
2009-06-02 17:34 ` Giridhar Malavali
0 siblings, 1 reply; 6+ messages in thread
From: Giridhar Malavali @ 2009-05-05 7:07 UTC (permalink / raw)
To: James Smart; +Cc: James Bottomley, linux-scsi
James,
By opening the FC as a bus, you meant a sysfs layout similar to
present scsi layout in 2.6 kernel (/sys/bus/scsi).
Below are the layout details that I had in my mind
/sys/bus/fc/devices/ - list of all devices
/sys/bus/fc/drivers/fc and fcoe ( links to all fc and fcoe hosts and
targets )
/sys/class/fc/fc_host/
/sys/class/fc/fc_remote_ports
/sys/class/fc/fcoe_host ( Initiator )
/sys/class/fc/fcoe_remote_ports ( Target )
Please comment whether I understood it right?
This makes things lot clear for me. This conveys better distinction
between FC and FCoE adapters present in the system. The FC and FCoE
specific details can be kept under respective directory and links can
be created to map their relations.
As stated by you, the scope of these changes looks bit heavy. I feel,
it will be good to add these changes incrementally. Keeping long term
perspective, is it a good idea to add FCoE related information as a
separate directory parallel to fc_host in /sys/class directory. I feel
this leads to easier migration when FC is opened up as a bus in future.
Thanks,
Giridhar.M.B
On Apr 30, 2009, at 10:00 AM, James Smart wrote:
> Giridhar,
>
> Yes I saw it. I'm really a little unsure. At LSF, we talked about the
> need for FC to open up as a bus, thus change the way the topology
> objects are created, and FCOE affects that as the attributes are
> pre-role (role being initiator or tgt or whatever) but where you are
> placing them is in a role-based location. So, if we say yes, as the
> set
> is rather small, then it starts us off on a divergence from this
> longer-term view. Yes, we can survive, but I hate telling mgmt code to
> deal with both variants in the future.
>
> -- james
>
>
>
> Giridhar Malavali wrote:
>> Hi James,
>>
>> I had submitted the following patch last week. I did not hear any
>> thing from you or from the community. Just a friendly check to find
>> out whether this patch looks fine?
>>
>> Thanks,
>> Giridhar.M.B
>>
>>
>>
>>
>>
>> Begin forwarded message:
>>
>>
>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>> Date: April 17, 2009 6:01:30 PM PDT
>>> To: James Smart <James.Smart@Emulex.Com>
>>> Cc: linux-scsi@vger.kernel.org
>>> Subject: Patch - To display FCoE Host information and statistics in
>>> sysfs
>>>
>>> Hi All,
>>>
>>> Attached is the patch for displaying FCoE Host bus adapter
>>> information in sysfs.
>>>
>>> This patch adds fcoe directory (/sys/class/fc_host/hostX/fcoe) and a
>>> statistics sub-directory (/sys/class/fc_host/hostX/fcoe/statistics)
>>> to display FCoE host specific and it statistics respectively.
>>>
>>> A new port type definition called VF_PORT (Virtual Fabric Port) is
>>> added to identify the virtual link between ENode and the FCF switch.
>>> The contents in the fcoe and its sub-directory will be valid, if the
>>> FC port type is a VF_PORT.
>>>
>>> Following are the members added to display host specific information
>>> under /sys/class/fc_host/hostX/fcoe directory
>>>
>>> 1) enode_mac_address - The burnt in MAC address used by the
>>> FCoE
>>> adapter
>>> 2) vn_port_mac_address - The current MAC address used by the
>>> FCoE
>>> adapter. This can be same as ENode MAC address or different based on
>>> the MAC addressing mode.
>>> 3) vf_port_mac_address - MAC address of the FCF.
>>> 4) mac_addressing_mode - The type of FCoE MAC addressing mode
>>> FCF_SELECTED - The FCF selects either SPMA or FPMA
>>> addressing mode
>>> SPMA_ONLY - Only SPMA addressing mode is supported by
>>> ENode
>>> FPMA_ONLY - Only FPMA addressing mode is supported by
>>> ENode
>>> SPMA_PREFERRED - SPMA addressing mode is preferred by
>>> ENode
>>> FPMA_PREFERRED - FPMA addressing mode is preferred by
>>> ENode
>>> 5) vlan_id - The Vlan ID used
>>>
>>> Following counters are added to display host statistics information
>>> under /sys/class/fc_host/hostX/fcoe/statistics directory
>>>
>>> The statistics information displayed is similar to FC host
>>> statistics. The FCoE statistics carries the same naming as FC host
>>> statistics but the information displayed are mapped to related MAC
>>> statistics.
>>>
>>> This information is based on the proposal made to T11 for
>>> mapping
>>> FC LESB ( Link Error Status Block) to FC_BB_E. Please refer to FCoE
>>> Link Error Status Block (09-204v1.pdf) document at www.t11.org/index.html
>>> .
>>>
>>> The various counters are
>>>
>>> 1) Invalid_crc_count
>>> 2) Invalid_tx_word_count
>>> 3) link_failure_count
>>> 4) loss_of_signal_count
>>> 5) loss_of_sync_count
>>> 6) prim_seq_protocol_error.
>>>
>>> Thanks,
>>> Giridhar.M.B
>>>
>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>> Date: Fri, 17 Apr 2009 15:51:42 -0700
>>> Subject: [PATCH 1/2] Addition of FCoE information to sysfs.
>>>
>>> The information contains both FCoE host specific as well as its
>>> statistics.
>>> The FCoE specific information are displayed under /sys/class/
>>> fc_host/
>>> hostX/fcoe
>>> and /sys/class/fc_host/hostX/fcoe/statistics directories
>>> respectively.
>>>
>>> Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>> ---
>>> drivers/scsi/scsi_transport_fc.c | 292 +++++++++++++++++++++++++++
>>> ++
>>> ++++-----
>>> include/scsi/scsi_transport_fc.h | 53 +++++++
>>> 2 files changed, 311 insertions(+), 34 deletions(-)
>>>
>>> diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/
>>> scsi_transport_fc.c
>>> index a152f89..55e3757 100644
>>> --- a/drivers/scsi/scsi_transport_fc.c
>>> +++ b/drivers/scsi/scsi_transport_fc.c
>>> @@ -44,6 +44,18 @@ static int fc_vport_setup(struct Scsi_Host
>>> *shost, int channel,
>>> struct device *pdev, struct fc_vport_identifiers *ids,
>>> struct fc_vport **vport);
>>>
>>> +/* Kobjects for /sys/class/fc_host/hostx/fcoe
>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>> + */
>>> +struct kobject *private_fcoe_kobj;
>>> +struct kobject *private_fcoe_stat_kobj;
>>> +
>>> +/* Kobject attributes defined for /sys/class/fc_host/hostx/fcoe
>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>> + */
>>> +#define
>>> FCOE_KOBJECT_ATTR(_prefix,_name,_mode,_show,_store) \
>>> +struct kobj_attribute kobject_attr_##_prefix##_##_name
>>> = \
>>> + __ATTR(_name,_mode,_show,_store)
>>> /*
>>> * Redefine so that we can have same named attributes in the
>>> * sdev/starget/host objects.
>>> @@ -97,6 +109,7 @@ static struct {
>>> { FC_PORTTYPE_LPORT, "LPort (private loop)" },
>>> { FC_PORTTYPE_PTP, "Point-To-Point (direct nport
>>> connection)" },
>>> { FC_PORTTYPE_NPIV, "NPIV VPORT" },
>>> + { FC_PORTTYPE_VFPORT, "VFPort (Virtual fabric via
>>> point-to-
>>> point)" },
>>> };
>>> fc_enum_name_search(port_type, fc_port_type, fc_port_type_names)
>>> #define FC_PORTTYPE_MAX_NAMELEN 50
>>> @@ -414,12 +427,6 @@ static int fc_host_setup(struct
>>> transport_container *tc, struct device *dev,
>>> return 0;
>>> }
>>>
>>> -static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>> - "fc_host",
>>> - fc_host_setup,
>>> - NULL,
>>> - NULL);
>>> -
>>> /*
>>> * Setup and Remove actions for remote ports are handled
>>> * in the service functions below.
>>> @@ -617,34 +624,6 @@ send_vendor_fail:
>>> }
>>> EXPORT_SYMBOL(fc_host_post_vendor_event);
>>>
>>> -
>>> -
>>> -static __init int fc_transport_init(void)
>>> -{
>>> - int error;
>>> -
>>> - atomic_set(&fc_event_seq, 0);
>>> -
>>> - error = transport_class_register(&fc_host_class);
>>> - if (error)
>>> - return error;
>>> - error = transport_class_register(&fc_vport_class);
>>> - if (error)
>>> - return error;
>>> - error = transport_class_register(&fc_rport_class);
>>> - if (error)
>>> - return error;
>>> - return transport_class_register(&fc_transport_class);
>>> -}
>>> -
>>> -static void __exit fc_transport_exit(void)
>>> -{
>>> - transport_class_unregister(&fc_transport_class);
>>> - transport_class_unregister(&fc_rport_class);
>>> - transport_class_unregister(&fc_host_class);
>>> - transport_class_unregister(&fc_vport_class);
>>> -}
>>> -
>>> /*
>>> * FC Remote Port Attribute Management
>>> */
>>> @@ -1655,6 +1634,224 @@ static struct attribute_group
>>> fc_statistics_group = {
>>> .attrs = fc_statistics_attrs,
>>> };
>>>
>>> +/*
>>> + * FCoE Host attributes
>>> + */
>>> +
>>> +/* Convert mac_addressing_mode values to ascii string name */
>>> +static struct {
>>> + enum fcoe_mac_addressing_mode value;
>>> + char *name;
>>> +} mac_addressing_mode_names [] = {
>>> + { FCF_SELECTED, "FCF selcted - Either FPMA
>>> or SPMA" },
>>> + { SPMA_ONLY, "SPMA only - ENode Solicites
>>> SPMA only support" },
>>> + { FPMA_ONLY, "FPMA only - ENode Solicites
>>> SPMA only support" },
>>> + { SPMA_PREFERRED, "SPMA preferred - ENode
>>> Solicites SPMA
>>> preferred support" },
>>> + { FPMA_PREFERRED, "FPMA preferred - ENode
>>> Solicites FPMA
>>> preferred support" },
>>> +};
>>> +fc_enum_name_search(mac_addressing_mode, fcoe_mac_addressing_mode,
>>> mac_addressing_mode_names)
>>> +#define FCOE_MAC_ADDR_MODE_MAX_NAMELEN 80
>>> +
>>> +/* Show MAC address members of the FCoE host attributes */
>>> +static ssize_t
>>> +fcoe_host_attrs_show(const struct kobject *kobj, char *buf,
>>> unsigned long offset)
>>> +{
>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>> + struct fcoe_host_attrs *stats;
>>> + ssize_t ret = -ENOENT;
>>> +
>>> + if (i->f->get_fcoe_host) {
>>> + stats = (i->f->get_fcoe_host)(shost);
>>> + if (stats)
>>> + ret = snprintf(buf, 20, "0x%x\n",
>>> + *(u16 *)(((u8 *) stats) + offset));
>>> + }
>>> + return ret;
>>> +}
>>> +
>>> +/* generate a read-only FCoE Host attribute */
>>> +#define
>>> fcoe_host_attrs
>>> (name) \
>>> +static ssize_t show_fcoe_host_attrs_##name(struct kobject
>>> *cd, \
>>> + struct kobj_attribute
>>> *attr, \
>>> + char
>>> *buf) \
>>> +
>>> { \
>>> + return fcoe_host_attrs_show(cd,
>>> buf, \
>>> + offsetof(struct fcoe_host_attrs,
>>> name)); \
>>> +
>>> } \
>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>> show_fcoe_host_attrs_##name, NULL)
>>> +
>>> +fcoe_host_attrs(vlan_id);
>>> +
>>> +#define fcoe_host_attrs_enum_attr(title,
>>> maxlen) \
>>> +static ssize_t show_fcoe_host_attrs_##title(struct kobject
>>> *kobj, \
>>> + struct kobj_attribute
>>> *attr, \
>>> + char
>>> *buf) \
>>> +
>>> { \
>>> + struct device *dev = kobj_to_dev(kobj-
>>> >parent); \
>>> + struct Scsi_Host *shost =
>>> transport_class_to_shost(dev); \
>>> + struct fc_internal *i = to_fc_internal(shost-
>>> >transportt); \
>>> + struct fcoe_host_attrs
>>> *stats; \
>>> + ssize_t ret = -
>>> EINVAL; \
>>> + const char
>>> *name; \
>>> +
>>> \
>>> + if (i->f->get_fcoe_host)
>>> { \
>>> + stats = (i->f->get_fcoe_host)
>>> (shost); \
>>> + if(stats)
>>> { \
>>> + name = get_fc_##title##_name(stats-
>>> >mac_addressing_mode); \
>>> + if
>>> (name) \
>>> + ret= snprintf(buf, maxlen, "%s\n",
>>> name); \
>>> +
>>> } \
>>> +
>>> } \
>>> + return
>>> ret
>>> ; \
>>> +
>>> } \
>>> +static FCOE_KOBJECT_ATTR(fcoe, title, S_IRUGO,
>>> show_fcoe_host_attrs_##title, NULL)
>>> +
>>> +fcoe_host_attrs_enum_attr(mac_addressing_mode,
>>> FCOE_MAC_ADDR_MODE_MAX_NAMELEN);
>>> +
>>> +static ssize_t
>>> +fcoe_mac_addr_show(const struct kobject *kobj, char *buf, unsigned
>>> long offset)
>>> +{
>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>> + struct fcoe_host_attrs *stats;
>>> + ssize_t ret = -ENOENT;
>>> +
>>> + if (offset > sizeof(struct fcoe_host_attrs))
>>> + WARN_ON(1);
>>> +
>>> + if (i->f->get_fcoe_host) {
>>> + stats = (i->f->get_fcoe_host)(shost);
>>> + if (stats) {
>>> + u8 *macaddr = (u8*)(((u8 *) stats) + offset);
>>> + ret = sysfs_format_mac(buf, macaddr, 6);
>>> + }
>>> + }
>>> + return ret;
>>> +}
>>> +
>>> +/* generate a read-only FCoE MAC address host attribute */
>>> +#define
>>> fcoe_mac_addr(name) \
>>> +static ssize_t show_fcoe_mac_addr_##name(struct kobject
>>> *cd, \
>>> + struct kobj_attribute
>>> *attr, \
>>> + char
>>> *buf) \
>>> +
>>> { \
>>> + return fcoe_mac_addr_show(cd,
>>> buf, \
>>> + offsetof(struct fcoe_host_attrs,
>>> name)); \
>>> +
>>> } \
>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>> show_fcoe_mac_addr_##name, NULL)
>>> +
>>> +fcoe_mac_addr(enode_mac_address);
>>> +fcoe_mac_addr(vn_port_mac_address);
>>> +fcoe_mac_addr(vf_port_mac_address);
>>> +
>>> +static struct attribute *fcoe_host_attrs[] = {
>>> + &kobject_attr_fcoe_enode_mac_address.attr,
>>> + &kobject_attr_fcoe_vn_port_mac_address.attr,
>>> + &kobject_attr_fcoe_vf_port_mac_address.attr,
>>> + &kobject_attr_fcoe_vlan_id.attr,
>>> + &kobject_attr_fcoe_mac_addressing_mode.attr,
>>> + NULL
>>> +};
>>> +
>>> +static struct attribute_group fcoe_host_attrs_group = {
>>> + .attrs = fcoe_host_attrs,
>>> +};
>>> +
>>> +/* Read FCoE host statistics */
>>> +static ssize_t
>>> +fcoe_host_statistics_show(const struct kobject *kobj, char *buf,
>>> unsigned long offset)
>>> +{
>>> + struct device *dev = kobj_to_dev(kobj->parent->parent);
>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>> + struct fcoe_host_statistics *stats;
>>> + ssize_t ret = -ENOENT;
>>> +
>>> + if (offset > sizeof(struct fcoe_host_statistics) ||
>>> + offset % sizeof(u64) != 0)
>>> + WARN_ON(1);
>>> +
>>> + if (i->f->get_fcoe_host_stats) {
>>> + stats = (i->f->get_fcoe_host_stats)(shost);
>>> + if (stats)
>>> + ret = snprintf(buf, 20, "0x%llx\n",
>>> + (unsigned long long)*(u64 *)(((u8 *)
>>> stats) + offset));
>>> + }
>>> + return ret;
>>> +}
>>> +
>>> +/* generate a read-only FCoE host staistics attribute */
>>> +#define
>>> fcoe_host_statistics
>>> (name) \
>>> +static ssize_t show_fcoe_host_statistics_##name(struct kobject
>>> *cd, \
>>> + struct kobj_attribute
>>> *attr, \
>>> + char
>>> *buf) \
>>> +
>>> { \
>>> + return fcoe_host_statistics_show(cd,
>>> buf, \
>>> + offsetof(struct fcoe_host_statistics,
>>> name)); \
>>> +
>>> } \
>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>> show_fcoe_host_statistics_##name, NULL)
>>> +
>>> +fcoe_host_statistics(link_failure_count);
>>> +fcoe_host_statistics(loss_of_sync_count);
>>> +fcoe_host_statistics(loss_of_signal_count);
>>> +fcoe_host_statistics(prim_seq_protocol_err_count);
>>> +fcoe_host_statistics(invalid_tx_word_count);
>>> +fcoe_host_statistics(invalid_crc_count);
>>> +
>>> +static struct attribute *fcoe_host_statistics[] = {
>>> + &kobject_attr_fcoe_link_failure_count.attr,
>>> + &kobject_attr_fcoe_loss_of_sync_count.attr,
>>> + &kobject_attr_fcoe_loss_of_signal_count.attr,
>>> + &kobject_attr_fcoe_prim_seq_protocol_err_count.attr,
>>> + &kobject_attr_fcoe_invalid_tx_word_count.attr,
>>> + &kobject_attr_fcoe_invalid_crc_count.attr,
>>> + NULL
>>> +};
>>> +
>>> +static struct attribute_group fcoe_host_statistics_group = {
>>> + .attrs = fcoe_host_statistics,
>>> +};
>>> +
>>> +static int fc_host_configure(struct transport_container *tc, struct
>>> device *dev,
>>> + struct device *cdev)
>>> +{
>>> + int error;
>>> +
>>> + /* Create a fcoe sub-directory under /sys/class/fc_host/
>>> hostX/ */
>>> + private_fcoe_kobj = kobject_create_and_add("fcoe", &cdev-
>>> >kobj);
>>> + if (private_fcoe_kobj) {
>>> + /* Create the files associated with this kobject */
>>> + error = sysfs_create_group(private_fcoe_kobj,
>>> &fcoe_host_attrs_group);
>>> + if (error) {
>>> + kobject_put(private_fcoe_kobj);
>>> + goto error;
>>> + }
>>> + }
>>> +
>>> + /* Create a statistics sub-directory under /sys/class/fc_host/
>>> hostX/fcoe/ */
>>> + private_fcoe_stat_kobj = kobject_create_and_add("statistics",
>>> private_fcoe_kobj);
>>> + if (private_fcoe_stat_kobj) {
>>> + /* Create the files associated with this kobject */
>>> + error = sysfs_create_group(private_fcoe_stat_kobj,
>>> &fcoe_host_statistics_group);
>>> + if (error) {
>>> + kobject_put(private_fcoe_stat_kobj);
>>> + goto error;
>>> + }
>>> + }
>>> +
>>> +error:
>>> + return 0;
>>> +}
>>> +
>>> +static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>> + "fc_host",
>>> + fc_host_setup,
>>> + NULL,
>>> + fc_host_configure);
>>> +
>>>
>>> /* Host Vport Attributes */
>>>
>>> @@ -3342,6 +3539,33 @@ fc_vport_sched_delete(struct work_struct
>>> *work)
>>> vport->channel, stat);
>>> }
>>>
>>> +static __init int fc_transport_init(void)
>>> +{
>>> + int error;
>>> +
>>> + atomic_set(&fc_event_seq, 0);
>>> +
>>> + error = transport_class_register(&fc_host_class);
>>> + if (error)
>>> + return error;
>>> + error = transport_class_register(&fc_vport_class);
>>> + if (error)
>>> + return error;
>>> + error = transport_class_register(&fc_rport_class);
>>> + if (error)
>>> + return error;
>>> + return transport_class_register(&fc_transport_class);
>>> +}
>>> +
>>> +static void __exit fc_transport_exit(void)
>>> +{
>>> + transport_class_unregister(&fc_transport_class);
>>> + transport_class_unregister(&fc_rport_class);
>>> + transport_class_unregister(&fc_host_class);
>>> + transport_class_unregister(&fc_vport_class);
>>> + kobject_put(private_fcoe_kobj);
>>> + kobject_put(private_fcoe_stat_kobj);
>>> +}
>>>
>>> /* Original Author: Martin Hicks */
>>> MODULE_AUTHOR("James Smart");
>>> diff --git a/include/scsi/scsi_transport_fc.h b/include/scsi/
>>> scsi_transport_fc.h
>>> index c9184f7..def2e23 100644
>>> --- a/include/scsi/scsi_transport_fc.h
>>> +++ b/include/scsi/scsi_transport_fc.h
>>> @@ -30,6 +30,8 @@
>>> #include <linux/sched.h>
>>> #include <scsi/scsi.h>
>>> #include <scsi/scsi_netlink.h>
>>> +#include <scsi/scsi_netlink.h>
>>> +#include <linux/etherdevice.h>
>>>
>>> struct scsi_transport_template;
>>>
>>> @@ -63,6 +65,7 @@ enum fc_port_type {
>>> FC_PORTTYPE_LPORT, /* (Private) Loop w/o FLPort */
>>> FC_PORTTYPE_PTP, /* Point to Point w/ another
>>> NPort */
>>> FC_PORTTYPE_NPIV, /* VPORT based on NPIV */
>>> + FC_PORTTYPE_VFPORT, /* Virtual Fabric PORT */
>>> };
>>>
>>>
>>> @@ -428,6 +431,53 @@ struct fc_host_statistics {
>>> u64 fcp_output_megabytes;
>>> };
>>>
>>> +/*
>>> + * FCoE Local Port (Host) Attributes
>>> + *
>>> + * Fixed attributes are not expected to change. The driver is
>>> + * expected to set these values after successfully calling
>>> scsi_add_host().
>>> + * The transport fully manages all get functions w/o driver
>>> interaction.
>>> + *
>>> + * Dynamic attributes are expected to change. The driver
>>> participates
>>> + * in all get/set operations via functions provided by the driver.
>>> + *
>>> + */
>>> +struct fcoe_host_attrs {
>>> + /* Fixed attributes */
>>> + uint8_t enode_mac_address[ETH_ALEN];
>>> +
>>> + /* Dynamic attributes */
>>> + uint8_t vn_port_mac_address[ETH_ALEN];
>>> + uint8_t vf_port_mac_address[ETH_ALEN];
>>> + u16 vlan_id;
>>> + u16 mac_addressing_mode;
>>> +};
>>> +
>>> +/* FCoE Statistics - Following proposal for FC_BB_E FC Link error
>>> status block representation
>>> + * from T11/09-204v0 guidelines
>>> + */
>>> +struct fcoe_host_statistics {
>>> + /* Port statistics */
>>> + u64 link_failure_count;
>>> + u64 loss_of_sync_count;
>>> + u64 loss_of_signal_count;
>>> + u64 prim_seq_protocol_err_count;
>>> + u64 invalid_tx_word_count;
>>> + u64 invalid_crc_count;
>>> +};
>>> +
>>> +
>>> +/*
>>> + * mac_addressing_mode: If you alter this, you also need to alter
>>> + * scsi_transport_fc.c (for the ascii descriptions).
>>> + */
>>> +enum fcoe_mac_addressing_mode {
>>> + FCF_SELECTED,
>>> + SPMA_ONLY,
>>> + FPMA_ONLY,
>>> + SPMA_PREFERRED,
>>> + FPMA_PREFERRED,
>>> +};
>>>
>>> /*
>>> * FC Event Codes - Polled and Async, following FC HBAAPI v2.0
>>> guidelines
>>> @@ -600,6 +650,9 @@ struct fc_function_template {
>>> struct fc_host_statistics * (*get_fc_host_stats)(struct
>>> Scsi_Host *);
>>> void (*reset_fc_host_stats)(struct Scsi_Host *);
>>>
>>> + struct fcoe_host_attrs* (*get_fcoe_host)(struct Scsi_Host *);
>>> + struct fcoe_host_statistics* (*get_fcoe_host_stats)(struct
>>> Scsi_Host *);
>>> +
>>> int (*issue_fc_host_lip)(struct Scsi_Host *);
>>>
>>> void (*dev_loss_tmo_callbk)(struct fc_rport *);
>>> --
>>> 1.6.2.rc1.30.gd43c
>>>
>>>
>>>
>>>
>>>
>>>
>>
>>
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Patch - To display FCoE Host information and statistics in sysfs
2009-05-05 7:07 ` Giridhar Malavali
@ 2009-06-02 17:34 ` Giridhar Malavali
2009-06-03 15:24 ` James Smart
0 siblings, 1 reply; 6+ messages in thread
From: Giridhar Malavali @ 2009-06-02 17:34 UTC (permalink / raw)
To: James Smart; +Cc: James Bottomley, linux-scsi
James,
Any thoughts about this.
Thanks,
Giridhar.M.B
On May 5, 2009, at 12:07 AM, Giridhar Malavali wrote:
> James,
>
> By opening the FC as a bus, you meant a sysfs layout similar to
> present scsi layout in 2.6 kernel (/sys/bus/scsi).
>
> Below are the layout details that I had in my mind
> /sys/bus/fc/devices/ - list of all devices
> /sys/bus/fc/drivers/fc and fcoe ( links to all fc and fcoe hosts and
> targets )
> /sys/class/fc/fc_host/
> /sys/class/fc/fc_remote_ports
> /sys/class/fc/fcoe_host ( Initiator )
> /sys/class/fc/fcoe_remote_ports ( Target )
>
> Please comment whether I understood it right?
>
> This makes things lot clear for me. This conveys better distinction
> between FC and FCoE adapters present in the system. The FC and FCoE
> specific details can be kept under respective directory and links
> can be created to map their relations.
>
> As stated by you, the scope of these changes looks bit heavy. I
> feel, it will be good to add these changes incrementally. Keeping
> long term perspective, is it a good idea to add FCoE related
> information as a separate directory parallel to fc_host in /sys/
> class directory. I feel this leads to easier migration when FC is
> opened up as a bus in future.
>
> Thanks,
> Giridhar.M.B
>
>
> On Apr 30, 2009, at 10:00 AM, James Smart wrote:
>
>> Giridhar,
>>
>> Yes I saw it. I'm really a little unsure. At LSF, we talked about the
>> need for FC to open up as a bus, thus change the way the topology
>> objects are created, and FCOE affects that as the attributes are
>> pre-role (role being initiator or tgt or whatever) but where you are
>> placing them is in a role-based location. So, if we say yes, as
>> the set
>> is rather small, then it starts us off on a divergence from this
>> longer-term view. Yes, we can survive, but I hate telling mgmt code
>> to
>> deal with both variants in the future.
>>
>> -- james
>>
>>
>>
>> Giridhar Malavali wrote:
>>> Hi James,
>>>
>>> I had submitted the following patch last week. I did not hear any
>>> thing from you or from the community. Just a friendly check to find
>>> out whether this patch looks fine?
>>>
>>> Thanks,
>>> Giridhar.M.B
>>>
>>>
>>>
>>>
>>>
>>> Begin forwarded message:
>>>
>>>
>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>> Date: April 17, 2009 6:01:30 PM PDT
>>>> To: James Smart <James.Smart@Emulex.Com>
>>>> Cc: linux-scsi@vger.kernel.org
>>>> Subject: Patch - To display FCoE Host information and statistics in
>>>> sysfs
>>>>
>>>> Hi All,
>>>>
>>>> Attached is the patch for displaying FCoE Host bus adapter
>>>> information in sysfs.
>>>>
>>>> This patch adds fcoe directory (/sys/class/fc_host/hostX/fcoe)
>>>> and a
>>>> statistics sub-directory (/sys/class/fc_host/hostX/fcoe/statistics)
>>>> to display FCoE host specific and it statistics respectively.
>>>>
>>>> A new port type definition called VF_PORT (Virtual Fabric Port) is
>>>> added to identify the virtual link between ENode and the FCF
>>>> switch.
>>>> The contents in the fcoe and its sub-directory will be valid, if
>>>> the
>>>> FC port type is a VF_PORT.
>>>>
>>>> Following are the members added to display host specific
>>>> information
>>>> under /sys/class/fc_host/hostX/fcoe directory
>>>>
>>>> 1) enode_mac_address - The burnt in MAC address used by the
>>>> FCoE
>>>> adapter
>>>> 2) vn_port_mac_address - The current MAC address used by the
>>>> FCoE
>>>> adapter. This can be same as ENode MAC address or different based
>>>> on
>>>> the MAC addressing mode.
>>>> 3) vf_port_mac_address - MAC address of the FCF.
>>>> 4) mac_addressing_mode - The type of FCoE MAC addressing mode
>>>> FCF_SELECTED - The FCF selects either SPMA or FPMA
>>>> addressing mode
>>>> SPMA_ONLY - Only SPMA addressing mode is supported by
>>>> ENode
>>>> FPMA_ONLY - Only FPMA addressing mode is supported by
>>>> ENode
>>>> SPMA_PREFERRED - SPMA addressing mode is preferred by
>>>> ENode
>>>> FPMA_PREFERRED - FPMA addressing mode is preferred by
>>>> ENode
>>>> 5) vlan_id - The Vlan ID used
>>>>
>>>> Following counters are added to display host statistics information
>>>> under /sys/class/fc_host/hostX/fcoe/statistics directory
>>>>
>>>> The statistics information displayed is similar to FC host
>>>> statistics. The FCoE statistics carries the same naming as FC host
>>>> statistics but the information displayed are mapped to related MAC
>>>> statistics.
>>>>
>>>> This information is based on the proposal made to T11 for
>>>> mapping
>>>> FC LESB ( Link Error Status Block) to FC_BB_E. Please refer to FCoE
>>>> Link Error Status Block (09-204v1.pdf) document at www.t11.org/index.html
>>>> .
>>>>
>>>> The various counters are
>>>>
>>>> 1) Invalid_crc_count
>>>> 2) Invalid_tx_word_count
>>>> 3) link_failure_count
>>>> 4) loss_of_signal_count
>>>> 5) loss_of_sync_count
>>>> 6) prim_seq_protocol_error.
>>>>
>>>> Thanks,
>>>> Giridhar.M.B
>>>>
>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>> Date: Fri, 17 Apr 2009 15:51:42 -0700
>>>> Subject: [PATCH 1/2] Addition of FCoE information to sysfs.
>>>>
>>>> The information contains both FCoE host specific as well as its
>>>> statistics.
>>>> The FCoE specific information are displayed under /sys/class/
>>>> fc_host/
>>>> hostX/fcoe
>>>> and /sys/class/fc_host/hostX/fcoe/statistics directories
>>>> respectively.
>>>>
>>>> Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>> ---
>>>> drivers/scsi/scsi_transport_fc.c | 292 ++++++++++++++++++++++++++
>>>> +++
>>>> ++++-----
>>>> include/scsi/scsi_transport_fc.h | 53 +++++++
>>>> 2 files changed, 311 insertions(+), 34 deletions(-)
>>>>
>>>> diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/
>>>> scsi_transport_fc.c
>>>> index a152f89..55e3757 100644
>>>> --- a/drivers/scsi/scsi_transport_fc.c
>>>> +++ b/drivers/scsi/scsi_transport_fc.c
>>>> @@ -44,6 +44,18 @@ static int fc_vport_setup(struct Scsi_Host
>>>> *shost, int channel,
>>>> struct device *pdev, struct fc_vport_identifiers *ids,
>>>> struct fc_vport **vport);
>>>>
>>>> +/* Kobjects for /sys/class/fc_host/hostx/fcoe
>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>> + */
>>>> +struct kobject *private_fcoe_kobj;
>>>> +struct kobject *private_fcoe_stat_kobj;
>>>> +
>>>> +/* Kobject attributes defined for /sys/class/fc_host/hostx/fcoe
>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>> + */
>>>> +#define
>>>> FCOE_KOBJECT_ATTR(_prefix,_name,_mode,_show,_store) \
>>>> +struct kobj_attribute kobject_attr_##_prefix##_##_name
>>>> = \
>>>> + __ATTR(_name,_mode,_show,_store)
>>>> /*
>>>> * Redefine so that we can have same named attributes in the
>>>> * sdev/starget/host objects.
>>>> @@ -97,6 +109,7 @@ static struct {
>>>> { FC_PORTTYPE_LPORT, "LPort (private loop)" },
>>>> { FC_PORTTYPE_PTP, "Point-To-Point (direct nport
>>>> connection)" },
>>>> { FC_PORTTYPE_NPIV, "NPIV VPORT" },
>>>> + { FC_PORTTYPE_VFPORT, "VFPort (Virtual fabric via
>>>> point-to-
>>>> point)" },
>>>> };
>>>> fc_enum_name_search(port_type, fc_port_type, fc_port_type_names)
>>>> #define FC_PORTTYPE_MAX_NAMELEN 50
>>>> @@ -414,12 +427,6 @@ static int fc_host_setup(struct
>>>> transport_container *tc, struct device *dev,
>>>> return 0;
>>>> }
>>>>
>>>> -static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>> - "fc_host",
>>>> - fc_host_setup,
>>>> - NULL,
>>>> - NULL);
>>>> -
>>>> /*
>>>> * Setup and Remove actions for remote ports are handled
>>>> * in the service functions below.
>>>> @@ -617,34 +624,6 @@ send_vendor_fail:
>>>> }
>>>> EXPORT_SYMBOL(fc_host_post_vendor_event);
>>>>
>>>> -
>>>> -
>>>> -static __init int fc_transport_init(void)
>>>> -{
>>>> - int error;
>>>> -
>>>> - atomic_set(&fc_event_seq, 0);
>>>> -
>>>> - error = transport_class_register(&fc_host_class);
>>>> - if (error)
>>>> - return error;
>>>> - error = transport_class_register(&fc_vport_class);
>>>> - if (error)
>>>> - return error;
>>>> - error = transport_class_register(&fc_rport_class);
>>>> - if (error)
>>>> - return error;
>>>> - return transport_class_register(&fc_transport_class);
>>>> -}
>>>> -
>>>> -static void __exit fc_transport_exit(void)
>>>> -{
>>>> - transport_class_unregister(&fc_transport_class);
>>>> - transport_class_unregister(&fc_rport_class);
>>>> - transport_class_unregister(&fc_host_class);
>>>> - transport_class_unregister(&fc_vport_class);
>>>> -}
>>>> -
>>>> /*
>>>> * FC Remote Port Attribute Management
>>>> */
>>>> @@ -1655,6 +1634,224 @@ static struct attribute_group
>>>> fc_statistics_group = {
>>>> .attrs = fc_statistics_attrs,
>>>> };
>>>>
>>>> +/*
>>>> + * FCoE Host attributes
>>>> + */
>>>> +
>>>> +/* Convert mac_addressing_mode values to ascii string name */
>>>> +static struct {
>>>> + enum fcoe_mac_addressing_mode value;
>>>> + char *name;
>>>> +} mac_addressing_mode_names [] = {
>>>> + { FCF_SELECTED, "FCF selcted - Either FPMA
>>>> or SPMA" },
>>>> + { SPMA_ONLY, "SPMA only - ENode
>>>> Solicites SPMA only support" },
>>>> + { FPMA_ONLY, "FPMA only - ENode
>>>> Solicites SPMA only support" },
>>>> + { SPMA_PREFERRED, "SPMA preferred - ENode
>>>> Solicites SPMA
>>>> preferred support" },
>>>> + { FPMA_PREFERRED, "FPMA preferred - ENode
>>>> Solicites FPMA
>>>> preferred support" },
>>>> +};
>>>> +fc_enum_name_search(mac_addressing_mode, fcoe_mac_addressing_mode,
>>>> mac_addressing_mode_names)
>>>> +#define FCOE_MAC_ADDR_MODE_MAX_NAMELEN 80
>>>> +
>>>> +/* Show MAC address members of the FCoE host attributes */
>>>> +static ssize_t
>>>> +fcoe_host_attrs_show(const struct kobject *kobj, char *buf,
>>>> unsigned long offset)
>>>> +{
>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>> + struct fcoe_host_attrs *stats;
>>>> + ssize_t ret = -ENOENT;
>>>> +
>>>> + if (i->f->get_fcoe_host) {
>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>> + if (stats)
>>>> + ret = snprintf(buf, 20, "0x%x\n",
>>>> + *(u16 *)(((u8 *) stats) + offset));
>>>> + }
>>>> + return ret;
>>>> +}
>>>> +
>>>> +/* generate a read-only FCoE Host attribute */
>>>> +#define
>>>> fcoe_host_attrs
>>>> (name) \
>>>> +static ssize_t show_fcoe_host_attrs_##name(struct kobject
>>>> *cd, \
>>>> + struct kobj_attribute
>>>> *attr, \
>>>> + char
>>>> *buf) \
>>>> +
>>>> { \
>>>> + return fcoe_host_attrs_show(cd,
>>>> buf, \
>>>> + offsetof(struct fcoe_host_attrs,
>>>> name)); \
>>>> +
>>>> } \
>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>> show_fcoe_host_attrs_##name, NULL)
>>>> +
>>>> +fcoe_host_attrs(vlan_id);
>>>> +
>>>> +#define fcoe_host_attrs_enum_attr(title,
>>>> maxlen) \
>>>> +static ssize_t show_fcoe_host_attrs_##title(struct kobject
>>>> *kobj, \
>>>> + struct kobj_attribute
>>>> *attr, \
>>>> + char
>>>> *buf) \
>>>> +
>>>> { \
>>>> + struct device *dev = kobj_to_dev(kobj-
>>>> >parent); \
>>>> + struct Scsi_Host *shost =
>>>> transport_class_to_shost(dev); \
>>>> + struct fc_internal *i = to_fc_internal(shost-
>>>> >transportt); \
>>>> + struct fcoe_host_attrs
>>>> *stats; \
>>>> + ssize_t ret = -
>>>> EINVAL; \
>>>> + const char
>>>> *name; \
>>>> +
>>>> \
>>>> + if (i->f->get_fcoe_host)
>>>> { \
>>>> + stats = (i->f->get_fcoe_host)
>>>> (shost); \
>>>> + if(stats)
>>>> { \
>>>> + name = get_fc_##title##_name(stats-
>>>> >mac_addressing_mode); \
>>>> + if
>>>> (name) \
>>>> + ret= snprintf(buf, maxlen, "%s\n",
>>>> name); \
>>>> +
>>>> } \
>>>> +
>>>> } \
>>>> + return
>>>> ret
>>>> ; \
>>>> +
>>>> } \
>>>> +static FCOE_KOBJECT_ATTR(fcoe, title, S_IRUGO,
>>>> show_fcoe_host_attrs_##title, NULL)
>>>> +
>>>> +fcoe_host_attrs_enum_attr(mac_addressing_mode,
>>>> FCOE_MAC_ADDR_MODE_MAX_NAMELEN);
>>>> +
>>>> +static ssize_t
>>>> +fcoe_mac_addr_show(const struct kobject *kobj, char *buf, unsigned
>>>> long offset)
>>>> +{
>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>> + struct fcoe_host_attrs *stats;
>>>> + ssize_t ret = -ENOENT;
>>>> +
>>>> + if (offset > sizeof(struct fcoe_host_attrs))
>>>> + WARN_ON(1);
>>>> +
>>>> + if (i->f->get_fcoe_host) {
>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>> + if (stats) {
>>>> + u8 *macaddr = (u8*)(((u8 *) stats) + offset);
>>>> + ret = sysfs_format_mac(buf, macaddr, 6);
>>>> + }
>>>> + }
>>>> + return ret;
>>>> +}
>>>> +
>>>> +/* generate a read-only FCoE MAC address host attribute */
>>>> +#define
>>>> fcoe_mac_addr(name) \
>>>> +static ssize_t show_fcoe_mac_addr_##name(struct kobject
>>>> *cd, \
>>>> + struct kobj_attribute
>>>> *attr, \
>>>> + char
>>>> *buf) \
>>>> +
>>>> { \
>>>> + return fcoe_mac_addr_show(cd,
>>>> buf, \
>>>> + offsetof(struct fcoe_host_attrs,
>>>> name)); \
>>>> +
>>>> } \
>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>> show_fcoe_mac_addr_##name, NULL)
>>>> +
>>>> +fcoe_mac_addr(enode_mac_address);
>>>> +fcoe_mac_addr(vn_port_mac_address);
>>>> +fcoe_mac_addr(vf_port_mac_address);
>>>> +
>>>> +static struct attribute *fcoe_host_attrs[] = {
>>>> + &kobject_attr_fcoe_enode_mac_address.attr,
>>>> + &kobject_attr_fcoe_vn_port_mac_address.attr,
>>>> + &kobject_attr_fcoe_vf_port_mac_address.attr,
>>>> + &kobject_attr_fcoe_vlan_id.attr,
>>>> + &kobject_attr_fcoe_mac_addressing_mode.attr,
>>>> + NULL
>>>> +};
>>>> +
>>>> +static struct attribute_group fcoe_host_attrs_group = {
>>>> + .attrs = fcoe_host_attrs,
>>>> +};
>>>> +
>>>> +/* Read FCoE host statistics */
>>>> +static ssize_t
>>>> +fcoe_host_statistics_show(const struct kobject *kobj, char *buf,
>>>> unsigned long offset)
>>>> +{
>>>> + struct device *dev = kobj_to_dev(kobj->parent->parent);
>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>> + struct fcoe_host_statistics *stats;
>>>> + ssize_t ret = -ENOENT;
>>>> +
>>>> + if (offset > sizeof(struct fcoe_host_statistics) ||
>>>> + offset % sizeof(u64) != 0)
>>>> + WARN_ON(1);
>>>> +
>>>> + if (i->f->get_fcoe_host_stats) {
>>>> + stats = (i->f->get_fcoe_host_stats)(shost);
>>>> + if (stats)
>>>> + ret = snprintf(buf, 20, "0x%llx\n",
>>>> + (unsigned long long)*(u64 *)(((u8 *)
>>>> stats) + offset));
>>>> + }
>>>> + return ret;
>>>> +}
>>>> +
>>>> +/* generate a read-only FCoE host staistics attribute */
>>>> +#define
>>>> fcoe_host_statistics
>>>> (name) \
>>>> +static ssize_t show_fcoe_host_statistics_##name(struct kobject
>>>> *cd, \
>>>> + struct kobj_attribute
>>>> *attr, \
>>>> + char
>>>> *buf) \
>>>> +
>>>> { \
>>>> + return fcoe_host_statistics_show(cd,
>>>> buf, \
>>>> + offsetof(struct fcoe_host_statistics,
>>>> name)); \
>>>> +
>>>> } \
>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>> show_fcoe_host_statistics_##name, NULL)
>>>> +
>>>> +fcoe_host_statistics(link_failure_count);
>>>> +fcoe_host_statistics(loss_of_sync_count);
>>>> +fcoe_host_statistics(loss_of_signal_count);
>>>> +fcoe_host_statistics(prim_seq_protocol_err_count);
>>>> +fcoe_host_statistics(invalid_tx_word_count);
>>>> +fcoe_host_statistics(invalid_crc_count);
>>>> +
>>>> +static struct attribute *fcoe_host_statistics[] = {
>>>> + &kobject_attr_fcoe_link_failure_count.attr,
>>>> + &kobject_attr_fcoe_loss_of_sync_count.attr,
>>>> + &kobject_attr_fcoe_loss_of_signal_count.attr,
>>>> + &kobject_attr_fcoe_prim_seq_protocol_err_count.attr,
>>>> + &kobject_attr_fcoe_invalid_tx_word_count.attr,
>>>> + &kobject_attr_fcoe_invalid_crc_count.attr,
>>>> + NULL
>>>> +};
>>>> +
>>>> +static struct attribute_group fcoe_host_statistics_group = {
>>>> + .attrs = fcoe_host_statistics,
>>>> +};
>>>> +
>>>> +static int fc_host_configure(struct transport_container *tc,
>>>> struct
>>>> device *dev,
>>>> + struct device *cdev)
>>>> +{
>>>> + int error;
>>>> +
>>>> + /* Create a fcoe sub-directory under /sys/class/fc_host/
>>>> hostX/ */
>>>> + private_fcoe_kobj = kobject_create_and_add("fcoe", &cdev-
>>>> >kobj);
>>>> + if (private_fcoe_kobj) {
>>>> + /* Create the files associated with this kobject */
>>>> + error = sysfs_create_group(private_fcoe_kobj,
>>>> &fcoe_host_attrs_group);
>>>> + if (error) {
>>>> + kobject_put(private_fcoe_kobj);
>>>> + goto error;
>>>> + }
>>>> + }
>>>> +
>>>> + /* Create a statistics sub-directory under /sys/class/
>>>> fc_host/
>>>> hostX/fcoe/ */
>>>> + private_fcoe_stat_kobj = kobject_create_and_add("statistics",
>>>> private_fcoe_kobj);
>>>> + if (private_fcoe_stat_kobj) {
>>>> + /* Create the files associated with this kobject */
>>>> + error = sysfs_create_group(private_fcoe_stat_kobj,
>>>> &fcoe_host_statistics_group);
>>>> + if (error) {
>>>> + kobject_put(private_fcoe_stat_kobj);
>>>> + goto error;
>>>> + }
>>>> + }
>>>> +
>>>> +error:
>>>> + return 0;
>>>> +}
>>>> +
>>>> +static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>> + "fc_host",
>>>> + fc_host_setup,
>>>> + NULL,
>>>> + fc_host_configure);
>>>> +
>>>>
>>>> /* Host Vport Attributes */
>>>>
>>>> @@ -3342,6 +3539,33 @@ fc_vport_sched_delete(struct work_struct
>>>> *work)
>>>> vport->channel, stat);
>>>> }
>>>>
>>>> +static __init int fc_transport_init(void)
>>>> +{
>>>> + int error;
>>>> +
>>>> + atomic_set(&fc_event_seq, 0);
>>>> +
>>>> + error = transport_class_register(&fc_host_class);
>>>> + if (error)
>>>> + return error;
>>>> + error = transport_class_register(&fc_vport_class);
>>>> + if (error)
>>>> + return error;
>>>> + error = transport_class_register(&fc_rport_class);
>>>> + if (error)
>>>> + return error;
>>>> + return transport_class_register(&fc_transport_class);
>>>> +}
>>>> +
>>>> +static void __exit fc_transport_exit(void)
>>>> +{
>>>> + transport_class_unregister(&fc_transport_class);
>>>> + transport_class_unregister(&fc_rport_class);
>>>> + transport_class_unregister(&fc_host_class);
>>>> + transport_class_unregister(&fc_vport_class);
>>>> + kobject_put(private_fcoe_kobj);
>>>> + kobject_put(private_fcoe_stat_kobj);
>>>> +}
>>>>
>>>> /* Original Author: Martin Hicks */
>>>> MODULE_AUTHOR("James Smart");
>>>> diff --git a/include/scsi/scsi_transport_fc.h b/include/scsi/
>>>> scsi_transport_fc.h
>>>> index c9184f7..def2e23 100644
>>>> --- a/include/scsi/scsi_transport_fc.h
>>>> +++ b/include/scsi/scsi_transport_fc.h
>>>> @@ -30,6 +30,8 @@
>>>> #include <linux/sched.h>
>>>> #include <scsi/scsi.h>
>>>> #include <scsi/scsi_netlink.h>
>>>> +#include <scsi/scsi_netlink.h>
>>>> +#include <linux/etherdevice.h>
>>>>
>>>> struct scsi_transport_template;
>>>>
>>>> @@ -63,6 +65,7 @@ enum fc_port_type {
>>>> FC_PORTTYPE_LPORT, /* (Private) Loop w/o FLPort */
>>>> FC_PORTTYPE_PTP, /* Point to Point w/ another
>>>> NPort */
>>>> FC_PORTTYPE_NPIV, /* VPORT based on NPIV */
>>>> + FC_PORTTYPE_VFPORT, /* Virtual Fabric PORT */
>>>> };
>>>>
>>>>
>>>> @@ -428,6 +431,53 @@ struct fc_host_statistics {
>>>> u64 fcp_output_megabytes;
>>>> };
>>>>
>>>> +/*
>>>> + * FCoE Local Port (Host) Attributes
>>>> + *
>>>> + * Fixed attributes are not expected to change. The driver is
>>>> + * expected to set these values after successfully calling
>>>> scsi_add_host().
>>>> + * The transport fully manages all get functions w/o driver
>>>> interaction.
>>>> + *
>>>> + * Dynamic attributes are expected to change. The driver
>>>> participates
>>>> + * in all get/set operations via functions provided by the driver.
>>>> + *
>>>> + */
>>>> +struct fcoe_host_attrs {
>>>> + /* Fixed attributes */
>>>> + uint8_t enode_mac_address[ETH_ALEN];
>>>> +
>>>> + /* Dynamic attributes */
>>>> + uint8_t vn_port_mac_address[ETH_ALEN];
>>>> + uint8_t vf_port_mac_address[ETH_ALEN];
>>>> + u16 vlan_id;
>>>> + u16 mac_addressing_mode;
>>>> +};
>>>> +
>>>> +/* FCoE Statistics - Following proposal for FC_BB_E FC Link error
>>>> status block representation
>>>> + * from T11/09-204v0 guidelines
>>>> + */
>>>> +struct fcoe_host_statistics {
>>>> + /* Port statistics */
>>>> + u64 link_failure_count;
>>>> + u64 loss_of_sync_count;
>>>> + u64 loss_of_signal_count;
>>>> + u64 prim_seq_protocol_err_count;
>>>> + u64 invalid_tx_word_count;
>>>> + u64 invalid_crc_count;
>>>> +};
>>>> +
>>>> +
>>>> +/*
>>>> + * mac_addressing_mode: If you alter this, you also need to alter
>>>> + * scsi_transport_fc.c (for the ascii descriptions).
>>>> + */
>>>> +enum fcoe_mac_addressing_mode {
>>>> + FCF_SELECTED,
>>>> + SPMA_ONLY,
>>>> + FPMA_ONLY,
>>>> + SPMA_PREFERRED,
>>>> + FPMA_PREFERRED,
>>>> +};
>>>>
>>>> /*
>>>> * FC Event Codes - Polled and Async, following FC HBAAPI v2.0
>>>> guidelines
>>>> @@ -600,6 +650,9 @@ struct fc_function_template {
>>>> struct fc_host_statistics * (*get_fc_host_stats)(struct
>>>> Scsi_Host *);
>>>> void (*reset_fc_host_stats)(struct Scsi_Host *);
>>>>
>>>> + struct fcoe_host_attrs* (*get_fcoe_host)(struct Scsi_Host *);
>>>> + struct fcoe_host_statistics* (*get_fcoe_host_stats)(struct
>>>> Scsi_Host *);
>>>> +
>>>> int (*issue_fc_host_lip)(struct Scsi_Host *);
>>>>
>>>> void (*dev_loss_tmo_callbk)(struct fc_rport *);
>>>> --
>>>> 1.6.2.rc1.30.gd43c
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>
>>>
>>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Patch - To display FCoE Host information and statistics in sysfs
2009-06-02 17:34 ` Giridhar Malavali
@ 2009-06-03 15:24 ` James Smart
2009-06-24 23:13 ` Giridhar Malavali
0 siblings, 1 reply; 6+ messages in thread
From: James Smart @ 2009-06-03 15:24 UTC (permalink / raw)
To: Giridhar Malavali; +Cc: James Bottomley, linux-scsi
Giridhar Malavali wrote:
>> James,
>>
>> By opening the FC as a bus, you meant a sysfs layout similar to
>> present scsi layout in 2.6 kernel (/sys/bus/scsi).
>>
>> Below are the layout details that I had in my mind
>> /sys/bus/fc/devices/ - list of all devices
>>
this one doesn't make sense, as what is a "device" ? If you are thinking
the adapters - no, as they should just fall out naturally from the
device tree. However, there probably will be a base class of
/sys/class/fc/fc_port which is the base fc class object bound to the
base adapter object (pci function).
>> /sys/bus/fc/drivers/fc and fcoe ( links to all fc and fcoe hosts and
>> targets )
>>
hadn't thought this far ahead.. probably not - we'll let the
/sys/devices..../<device>->driver object keep this relationship, with
the <class entity>->device getting you to the <device> object.
>> /sys/class/fc/fc_host/
>> /sys/class/fc/fc_remote_ports
>> /sys/class/fc/fcoe_host ( Initiator )
>> /sys/class/fc/fcoe_remote_ports ( Target )
>>
yes, something like this - although fcoe_remote_ports won't exist (they
are no different from a fc_remote_port). And I don't see a distinction
between an fc_host and and fcoe_host - we'd have one object (fc port), a
couple of base attributes which are generic for it including one that
says "type" (is either "fc" or "fcoe), and then attributes which are
either fc-specific, or fcoe-specific. Optionally, we could make these
type-specific attributes a subdirectory/subclass.
Looking at the device tree is probably a better representation:
- There would be a base object, under the pci function, for the fc port.
- Under the fc port, we'd likely have a fabric entry point object - a
fcfport.
Theoretically, there could be many of these (and do multiple FCF's to
the same fabric count as different fcfports, or are they "paths" under
a single fcfport ?). For now, assume there's only 1, which may be FC
(inherent in an Fx_port) or a FCOE FCF. There would be a generic set of
attributes, including a "type", and then type-specific attributes as well.
- Under the fcfport, we'd have fcfabric objects. These would have
attributes such as fabric name, switchname, vfid header values, fabric
e_d_tov/r_a_tov, etc values and so on. From this object on, everything
should be generic FC and fcoe shouldn't change anything.
Note: FCF's with different vlan tags would be considered different
fcfports each with a single fcfabric object underneath, while a single
FCF with VFID headers is a single fcfport, with multiple fcfabric
objects under it.
- Under the fcfabric object, we would then have a fcvport object. This
object corresponds to each N_Port_ID we have on the the object (thus
FLOGI's and FDISCs with the fabric).
- Under the fcvport object, we then have fc4 protocol roles for the
n_port_id, such as fcpinitiator (taking the place of the old fc_host) or
fctgt (for a target mode role) or ficon or..., as well as objects for
each of the fc_remote_ports
- Under the fcpinitiator role, we would have the scsi_host object
- Under the scsi_host object, we would have scsi_target objects. We
would need to figure out how to cross-link (in some manner) between the
scsi_target object and the fc_remote_port object.
- Under the scsi_target object, we would have the scsi device objects,
and so on...
Which leads to a device tree something like the following:
/sys/devices/.../pci.../fcportX/fcfportY/fcfabricZ/fcvportA/fcrportB
/sys/devices/.../pci.../fcportX/fcfportY/fcfabricZ/fcvportA/fcpinitC/hostD/target<H:C:T>/<H:C:T:L>/...
hmmm... if fcoe, there's probably some kind of object that either sits
at the same level as fcportX, or more likely, inbet6ween the pci
function object and the fcportX object, that reflects the shared NIC port.
(and I'm open to renaming the above classes to make more sense).
>> Please comment whether I understood it right?
>>
>> This makes things lot clear for me. This conveys better distinction
>> between FC and FCoE adapters present in the system. The FC and FCoE
>> specific details can be kept under respective directory and links
>> can be created to map their relations.
>>
>>
Yes. It is heavy, and at some point we need to figure out if
illustrating the topology in the device tree is actually worth it.
Note that I'm not seeing separate object based on fc or fcoe, but rather
a single offset, with a varying set of attributes. We could change this
and make the varying set a subdirectory if it's clearer.
>> As stated by you, the scope of these changes looks bit heavy. I
>> feel, it will be good to add these changes incrementally. Keeping
>> long term perspective, is it a good idea to add FCoE related
>> information as a separate directory parallel to fc_host in /sys/
>> class directory. I feel this leads to easier migration when FC is
>> opened up as a bus in future.
>>
Right now - I'm partial to a subdirectory under the fc_host that is
titled "fcoe", that has fcoe-specific attributes for the port.
-- james
>> Thanks,
>> Giridhar.M.B
>>
>>
>> On Apr 30, 2009, at 10:00 AM, James Smart wrote:
>>
>>
>>> Giridhar,
>>>
>>> Yes I saw it. I'm really a little unsure. At LSF, we talked about the
>>> need for FC to open up as a bus, thus change the way the topology
>>> objects are created, and FCOE affects that as the attributes are
>>> pre-role (role being initiator or tgt or whatever) but where you are
>>> placing them is in a role-based location. So, if we say yes, as
>>> the set
>>> is rather small, then it starts us off on a divergence from this
>>> longer-term view. Yes, we can survive, but I hate telling mgmt code
>>> to
>>> deal with both variants in the future.
>>>
>>> -- james
>>>
>>>
>>>
>>> Giridhar Malavali wrote:
>>>
>>>> Hi James,
>>>>
>>>> I had submitted the following patch last week. I did not hear any
>>>> thing from you or from the community. Just a friendly check to find
>>>> out whether this patch looks fine?
>>>>
>>>> Thanks,
>>>> Giridhar.M.B
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> Begin forwarded message:
>>>>
>>>>
>>>>
>>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>> Date: April 17, 2009 6:01:30 PM PDT
>>>>> To: James Smart <James.Smart@Emulex.Com>
>>>>> Cc: linux-scsi@vger.kernel.org
>>>>> Subject: Patch - To display FCoE Host information and statistics in
>>>>> sysfs
>>>>>
>>>>> Hi All,
>>>>>
>>>>> Attached is the patch for displaying FCoE Host bus adapter
>>>>> information in sysfs.
>>>>>
>>>>> This patch adds fcoe directory (/sys/class/fc_host/hostX/fcoe)
>>>>> and a
>>>>> statistics sub-directory (/sys/class/fc_host/hostX/fcoe/statistics)
>>>>> to display FCoE host specific and it statistics respectively.
>>>>>
>>>>> A new port type definition called VF_PORT (Virtual Fabric Port) is
>>>>> added to identify the virtual link between ENode and the FCF
>>>>> switch.
>>>>> The contents in the fcoe and its sub-directory will be valid, if
>>>>> the
>>>>> FC port type is a VF_PORT.
>>>>>
>>>>> Following are the members added to display host specific
>>>>> information
>>>>> under /sys/class/fc_host/hostX/fcoe directory
>>>>>
>>>>> 1) enode_mac_address - The burnt in MAC address used by the
>>>>> FCoE
>>>>> adapter
>>>>> 2) vn_port_mac_address - The current MAC address used by the
>>>>> FCoE
>>>>> adapter. This can be same as ENode MAC address or different based
>>>>> on
>>>>> the MAC addressing mode.
>>>>> 3) vf_port_mac_address - MAC address of the FCF.
>>>>> 4) mac_addressing_mode - The type of FCoE MAC addressing mode
>>>>> FCF_SELECTED - The FCF selects either SPMA or FPMA
>>>>> addressing mode
>>>>> SPMA_ONLY - Only SPMA addressing mode is supported by
>>>>> ENode
>>>>> FPMA_ONLY - Only FPMA addressing mode is supported by
>>>>> ENode
>>>>> SPMA_PREFERRED - SPMA addressing mode is preferred by
>>>>> ENode
>>>>> FPMA_PREFERRED - FPMA addressing mode is preferred by
>>>>> ENode
>>>>> 5) vlan_id - The Vlan ID used
>>>>>
>>>>> Following counters are added to display host statistics information
>>>>> under /sys/class/fc_host/hostX/fcoe/statistics directory
>>>>>
>>>>> The statistics information displayed is similar to FC host
>>>>> statistics. The FCoE statistics carries the same naming as FC host
>>>>> statistics but the information displayed are mapped to related MAC
>>>>> statistics.
>>>>>
>>>>> This information is based on the proposal made to T11 for
>>>>> mapping
>>>>> FC LESB ( Link Error Status Block) to FC_BB_E. Please refer to FCoE
>>>>> Link Error Status Block (09-204v1.pdf) document at www.t11.org/index.html
>>>>> .
>>>>>
>>>>> The various counters are
>>>>>
>>>>> 1) Invalid_crc_count
>>>>> 2) Invalid_tx_word_count
>>>>> 3) link_failure_count
>>>>> 4) loss_of_signal_count
>>>>> 5) loss_of_sync_count
>>>>> 6) prim_seq_protocol_error.
>>>>>
>>>>> Thanks,
>>>>> Giridhar.M.B
>>>>>
>>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>> Date: Fri, 17 Apr 2009 15:51:42 -0700
>>>>> Subject: [PATCH 1/2] Addition of FCoE information to sysfs.
>>>>>
>>>>> The information contains both FCoE host specific as well as its
>>>>> statistics.
>>>>> The FCoE specific information are displayed under /sys/class/
>>>>> fc_host/
>>>>> hostX/fcoe
>>>>> and /sys/class/fc_host/hostX/fcoe/statistics directories
>>>>> respectively.
>>>>>
>>>>> Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>> ---
>>>>> drivers/scsi/scsi_transport_fc.c | 292 ++++++++++++++++++++++++++
>>>>> +++
>>>>> ++++-----
>>>>> include/scsi/scsi_transport_fc.h | 53 +++++++
>>>>> 2 files changed, 311 insertions(+), 34 deletions(-)
>>>>>
>>>>> diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/
>>>>> scsi_transport_fc.c
>>>>> index a152f89..55e3757 100644
>>>>> --- a/drivers/scsi/scsi_transport_fc.c
>>>>> +++ b/drivers/scsi/scsi_transport_fc.c
>>>>> @@ -44,6 +44,18 @@ static int fc_vport_setup(struct Scsi_Host
>>>>> *shost, int channel,
>>>>> struct device *pdev, struct fc_vport_identifiers *ids,
>>>>> struct fc_vport **vport);
>>>>>
>>>>> +/* Kobjects for /sys/class/fc_host/hostx/fcoe
>>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>>> + */
>>>>> +struct kobject *private_fcoe_kobj;
>>>>> +struct kobject *private_fcoe_stat_kobj;
>>>>> +
>>>>> +/* Kobject attributes defined for /sys/class/fc_host/hostx/fcoe
>>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>>> + */
>>>>> +#define
>>>>> FCOE_KOBJECT_ATTR(_prefix,_name,_mode,_show,_store) \
>>>>> +struct kobj_attribute kobject_attr_##_prefix##_##_name
>>>>> = \
>>>>> + __ATTR(_name,_mode,_show,_store)
>>>>> /*
>>>>> * Redefine so that we can have same named attributes in the
>>>>> * sdev/starget/host objects.
>>>>> @@ -97,6 +109,7 @@ static struct {
>>>>> { FC_PORTTYPE_LPORT, "LPort (private loop)" },
>>>>> { FC_PORTTYPE_PTP, "Point-To-Point (direct nport
>>>>> connection)" },
>>>>> { FC_PORTTYPE_NPIV, "NPIV VPORT" },
>>>>> + { FC_PORTTYPE_VFPORT, "VFPort (Virtual fabric via
>>>>> point-to-
>>>>> point)" },
>>>>> };
>>>>> fc_enum_name_search(port_type, fc_port_type, fc_port_type_names)
>>>>> #define FC_PORTTYPE_MAX_NAMELEN 50
>>>>> @@ -414,12 +427,6 @@ static int fc_host_setup(struct
>>>>> transport_container *tc, struct device *dev,
>>>>> return 0;
>>>>> }
>>>>>
>>>>> -static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>>> - "fc_host",
>>>>> - fc_host_setup,
>>>>> - NULL,
>>>>> - NULL);
>>>>> -
>>>>> /*
>>>>> * Setup and Remove actions for remote ports are handled
>>>>> * in the service functions below.
>>>>> @@ -617,34 +624,6 @@ send_vendor_fail:
>>>>> }
>>>>> EXPORT_SYMBOL(fc_host_post_vendor_event);
>>>>>
>>>>> -
>>>>> -
>>>>> -static __init int fc_transport_init(void)
>>>>> -{
>>>>> - int error;
>>>>> -
>>>>> - atomic_set(&fc_event_seq, 0);
>>>>> -
>>>>> - error = transport_class_register(&fc_host_class);
>>>>> - if (error)
>>>>> - return error;
>>>>> - error = transport_class_register(&fc_vport_class);
>>>>> - if (error)
>>>>> - return error;
>>>>> - error = transport_class_register(&fc_rport_class);
>>>>> - if (error)
>>>>> - return error;
>>>>> - return transport_class_register(&fc_transport_class);
>>>>> -}
>>>>> -
>>>>> -static void __exit fc_transport_exit(void)
>>>>> -{
>>>>> - transport_class_unregister(&fc_transport_class);
>>>>> - transport_class_unregister(&fc_rport_class);
>>>>> - transport_class_unregister(&fc_host_class);
>>>>> - transport_class_unregister(&fc_vport_class);
>>>>> -}
>>>>> -
>>>>> /*
>>>>> * FC Remote Port Attribute Management
>>>>> */
>>>>> @@ -1655,6 +1634,224 @@ static struct attribute_group
>>>>> fc_statistics_group = {
>>>>> .attrs = fc_statistics_attrs,
>>>>> };
>>>>>
>>>>> +/*
>>>>> + * FCoE Host attributes
>>>>> + */
>>>>> +
>>>>> +/* Convert mac_addressing_mode values to ascii string name */
>>>>> +static struct {
>>>>> + enum fcoe_mac_addressing_mode value;
>>>>> + char *name;
>>>>> +} mac_addressing_mode_names [] = {
>>>>> + { FCF_SELECTED, "FCF selcted - Either FPMA
>>>>> or SPMA" },
>>>>> + { SPMA_ONLY, "SPMA only - ENode
>>>>> Solicites SPMA only support" },
>>>>> + { FPMA_ONLY, "FPMA only - ENode
>>>>> Solicites SPMA only support" },
>>>>> + { SPMA_PREFERRED, "SPMA preferred - ENode
>>>>> Solicites SPMA
>>>>> preferred support" },
>>>>> + { FPMA_PREFERRED, "FPMA preferred - ENode
>>>>> Solicites FPMA
>>>>> preferred support" },
>>>>> +};
>>>>> +fc_enum_name_search(mac_addressing_mode, fcoe_mac_addressing_mode,
>>>>> mac_addressing_mode_names)
>>>>> +#define FCOE_MAC_ADDR_MODE_MAX_NAMELEN 80
>>>>> +
>>>>> +/* Show MAC address members of the FCoE host attributes */
>>>>> +static ssize_t
>>>>> +fcoe_host_attrs_show(const struct kobject *kobj, char *buf,
>>>>> unsigned long offset)
>>>>> +{
>>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>> + struct fcoe_host_attrs *stats;
>>>>> + ssize_t ret = -ENOENT;
>>>>> +
>>>>> + if (i->f->get_fcoe_host) {
>>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>>> + if (stats)
>>>>> + ret = snprintf(buf, 20, "0x%x\n",
>>>>> + *(u16 *)(((u8 *) stats) + offset));
>>>>> + }
>>>>> + return ret;
>>>>> +}
>>>>> +
>>>>> +/* generate a read-only FCoE Host attribute */
>>>>> +#define
>>>>> fcoe_host_attrs
>>>>> (name) \
>>>>> +static ssize_t show_fcoe_host_attrs_##name(struct kobject
>>>>> *cd, \
>>>>> + struct kobj_attribute
>>>>> *attr, \
>>>>> + char
>>>>> *buf) \
>>>>> +
>>>>> { \
>>>>> + return fcoe_host_attrs_show(cd,
>>>>> buf, \
>>>>> + offsetof(struct fcoe_host_attrs,
>>>>> name)); \
>>>>> +
>>>>> } \
>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>> show_fcoe_host_attrs_##name, NULL)
>>>>> +
>>>>> +fcoe_host_attrs(vlan_id);
>>>>> +
>>>>> +#define fcoe_host_attrs_enum_attr(title,
>>>>> maxlen) \
>>>>> +static ssize_t show_fcoe_host_attrs_##title(struct kobject
>>>>> *kobj, \
>>>>> + struct kobj_attribute
>>>>> *attr, \
>>>>> + char
>>>>> *buf) \
>>>>> +
>>>>> { \
>>>>> + struct device *dev = kobj_to_dev(kobj-
>>>>>
>>>>>> parent); \
>>>>>>
>>>>> + struct Scsi_Host *shost =
>>>>> transport_class_to_shost(dev); \
>>>>> + struct fc_internal *i = to_fc_internal(shost-
>>>>>
>>>>>> transportt); \
>>>>>>
>>>>> + struct fcoe_host_attrs
>>>>> *stats; \
>>>>> + ssize_t ret = -
>>>>> EINVAL; \
>>>>> + const char
>>>>> *name; \
>>>>> +
>>>>> \
>>>>> + if (i->f->get_fcoe_host)
>>>>> { \
>>>>> + stats = (i->f->get_fcoe_host)
>>>>> (shost); \
>>>>> + if(stats)
>>>>> { \
>>>>> + name = get_fc_##title##_name(stats-
>>>>>
>>>>>> mac_addressing_mode); \
>>>>>>
>>>>> + if
>>>>> (name) \
>>>>> + ret= snprintf(buf, maxlen, "%s\n",
>>>>> name); \
>>>>> +
>>>>> } \
>>>>> +
>>>>> } \
>>>>> + return
>>>>> ret
>>>>> ; \
>>>>> +
>>>>> } \
>>>>> +static FCOE_KOBJECT_ATTR(fcoe, title, S_IRUGO,
>>>>> show_fcoe_host_attrs_##title, NULL)
>>>>> +
>>>>> +fcoe_host_attrs_enum_attr(mac_addressing_mode,
>>>>> FCOE_MAC_ADDR_MODE_MAX_NAMELEN);
>>>>> +
>>>>> +static ssize_t
>>>>> +fcoe_mac_addr_show(const struct kobject *kobj, char *buf, unsigned
>>>>> long offset)
>>>>> +{
>>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>> + struct fcoe_host_attrs *stats;
>>>>> + ssize_t ret = -ENOENT;
>>>>> +
>>>>> + if (offset > sizeof(struct fcoe_host_attrs))
>>>>> + WARN_ON(1);
>>>>> +
>>>>> + if (i->f->get_fcoe_host) {
>>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>>> + if (stats) {
>>>>> + u8 *macaddr = (u8*)(((u8 *) stats) + offset);
>>>>> + ret = sysfs_format_mac(buf, macaddr, 6);
>>>>> + }
>>>>> + }
>>>>> + return ret;
>>>>> +}
>>>>> +
>>>>> +/* generate a read-only FCoE MAC address host attribute */
>>>>> +#define
>>>>> fcoe_mac_addr(name) \
>>>>> +static ssize_t show_fcoe_mac_addr_##name(struct kobject
>>>>> *cd, \
>>>>> + struct kobj_attribute
>>>>> *attr, \
>>>>> + char
>>>>> *buf) \
>>>>> +
>>>>> { \
>>>>> + return fcoe_mac_addr_show(cd,
>>>>> buf, \
>>>>> + offsetof(struct fcoe_host_attrs,
>>>>> name)); \
>>>>> +
>>>>> } \
>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>> show_fcoe_mac_addr_##name, NULL)
>>>>> +
>>>>> +fcoe_mac_addr(enode_mac_address);
>>>>> +fcoe_mac_addr(vn_port_mac_address);
>>>>> +fcoe_mac_addr(vf_port_mac_address);
>>>>> +
>>>>> +static struct attribute *fcoe_host_attrs[] = {
>>>>> + &kobject_attr_fcoe_enode_mac_address.attr,
>>>>> + &kobject_attr_fcoe_vn_port_mac_address.attr,
>>>>> + &kobject_attr_fcoe_vf_port_mac_address.attr,
>>>>> + &kobject_attr_fcoe_vlan_id.attr,
>>>>> + &kobject_attr_fcoe_mac_addressing_mode.attr,
>>>>> + NULL
>>>>> +};
>>>>> +
>>>>> +static struct attribute_group fcoe_host_attrs_group = {
>>>>> + .attrs = fcoe_host_attrs,
>>>>> +};
>>>>> +
>>>>> +/* Read FCoE host statistics */
>>>>> +static ssize_t
>>>>> +fcoe_host_statistics_show(const struct kobject *kobj, char *buf,
>>>>> unsigned long offset)
>>>>> +{
>>>>> + struct device *dev = kobj_to_dev(kobj->parent->parent);
>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>> + struct fcoe_host_statistics *stats;
>>>>> + ssize_t ret = -ENOENT;
>>>>> +
>>>>> + if (offset > sizeof(struct fcoe_host_statistics) ||
>>>>> + offset % sizeof(u64) != 0)
>>>>> + WARN_ON(1);
>>>>> +
>>>>> + if (i->f->get_fcoe_host_stats) {
>>>>> + stats = (i->f->get_fcoe_host_stats)(shost);
>>>>> + if (stats)
>>>>> + ret = snprintf(buf, 20, "0x%llx\n",
>>>>> + (unsigned long long)*(u64 *)(((u8 *)
>>>>> stats) + offset));
>>>>> + }
>>>>> + return ret;
>>>>> +}
>>>>> +
>>>>> +/* generate a read-only FCoE host staistics attribute */
>>>>> +#define
>>>>> fcoe_host_statistics
>>>>> (name) \
>>>>> +static ssize_t show_fcoe_host_statistics_##name(struct kobject
>>>>> *cd, \
>>>>> + struct kobj_attribute
>>>>> *attr, \
>>>>> + char
>>>>> *buf) \
>>>>> +
>>>>> { \
>>>>> + return fcoe_host_statistics_show(cd,
>>>>> buf, \
>>>>> + offsetof(struct fcoe_host_statistics,
>>>>> name)); \
>>>>> +
>>>>> } \
>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>> show_fcoe_host_statistics_##name, NULL)
>>>>> +
>>>>> +fcoe_host_statistics(link_failure_count);
>>>>> +fcoe_host_statistics(loss_of_sync_count);
>>>>> +fcoe_host_statistics(loss_of_signal_count);
>>>>> +fcoe_host_statistics(prim_seq_protocol_err_count);
>>>>> +fcoe_host_statistics(invalid_tx_word_count);
>>>>> +fcoe_host_statistics(invalid_crc_count);
>>>>> +
>>>>> +static struct attribute *fcoe_host_statistics[] = {
>>>>> + &kobject_attr_fcoe_link_failure_count.attr,
>>>>> + &kobject_attr_fcoe_loss_of_sync_count.attr,
>>>>> + &kobject_attr_fcoe_loss_of_signal_count.attr,
>>>>> + &kobject_attr_fcoe_prim_seq_protocol_err_count.attr,
>>>>> + &kobject_attr_fcoe_invalid_tx_word_count.attr,
>>>>> + &kobject_attr_fcoe_invalid_crc_count.attr,
>>>>> + NULL
>>>>> +};
>>>>> +
>>>>> +static struct attribute_group fcoe_host_statistics_group = {
>>>>> + .attrs = fcoe_host_statistics,
>>>>> +};
>>>>> +
>>>>> +static int fc_host_configure(struct transport_container *tc,
>>>>> struct
>>>>> device *dev,
>>>>> + struct device *cdev)
>>>>> +{
>>>>> + int error;
>>>>> +
>>>>> + /* Create a fcoe sub-directory under /sys/class/fc_host/
>>>>> hostX/ */
>>>>> + private_fcoe_kobj = kobject_create_and_add("fcoe", &cdev-
>>>>>
>>>>>> kobj);
>>>>>>
>>>>> + if (private_fcoe_kobj) {
>>>>> + /* Create the files associated with this kobject */
>>>>> + error = sysfs_create_group(private_fcoe_kobj,
>>>>> &fcoe_host_attrs_group);
>>>>> + if (error) {
>>>>> + kobject_put(private_fcoe_kobj);
>>>>> + goto error;
>>>>> + }
>>>>> + }
>>>>> +
>>>>> + /* Create a statistics sub-directory under /sys/class/
>>>>> fc_host/
>>>>> hostX/fcoe/ */
>>>>> + private_fcoe_stat_kobj = kobject_create_and_add("statistics",
>>>>> private_fcoe_kobj);
>>>>> + if (private_fcoe_stat_kobj) {
>>>>> + /* Create the files associated with this kobject */
>>>>> + error = sysfs_create_group(private_fcoe_stat_kobj,
>>>>> &fcoe_host_statistics_group);
>>>>> + if (error) {
>>>>> + kobject_put(private_fcoe_stat_kobj);
>>>>> + goto error;
>>>>> + }
>>>>> + }
>>>>> +
>>>>> +error:
>>>>> + return 0;
>>>>> +}
>>>>> +
>>>>> +static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>>> + "fc_host",
>>>>> + fc_host_setup,
>>>>> + NULL,
>>>>> + fc_host_configure);
>>>>> +
>>>>>
>>>>> /* Host Vport Attributes */
>>>>>
>>>>> @@ -3342,6 +3539,33 @@ fc_vport_sched_delete(struct work_struct
>>>>> *work)
>>>>> vport->channel, stat);
>>>>> }
>>>>>
>>>>> +static __init int fc_transport_init(void)
>>>>> +{
>>>>> + int error;
>>>>> +
>>>>> + atomic_set(&fc_event_seq, 0);
>>>>> +
>>>>> + error = transport_class_register(&fc_host_class);
>>>>> + if (error)
>>>>> + return error;
>>>>> + error = transport_class_register(&fc_vport_class);
>>>>> + if (error)
>>>>> + return error;
>>>>> + error = transport_class_register(&fc_rport_class);
>>>>> + if (error)
>>>>> + return error;
>>>>> + return transport_class_register(&fc_transport_class);
>>>>> +}
>>>>> +
>>>>> +static void __exit fc_transport_exit(void)
>>>>> +{
>>>>> + transport_class_unregister(&fc_transport_class);
>>>>> + transport_class_unregister(&fc_rport_class);
>>>>> + transport_class_unregister(&fc_host_class);
>>>>> + transport_class_unregister(&fc_vport_class);
>>>>> + kobject_put(private_fcoe_kobj);
>>>>> + kobject_put(private_fcoe_stat_kobj);
>>>>> +}
>>>>>
>>>>> /* Original Author: Martin Hicks */
>>>>> MODULE_AUTHOR("James Smart");
>>>>> diff --git a/include/scsi/scsi_transport_fc.h b/include/scsi/
>>>>> scsi_transport_fc.h
>>>>> index c9184f7..def2e23 100644
>>>>> --- a/include/scsi/scsi_transport_fc.h
>>>>> +++ b/include/scsi/scsi_transport_fc.h
>>>>> @@ -30,6 +30,8 @@
>>>>> #include <linux/sched.h>
>>>>> #include <scsi/scsi.h>
>>>>> #include <scsi/scsi_netlink.h>
>>>>> +#include <scsi/scsi_netlink.h>
>>>>> +#include <linux/etherdevice.h>
>>>>>
>>>>> struct scsi_transport_template;
>>>>>
>>>>> @@ -63,6 +65,7 @@ enum fc_port_type {
>>>>> FC_PORTTYPE_LPORT, /* (Private) Loop w/o FLPort */
>>>>> FC_PORTTYPE_PTP, /* Point to Point w/ another
>>>>> NPort */
>>>>> FC_PORTTYPE_NPIV, /* VPORT based on NPIV */
>>>>> + FC_PORTTYPE_VFPORT, /* Virtual Fabric PORT */
>>>>> };
>>>>>
>>>>>
>>>>> @@ -428,6 +431,53 @@ struct fc_host_statistics {
>>>>> u64 fcp_output_megabytes;
>>>>> };
>>>>>
>>>>> +/*
>>>>> + * FCoE Local Port (Host) Attributes
>>>>> + *
>>>>> + * Fixed attributes are not expected to change. The driver is
>>>>> + * expected to set these values after successfully calling
>>>>> scsi_add_host().
>>>>> + * The transport fully manages all get functions w/o driver
>>>>> interaction.
>>>>> + *
>>>>> + * Dynamic attributes are expected to change. The driver
>>>>> participates
>>>>> + * in all get/set operations via functions provided by the driver.
>>>>> + *
>>>>> + */
>>>>> +struct fcoe_host_attrs {
>>>>> + /* Fixed attributes */
>>>>> + uint8_t enode_mac_address[ETH_ALEN];
>>>>> +
>>>>> + /* Dynamic attributes */
>>>>> + uint8_t vn_port_mac_address[ETH_ALEN];
>>>>> + uint8_t vf_port_mac_address[ETH_ALEN];
>>>>> + u16 vlan_id;
>>>>> + u16 mac_addressing_mode;
>>>>> +};
>>>>> +
>>>>> +/* FCoE Statistics - Following proposal for FC_BB_E FC Link error
>>>>> status block representation
>>>>> + * from T11/09-204v0 guidelines
>>>>> + */
>>>>> +struct fcoe_host_statistics {
>>>>> + /* Port statistics */
>>>>> + u64 link_failure_count;
>>>>> + u64 loss_of_sync_count;
>>>>> + u64 loss_of_signal_count;
>>>>> + u64 prim_seq_protocol_err_count;
>>>>> + u64 invalid_tx_word_count;
>>>>> + u64 invalid_crc_count;
>>>>> +};
>>>>> +
>>>>> +
>>>>> +/*
>>>>> + * mac_addressing_mode: If you alter this, you also need to alter
>>>>> + * scsi_transport_fc.c (for the ascii descriptions).
>>>>> + */
>>>>> +enum fcoe_mac_addressing_mode {
>>>>> + FCF_SELECTED,
>>>>> + SPMA_ONLY,
>>>>> + FPMA_ONLY,
>>>>> + SPMA_PREFERRED,
>>>>> + FPMA_PREFERRED,
>>>>> +};
>>>>>
>>>>> /*
>>>>> * FC Event Codes - Polled and Async, following FC HBAAPI v2.0
>>>>> guidelines
>>>>> @@ -600,6 +650,9 @@ struct fc_function_template {
>>>>> struct fc_host_statistics * (*get_fc_host_stats)(struct
>>>>> Scsi_Host *);
>>>>> void (*reset_fc_host_stats)(struct Scsi_Host *);
>>>>>
>>>>> + struct fcoe_host_attrs* (*get_fcoe_host)(struct Scsi_Host *);
>>>>> + struct fcoe_host_statistics* (*get_fcoe_host_stats)(struct
>>>>> Scsi_Host *);
>>>>> +
>>>>> int (*issue_fc_host_lip)(struct Scsi_Host *);
>>>>>
>>>>> void (*dev_loss_tmo_callbk)(struct fc_rport *);
>>>>> --
>>>>> 1.6.2.rc1.30.gd43c
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>
>>>>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Patch - To display FCoE Host information and statistics in sysfs
2009-06-03 15:24 ` James Smart
@ 2009-06-24 23:13 ` Giridhar Malavali
2009-07-06 21:14 ` Giridhar Malavali
0 siblings, 1 reply; 6+ messages in thread
From: Giridhar Malavali @ 2009-06-24 23:13 UTC (permalink / raw)
To: James Smart; +Cc: James Bottomley, LinuxSCSI, Andrew Vasquez
On Jun 3, 2009, at 8:24 AM, James Smart wrote:
>
> Giridhar Malavali wrote:
>>> James,
>>>
>>> By opening the FC as a bus, you meant a sysfs layout similar to
>>> present scsi layout in 2.6 kernel (/sys/bus/scsi).
>>>
>>> Below are the layout details that I had in my mind
>>> /sys/bus/fc/devices/ - list of all devices
>>>
> this one doesn't make sense, as what is a "device" ? If you are
> thinking
> the adapters - no, as they should just fall out naturally from the
> device tree. However, there probably will be a base class of
> /sys/class/fc/fc_port which is the base fc class object bound to the
> base adapter object (pci function).
>
>>> /sys/bus/fc/drivers/fc and fcoe ( links to all fc and fcoe hosts and
>>> targets )
>>>
> hadn't thought this far ahead.. probably not - we'll let the
> /sys/devices..../<device>->driver object keep this relationship, with
> the <class entity>->device getting you to the <device> object.
>
>>> /sys/class/fc/fc_host/
>>> /sys/class/fc/fc_remote_ports
>>> /sys/class/fc/fcoe_host ( Initiator )
>>> /sys/class/fc/fcoe_remote_ports ( Target )
>>>
> yes, something like this - although fcoe_remote_ports won't exist
> (they
> are no different from a fc_remote_port). And I don't see a distinction
> between an fc_host and and fcoe_host - we'd have one object (fc
> port), a
> couple of base attributes which are generic for it including one that
> says "type" (is either "fc" or "fcoe), and then attributes which are
> either fc-specific, or fcoe-specific. Optionally, we could make these
> type-specific attributes a subdirectory/subclass.
>
> Looking at the device tree is probably a better representation:
>
> - There would be a base object, under the pci function, for the fc
> port.
>
> - Under the fc port, we'd likely have a fabric entry point object - a
> fcfport.
> Theoretically, there could be many of these (and do multiple FCF's to
> the same fabric count as different fcfports, or are they "paths"
> under
> a single fcfport ?). For now, assume there's only 1, which may be FC
> (inherent in an Fx_port) or a FCOE FCF. There would be a generic set
> of
> attributes, including a "type", and then type-specific attributes as
> well.
>
> - Under the fcfport, we'd have fcfabric objects. These would have
> attributes such as fabric name, switchname, vfid header values, fabric
> e_d_tov/r_a_tov, etc values and so on. From this object on,
> everything
> should be generic FC and fcoe shouldn't change anything.
>
> Note: FCF's with different vlan tags would be considered different
> fcfports each with a single fcfabric object underneath, while a single
> FCF with VFID headers is a single fcfport, with multiple fcfabric
> objects under it.
>
> - Under the fcfabric object, we would then have a fcvport object.
> This
> object corresponds to each N_Port_ID we have on the the object (thus
> FLOGI's and FDISCs with the fabric).
>
> - Under the fcvport object, we then have fc4 protocol roles for the
> n_port_id, such as fcpinitiator (taking the place of the old
> fc_host) or
> fctgt (for a target mode role) or ficon or..., as well as objects for
> each of the fc_remote_ports
>
> - Under the fcpinitiator role, we would have the scsi_host object
>
> - Under the scsi_host object, we would have scsi_target objects. We
> would need to figure out how to cross-link (in some manner) between
> the
> scsi_target object and the fc_remote_port object.
>
> - Under the scsi_target object, we would have the scsi device objects,
> and so on...
>
>
> Which leads to a device tree something like the following:
>
> /sys/devices/.../pci.../fcportX/fcfportY/fcfabricZ/fcvportA/fcrportB
> /sys/devices/.../pci.../fcportX/fcfportY/fcfabricZ/fcvportA/fcpinitC/
> hostD/target<H:C:T>/<H:C:T:L>/...
>
> hmmm... if fcoe, there's probably some kind of object that either sits
> at the same level as fcportX, or more likely, inbet6ween the pci
> function object and the fcportX object, that reflects the shared NIC
> port.
>
> (and I'm open to renaming the above classes to make more sense).
Thanks for the detailed explanation. This definitely gives better
picture than existing one.
Any plans when to support this change?
>
>
>>> Please comment whether I understood it right?
>>>
>>> This makes things lot clear for me. This conveys better
>>> distinction
>>> between FC and FCoE adapters present in the system. The FC and FCoE
>>> specific details can be kept under respective directory and links
>>> can be created to map their relations.
>>>
>>>
> Yes. It is heavy, and at some point we need to figure out if
> illustrating the topology in the device tree is actually worth it.
>
> Note that I'm not seeing separate object based on fc or fcoe, but
> rather
> a single offset, with a varying set of attributes. We could change
> this
> and make the varying set a subdirectory if it's clearer.
>
>>> As stated by you, the scope of these changes looks bit heavy. I
>>> feel, it will be good to add these changes incrementally. Keeping
>>> long term perspective, is it a good idea to add FCoE related
>>> information as a separate directory parallel to fc_host in /sys/
>>> class directory. I feel this leads to easier migration when FC is
>>> opened up as a bus in future.
>>>
> Right now - I'm partial to a subdirectory under the fc_host that is
> titled "fcoe", that has fcoe-specific attributes for the port.
>
I think in such a case, both fc and fcoe should be a separate sub-
directories under fc_host. A new attribute called host_type should be
added to mention which sub-directory to refer. The scis hostX will be
present either in fcoe or fc directory based on host_type attribute.
/sys/class/fc_host/fcoeX/ - This will have scsi_host and FCoE port
specific information
/sys/class/fc_host/fcX/ - This will have scsi_host and FC port
specific information.
/sys/class/fc_host/hostX_type - basically tells which sub-directories
to refer for a particular host.
Let me know what do u think about it.
-- Giridhar.M.B
> -- james
>
>
>>> Thanks,
>>> Giridhar.M.B
>>>
>>>
>>> On Apr 30, 2009, at 10:00 AM, James Smart wrote:
>>>
>>>
>>>> Giridhar,
>>>>
>>>> Yes I saw it. I'm really a little unsure. At LSF, we talked about
>>>> the
>>>> need for FC to open up as a bus, thus change the way the topology
>>>> objects are created, and FCOE affects that as the attributes are
>>>> pre-role (role being initiator or tgt or whatever) but where you
>>>> are
>>>> placing them is in a role-based location. So, if we say yes, as
>>>> the set
>>>> is rather small, then it starts us off on a divergence from this
>>>> longer-term view. Yes, we can survive, but I hate telling mgmt code
>>>> to
>>>> deal with both variants in the future.
>>>>
>>>> -- james
>>>>
>>>>
>>>>
>>>> Giridhar Malavali wrote:
>>>>
>>>>> Hi James,
>>>>>
>>>>> I had submitted the following patch last week. I did not hear any
>>>>> thing from you or from the community. Just a friendly check to
>>>>> find
>>>>> out whether this patch looks fine?
>>>>>
>>>>> Thanks,
>>>>> Giridhar.M.B
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> Begin forwarded message:
>>>>>
>>>>>
>>>>>
>>>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>>> Date: April 17, 2009 6:01:30 PM PDT
>>>>>> To: James Smart <James.Smart@Emulex.Com>
>>>>>> Cc: linux-scsi@vger.kernel.org
>>>>>> Subject: Patch - To display FCoE Host information and
>>>>>> statistics in
>>>>>> sysfs
>>>>>>
>>>>>> Hi All,
>>>>>>
>>>>>> Attached is the patch for displaying FCoE Host bus adapter
>>>>>> information in sysfs.
>>>>>>
>>>>>> This patch adds fcoe directory (/sys/class/fc_host/hostX/fcoe)
>>>>>> and a
>>>>>> statistics sub-directory (/sys/class/fc_host/hostX/fcoe/
>>>>>> statistics)
>>>>>> to display FCoE host specific and it statistics respectively.
>>>>>>
>>>>>> A new port type definition called VF_PORT (Virtual Fabric Port)
>>>>>> is
>>>>>> added to identify the virtual link between ENode and the FCF
>>>>>> switch.
>>>>>> The contents in the fcoe and its sub-directory will be valid, if
>>>>>> the
>>>>>> FC port type is a VF_PORT.
>>>>>>
>>>>>> Following are the members added to display host specific
>>>>>> information
>>>>>> under /sys/class/fc_host/hostX/fcoe directory
>>>>>>
>>>>>> 1) enode_mac_address - The burnt in MAC address used by the
>>>>>> FCoE
>>>>>> adapter
>>>>>> 2) vn_port_mac_address - The current MAC address used by the
>>>>>> FCoE
>>>>>> adapter. This can be same as ENode MAC address or different based
>>>>>> on
>>>>>> the MAC addressing mode.
>>>>>> 3) vf_port_mac_address - MAC address of the FCF.
>>>>>> 4) mac_addressing_mode - The type of FCoE MAC addressing mode
>>>>>> FCF_SELECTED - The FCF selects either SPMA or FPMA
>>>>>> addressing mode
>>>>>> SPMA_ONLY - Only SPMA addressing mode is supported by
>>>>>> ENode
>>>>>> FPMA_ONLY - Only FPMA addressing mode is supported by
>>>>>> ENode
>>>>>> SPMA_PREFERRED - SPMA addressing mode is preferred by
>>>>>> ENode
>>>>>> FPMA_PREFERRED - FPMA addressing mode is preferred by
>>>>>> ENode
>>>>>> 5) vlan_id - The Vlan ID used
>>>>>>
>>>>>> Following counters are added to display host statistics
>>>>>> information
>>>>>> under /sys/class/fc_host/hostX/fcoe/statistics directory
>>>>>>
>>>>>> The statistics information displayed is similar to FC host
>>>>>> statistics. The FCoE statistics carries the same naming as FC
>>>>>> host
>>>>>> statistics but the information displayed are mapped to related
>>>>>> MAC
>>>>>> statistics.
>>>>>>
>>>>>> This information is based on the proposal made to T11 for
>>>>>> mapping
>>>>>> FC LESB ( Link Error Status Block) to FC_BB_E. Please refer to
>>>>>> FCoE
>>>>>> Link Error Status Block (09-204v1.pdf) document at www.t11.org/index.html
>>>>>> .
>>>>>>
>>>>>> The various counters are
>>>>>>
>>>>>> 1) Invalid_crc_count
>>>>>> 2) Invalid_tx_word_count
>>>>>> 3) link_failure_count
>>>>>> 4) loss_of_signal_count
>>>>>> 5) loss_of_sync_count
>>>>>> 6) prim_seq_protocol_error.
>>>>>>
>>>>>> Thanks,
>>>>>> Giridhar.M.B
>>>>>>
>>>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>>> Date: Fri, 17 Apr 2009 15:51:42 -0700
>>>>>> Subject: [PATCH 1/2] Addition of FCoE information to sysfs.
>>>>>>
>>>>>> The information contains both FCoE host specific as well as its
>>>>>> statistics.
>>>>>> The FCoE specific information are displayed under /sys/class/
>>>>>> fc_host/
>>>>>> hostX/fcoe
>>>>>> and /sys/class/fc_host/hostX/fcoe/statistics directories
>>>>>> respectively.
>>>>>>
>>>>>> Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>>> ---
>>>>>> drivers/scsi/scsi_transport_fc.c | 292 ++++++++++++++++++++++++
>>>>>> ++
>>>>>> +++
>>>>>> ++++-----
>>>>>> include/scsi/scsi_transport_fc.h | 53 +++++++
>>>>>> 2 files changed, 311 insertions(+), 34 deletions(-)
>>>>>>
>>>>>> diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/
>>>>>> scsi_transport_fc.c
>>>>>> index a152f89..55e3757 100644
>>>>>> --- a/drivers/scsi/scsi_transport_fc.c
>>>>>> +++ b/drivers/scsi/scsi_transport_fc.c
>>>>>> @@ -44,6 +44,18 @@ static int fc_vport_setup(struct Scsi_Host
>>>>>> *shost, int channel,
>>>>>> struct device *pdev, struct fc_vport_identifiers *ids,
>>>>>> struct fc_vport **vport);
>>>>>>
>>>>>> +/* Kobjects for /sys/class/fc_host/hostx/fcoe
>>>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>>>> + */
>>>>>> +struct kobject *private_fcoe_kobj;
>>>>>> +struct kobject *private_fcoe_stat_kobj;
>>>>>> +
>>>>>> +/* Kobject attributes defined for /sys/class/fc_host/hostx/fcoe
>>>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>>>> + */
>>>>>> +#define
>>>>>> FCOE_KOBJECT_ATTR(_prefix,_name,_mode,_show,_store) \
>>>>>> +struct kobj_attribute kobject_attr_##_prefix##_##_name
>>>>>> = \
>>>>>> + __ATTR(_name,_mode,_show,_store)
>>>>>> /*
>>>>>> * Redefine so that we can have same named attributes in the
>>>>>> * sdev/starget/host objects.
>>>>>> @@ -97,6 +109,7 @@ static struct {
>>>>>> { FC_PORTTYPE_LPORT, "LPort (private loop)" },
>>>>>> { FC_PORTTYPE_PTP, "Point-To-Point (direct nport
>>>>>> connection)" },
>>>>>> { FC_PORTTYPE_NPIV, "NPIV VPORT" },
>>>>>> + { FC_PORTTYPE_VFPORT, "VFPort (Virtual fabric via
>>>>>> point-to-
>>>>>> point)" },
>>>>>> };
>>>>>> fc_enum_name_search(port_type, fc_port_type, fc_port_type_names)
>>>>>> #define FC_PORTTYPE_MAX_NAMELEN 50
>>>>>> @@ -414,12 +427,6 @@ static int fc_host_setup(struct
>>>>>> transport_container *tc, struct device *dev,
>>>>>> return 0;
>>>>>> }
>>>>>>
>>>>>> -static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>>>> - "fc_host",
>>>>>> - fc_host_setup,
>>>>>> - NULL,
>>>>>> - NULL);
>>>>>> -
>>>>>> /*
>>>>>> * Setup and Remove actions for remote ports are handled
>>>>>> * in the service functions below.
>>>>>> @@ -617,34 +624,6 @@ send_vendor_fail:
>>>>>> }
>>>>>> EXPORT_SYMBOL(fc_host_post_vendor_event);
>>>>>>
>>>>>> -
>>>>>> -
>>>>>> -static __init int fc_transport_init(void)
>>>>>> -{
>>>>>> - int error;
>>>>>> -
>>>>>> - atomic_set(&fc_event_seq, 0);
>>>>>> -
>>>>>> - error = transport_class_register(&fc_host_class);
>>>>>> - if (error)
>>>>>> - return error;
>>>>>> - error = transport_class_register(&fc_vport_class);
>>>>>> - if (error)
>>>>>> - return error;
>>>>>> - error = transport_class_register(&fc_rport_class);
>>>>>> - if (error)
>>>>>> - return error;
>>>>>> - return transport_class_register(&fc_transport_class);
>>>>>> -}
>>>>>> -
>>>>>> -static void __exit fc_transport_exit(void)
>>>>>> -{
>>>>>> - transport_class_unregister(&fc_transport_class);
>>>>>> - transport_class_unregister(&fc_rport_class);
>>>>>> - transport_class_unregister(&fc_host_class);
>>>>>> - transport_class_unregister(&fc_vport_class);
>>>>>> -}
>>>>>> -
>>>>>> /*
>>>>>> * FC Remote Port Attribute Management
>>>>>> */
>>>>>> @@ -1655,6 +1634,224 @@ static struct attribute_group
>>>>>> fc_statistics_group = {
>>>>>> .attrs = fc_statistics_attrs,
>>>>>> };
>>>>>>
>>>>>> +/*
>>>>>> + * FCoE Host attributes
>>>>>> + */
>>>>>> +
>>>>>> +/* Convert mac_addressing_mode values to ascii string name */
>>>>>> +static struct {
>>>>>> + enum fcoe_mac_addressing_mode value;
>>>>>> + char *name;
>>>>>> +} mac_addressing_mode_names [] = {
>>>>>> + { FCF_SELECTED, "FCF selcted - Either FPMA
>>>>>> or SPMA" },
>>>>>> + { SPMA_ONLY, "SPMA only - ENode
>>>>>> Solicites SPMA only support" },
>>>>>> + { FPMA_ONLY, "FPMA only - ENode
>>>>>> Solicites SPMA only support" },
>>>>>> + { SPMA_PREFERRED, "SPMA preferred - ENode
>>>>>> Solicites SPMA
>>>>>> preferred support" },
>>>>>> + { FPMA_PREFERRED, "FPMA preferred - ENode
>>>>>> Solicites FPMA
>>>>>> preferred support" },
>>>>>> +};
>>>>>> +fc_enum_name_search(mac_addressing_mode,
>>>>>> fcoe_mac_addressing_mode,
>>>>>> mac_addressing_mode_names)
>>>>>> +#define FCOE_MAC_ADDR_MODE_MAX_NAMELEN 80
>>>>>> +
>>>>>> +/* Show MAC address members of the FCoE host attributes */
>>>>>> +static ssize_t
>>>>>> +fcoe_host_attrs_show(const struct kobject *kobj, char *buf,
>>>>>> unsigned long offset)
>>>>>> +{
>>>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>>> + struct fcoe_host_attrs *stats;
>>>>>> + ssize_t ret = -ENOENT;
>>>>>> +
>>>>>> + if (i->f->get_fcoe_host) {
>>>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>>>> + if (stats)
>>>>>> + ret = snprintf(buf, 20, "0x%x\n",
>>>>>> + *(u16 *)(((u8 *) stats) + offset));
>>>>>> + }
>>>>>> + return ret;
>>>>>> +}
>>>>>> +
>>>>>> +/* generate a read-only FCoE Host attribute */
>>>>>> +#define
>>>>>> fcoe_host_attrs
>>>>>> (name) \
>>>>>> +static ssize_t show_fcoe_host_attrs_##name(struct kobject
>>>>>> *cd, \
>>>>>> + struct kobj_attribute
>>>>>> *attr, \
>>>>>> + char
>>>>>> *buf) \
>>>>>> +
>>>>>> { \
>>>>>> + return fcoe_host_attrs_show(cd,
>>>>>> buf, \
>>>>>> + offsetof(struct fcoe_host_attrs,
>>>>>> name)); \
>>>>>> +
>>>>>> } \
>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>>> show_fcoe_host_attrs_##name, NULL)
>>>>>> +
>>>>>> +fcoe_host_attrs(vlan_id);
>>>>>> +
>>>>>> +#define fcoe_host_attrs_enum_attr(title,
>>>>>> maxlen) \
>>>>>> +static ssize_t show_fcoe_host_attrs_##title(struct kobject
>>>>>> *kobj, \
>>>>>> + struct kobj_attribute
>>>>>> *attr, \
>>>>>> + char
>>>>>> *buf) \
>>>>>> +
>>>>>> { \
>>>>>> + struct device *dev = kobj_to_dev(kobj-
>>>>>>
>>>>>>> parent); \
>>>>>>>
>>>>>> + struct Scsi_Host *shost =
>>>>>> transport_class_to_shost(dev); \
>>>>>> + struct fc_internal *i = to_fc_internal(shost-
>>>>>>
>>>>>>> transportt); \
>>>>>>>
>>>>>> + struct fcoe_host_attrs
>>>>>> *stats; \
>>>>>> + ssize_t ret = -
>>>>>> EINVAL; \
>>>>>> + const char
>>>>>> *name; \
>>>>>> +
>>>>>> \
>>>>>> + if (i->f->get_fcoe_host)
>>>>>> { \
>>>>>> + stats = (i->f->get_fcoe_host)
>>>>>> (shost); \
>>>>>> + if(stats)
>>>>>> { \
>>>>>> + name = get_fc_##title##_name(stats-
>>>>>>
>>>>>>> mac_addressing_mode); \
>>>>>>>
>>>>>> + if
>>>>>> (name) \
>>>>>> + ret= snprintf(buf, maxlen, "%s\n",
>>>>>> name); \
>>>>>> +
>>>>>> } \
>>>>>> +
>>>>>> } \
>>>>>> + return
>>>>>> ret
>>>>>> ; \
>>>>>> +
>>>>>> } \
>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, title, S_IRUGO,
>>>>>> show_fcoe_host_attrs_##title, NULL)
>>>>>> +
>>>>>> +fcoe_host_attrs_enum_attr(mac_addressing_mode,
>>>>>> FCOE_MAC_ADDR_MODE_MAX_NAMELEN);
>>>>>> +
>>>>>> +static ssize_t
>>>>>> +fcoe_mac_addr_show(const struct kobject *kobj, char *buf,
>>>>>> unsigned
>>>>>> long offset)
>>>>>> +{
>>>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>>> + struct fcoe_host_attrs *stats;
>>>>>> + ssize_t ret = -ENOENT;
>>>>>> +
>>>>>> + if (offset > sizeof(struct fcoe_host_attrs))
>>>>>> + WARN_ON(1);
>>>>>> +
>>>>>> + if (i->f->get_fcoe_host) {
>>>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>>>> + if (stats) {
>>>>>> + u8 *macaddr = (u8*)(((u8 *) stats) +
>>>>>> offset);
>>>>>> + ret = sysfs_format_mac(buf, macaddr, 6);
>>>>>> + }
>>>>>> + }
>>>>>> + return ret;
>>>>>> +}
>>>>>> +
>>>>>> +/* generate a read-only FCoE MAC address host attribute */
>>>>>> +#define
>>>>>> fcoe_mac_addr(name) \
>>>>>> +static ssize_t show_fcoe_mac_addr_##name(struct kobject
>>>>>> *cd, \
>>>>>> + struct kobj_attribute
>>>>>> *attr, \
>>>>>> + char
>>>>>> *buf) \
>>>>>> +
>>>>>> { \
>>>>>> + return fcoe_mac_addr_show(cd,
>>>>>> buf, \
>>>>>> + offsetof(struct fcoe_host_attrs,
>>>>>> name)); \
>>>>>> +
>>>>>> } \
>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>>> show_fcoe_mac_addr_##name, NULL)
>>>>>> +
>>>>>> +fcoe_mac_addr(enode_mac_address);
>>>>>> +fcoe_mac_addr(vn_port_mac_address);
>>>>>> +fcoe_mac_addr(vf_port_mac_address);
>>>>>> +
>>>>>> +static struct attribute *fcoe_host_attrs[] = {
>>>>>> + &kobject_attr_fcoe_enode_mac_address.attr,
>>>>>> + &kobject_attr_fcoe_vn_port_mac_address.attr,
>>>>>> + &kobject_attr_fcoe_vf_port_mac_address.attr,
>>>>>> + &kobject_attr_fcoe_vlan_id.attr,
>>>>>> + &kobject_attr_fcoe_mac_addressing_mode.attr,
>>>>>> + NULL
>>>>>> +};
>>>>>> +
>>>>>> +static struct attribute_group fcoe_host_attrs_group = {
>>>>>> + .attrs = fcoe_host_attrs,
>>>>>> +};
>>>>>> +
>>>>>> +/* Read FCoE host statistics */
>>>>>> +static ssize_t
>>>>>> +fcoe_host_statistics_show(const struct kobject *kobj, char *buf,
>>>>>> unsigned long offset)
>>>>>> +{
>>>>>> + struct device *dev = kobj_to_dev(kobj->parent->parent);
>>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>>> + struct fcoe_host_statistics *stats;
>>>>>> + ssize_t ret = -ENOENT;
>>>>>> +
>>>>>> + if (offset > sizeof(struct fcoe_host_statistics) ||
>>>>>> + offset % sizeof(u64) != 0)
>>>>>> + WARN_ON(1);
>>>>>> +
>>>>>> + if (i->f->get_fcoe_host_stats) {
>>>>>> + stats = (i->f->get_fcoe_host_stats)(shost);
>>>>>> + if (stats)
>>>>>> + ret = snprintf(buf, 20, "0x%llx\n",
>>>>>> + (unsigned long long)*(u64 *)(((u8 *)
>>>>>> stats) + offset));
>>>>>> + }
>>>>>> + return ret;
>>>>>> +}
>>>>>> +
>>>>>> +/* generate a read-only FCoE host staistics attribute */
>>>>>> +#define
>>>>>> fcoe_host_statistics
>>>>>> (name) \
>>>>>> +static ssize_t show_fcoe_host_statistics_##name(struct kobject
>>>>>> *cd, \
>>>>>> + struct kobj_attribute
>>>>>> *attr, \
>>>>>> + char
>>>>>> *buf) \
>>>>>> +
>>>>>> { \
>>>>>> + return fcoe_host_statistics_show(cd,
>>>>>> buf, \
>>>>>> + offsetof(struct fcoe_host_statistics,
>>>>>> name)); \
>>>>>> +
>>>>>> } \
>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>>> show_fcoe_host_statistics_##name, NULL)
>>>>>> +
>>>>>> +fcoe_host_statistics(link_failure_count);
>>>>>> +fcoe_host_statistics(loss_of_sync_count);
>>>>>> +fcoe_host_statistics(loss_of_signal_count);
>>>>>> +fcoe_host_statistics(prim_seq_protocol_err_count);
>>>>>> +fcoe_host_statistics(invalid_tx_word_count);
>>>>>> +fcoe_host_statistics(invalid_crc_count);
>>>>>> +
>>>>>> +static struct attribute *fcoe_host_statistics[] = {
>>>>>> + &kobject_attr_fcoe_link_failure_count.attr,
>>>>>> + &kobject_attr_fcoe_loss_of_sync_count.attr,
>>>>>> + &kobject_attr_fcoe_loss_of_signal_count.attr,
>>>>>> + &kobject_attr_fcoe_prim_seq_protocol_err_count.attr,
>>>>>> + &kobject_attr_fcoe_invalid_tx_word_count.attr,
>>>>>> + &kobject_attr_fcoe_invalid_crc_count.attr,
>>>>>> + NULL
>>>>>> +};
>>>>>> +
>>>>>> +static struct attribute_group fcoe_host_statistics_group = {
>>>>>> + .attrs = fcoe_host_statistics,
>>>>>> +};
>>>>>> +
>>>>>> +static int fc_host_configure(struct transport_container *tc,
>>>>>> struct
>>>>>> device *dev,
>>>>>> + struct device *cdev)
>>>>>> +{
>>>>>> + int error;
>>>>>> +
>>>>>> + /* Create a fcoe sub-directory under /sys/class/fc_host/
>>>>>> hostX/ */
>>>>>> + private_fcoe_kobj = kobject_create_and_add("fcoe", &cdev-
>>>>>>
>>>>>>> kobj);
>>>>>>>
>>>>>> + if (private_fcoe_kobj) {
>>>>>> + /* Create the files associated with this kobject */
>>>>>> + error = sysfs_create_group(private_fcoe_kobj,
>>>>>> &fcoe_host_attrs_group);
>>>>>> + if (error) {
>>>>>> + kobject_put(private_fcoe_kobj);
>>>>>> + goto error;
>>>>>> + }
>>>>>> + }
>>>>>> +
>>>>>> + /* Create a statistics sub-directory under /sys/class/
>>>>>> fc_host/
>>>>>> hostX/fcoe/ */
>>>>>> + private_fcoe_stat_kobj =
>>>>>> kobject_create_and_add("statistics",
>>>>>> private_fcoe_kobj);
>>>>>> + if (private_fcoe_stat_kobj) {
>>>>>> + /* Create the files associated with this kobject */
>>>>>> + error = sysfs_create_group(private_fcoe_stat_kobj,
>>>>>> &fcoe_host_statistics_group);
>>>>>> + if (error) {
>>>>>> + kobject_put(private_fcoe_stat_kobj);
>>>>>> + goto error;
>>>>>> + }
>>>>>> + }
>>>>>> +
>>>>>> +error:
>>>>>> + return 0;
>>>>>> +}
>>>>>> +
>>>>>> +static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>>>> + "fc_host",
>>>>>> + fc_host_setup,
>>>>>> + NULL,
>>>>>> + fc_host_configure);
>>>>>> +
>>>>>>
>>>>>> /* Host Vport Attributes */
>>>>>>
>>>>>> @@ -3342,6 +3539,33 @@ fc_vport_sched_delete(struct work_struct
>>>>>> *work)
>>>>>> vport->channel, stat);
>>>>>> }
>>>>>>
>>>>>> +static __init int fc_transport_init(void)
>>>>>> +{
>>>>>> + int error;
>>>>>> +
>>>>>> + atomic_set(&fc_event_seq, 0);
>>>>>> +
>>>>>> + error = transport_class_register(&fc_host_class);
>>>>>> + if (error)
>>>>>> + return error;
>>>>>> + error = transport_class_register(&fc_vport_class);
>>>>>> + if (error)
>>>>>> + return error;
>>>>>> + error = transport_class_register(&fc_rport_class);
>>>>>> + if (error)
>>>>>> + return error;
>>>>>> + return transport_class_register(&fc_transport_class);
>>>>>> +}
>>>>>> +
>>>>>> +static void __exit fc_transport_exit(void)
>>>>>> +{
>>>>>> + transport_class_unregister(&fc_transport_class);
>>>>>> + transport_class_unregister(&fc_rport_class);
>>>>>> + transport_class_unregister(&fc_host_class);
>>>>>> + transport_class_unregister(&fc_vport_class);
>>>>>> + kobject_put(private_fcoe_kobj);
>>>>>> + kobject_put(private_fcoe_stat_kobj);
>>>>>> +}
>>>>>>
>>>>>> /* Original Author: Martin Hicks */
>>>>>> MODULE_AUTHOR("James Smart");
>>>>>> diff --git a/include/scsi/scsi_transport_fc.h b/include/scsi/
>>>>>> scsi_transport_fc.h
>>>>>> index c9184f7..def2e23 100644
>>>>>> --- a/include/scsi/scsi_transport_fc.h
>>>>>> +++ b/include/scsi/scsi_transport_fc.h
>>>>>> @@ -30,6 +30,8 @@
>>>>>> #include <linux/sched.h>
>>>>>> #include <scsi/scsi.h>
>>>>>> #include <scsi/scsi_netlink.h>
>>>>>> +#include <scsi/scsi_netlink.h>
>>>>>> +#include <linux/etherdevice.h>
>>>>>>
>>>>>> struct scsi_transport_template;
>>>>>>
>>>>>> @@ -63,6 +65,7 @@ enum fc_port_type {
>>>>>> FC_PORTTYPE_LPORT, /* (Private) Loop w/o FLPort
>>>>>> */
>>>>>> FC_PORTTYPE_PTP, /* Point to Point w/ another
>>>>>> NPort */
>>>>>> FC_PORTTYPE_NPIV, /* VPORT based on NPIV */
>>>>>> + FC_PORTTYPE_VFPORT, /* Virtual Fabric PORT */
>>>>>> };
>>>>>>
>>>>>>
>>>>>> @@ -428,6 +431,53 @@ struct fc_host_statistics {
>>>>>> u64 fcp_output_megabytes;
>>>>>> };
>>>>>>
>>>>>> +/*
>>>>>> + * FCoE Local Port (Host) Attributes
>>>>>> + *
>>>>>> + * Fixed attributes are not expected to change. The driver is
>>>>>> + * expected to set these values after successfully calling
>>>>>> scsi_add_host().
>>>>>> + * The transport fully manages all get functions w/o driver
>>>>>> interaction.
>>>>>> + *
>>>>>> + * Dynamic attributes are expected to change. The driver
>>>>>> participates
>>>>>> + * in all get/set operations via functions provided by the
>>>>>> driver.
>>>>>> + *
>>>>>> + */
>>>>>> +struct fcoe_host_attrs {
>>>>>> + /* Fixed attributes */
>>>>>> + uint8_t enode_mac_address[ETH_ALEN];
>>>>>> +
>>>>>> + /* Dynamic attributes */
>>>>>> + uint8_t vn_port_mac_address[ETH_ALEN];
>>>>>> + uint8_t vf_port_mac_address[ETH_ALEN];
>>>>>> + u16 vlan_id;
>>>>>> + u16 mac_addressing_mode;
>>>>>> +};
>>>>>> +
>>>>>> +/* FCoE Statistics - Following proposal for FC_BB_E FC Link
>>>>>> error
>>>>>> status block representation
>>>>>> + * from T11/09-204v0 guidelines
>>>>>> + */
>>>>>> +struct fcoe_host_statistics {
>>>>>> + /* Port statistics */
>>>>>> + u64 link_failure_count;
>>>>>> + u64 loss_of_sync_count;
>>>>>> + u64 loss_of_signal_count;
>>>>>> + u64 prim_seq_protocol_err_count;
>>>>>> + u64 invalid_tx_word_count;
>>>>>> + u64 invalid_crc_count;
>>>>>> +};
>>>>>> +
>>>>>> +
>>>>>> +/*
>>>>>> + * mac_addressing_mode: If you alter this, you also need to
>>>>>> alter
>>>>>> + * scsi_transport_fc.c (for the ascii descriptions).
>>>>>> + */
>>>>>> +enum fcoe_mac_addressing_mode {
>>>>>> + FCF_SELECTED,
>>>>>> + SPMA_ONLY,
>>>>>> + FPMA_ONLY,
>>>>>> + SPMA_PREFERRED,
>>>>>> + FPMA_PREFERRED,
>>>>>> +};
>>>>>>
>>>>>> /*
>>>>>> * FC Event Codes - Polled and Async, following FC HBAAPI v2.0
>>>>>> guidelines
>>>>>> @@ -600,6 +650,9 @@ struct fc_function_template {
>>>>>> struct fc_host_statistics * (*get_fc_host_stats)(struct
>>>>>> Scsi_Host *);
>>>>>> void (*reset_fc_host_stats)(struct Scsi_Host *);
>>>>>>
>>>>>> + struct fcoe_host_attrs* (*get_fcoe_host)(struct Scsi_Host
>>>>>> *);
>>>>>> + struct fcoe_host_statistics* (*get_fcoe_host_stats)(struct
>>>>>> Scsi_Host *);
>>>>>> +
>>>>>> int (*issue_fc_host_lip)(struct Scsi_Host *);
>>>>>>
>>>>>> void (*dev_loss_tmo_callbk)(struct fc_rport *);
>>>>>> --
>>>>>> 1.6.2.rc1.30.gd43c
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>
>>
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Patch - To display FCoE Host information and statistics in sysfs
2009-06-24 23:13 ` Giridhar Malavali
@ 2009-07-06 21:14 ` Giridhar Malavali
0 siblings, 0 replies; 6+ messages in thread
From: Giridhar Malavali @ 2009-07-06 21:14 UTC (permalink / raw)
To: James Smart; +Cc: James Bottomley, LinuxSCSI, Andrew Vasquez, Giridhar Malavali
On Jun 24, 2009, at 4:13 PM, Giridhar Malavali wrote:
>
> On Jun 3, 2009, at 8:24 AM, James Smart wrote:
>
>>
>> Giridhar Malavali wrote:
>>>> James,
>>>>
>>>> By opening the FC as a bus, you meant a sysfs layout similar to
>>>> present scsi layout in 2.6 kernel (/sys/bus/scsi).
>>>>
>>>> Below are the layout details that I had in my mind
>>>> /sys/bus/fc/devices/ - list of all devices
>>>>
>> this one doesn't make sense, as what is a "device" ? If you are
>> thinking
>> the adapters - no, as they should just fall out naturally from the
>> device tree. However, there probably will be a base class of
>> /sys/class/fc/fc_port which is the base fc class object bound to the
>> base adapter object (pci function).
>>
>>>> /sys/bus/fc/drivers/fc and fcoe ( links to all fc and fcoe hosts
>>>> and
>>>> targets )
>>>>
>> hadn't thought this far ahead.. probably not - we'll let the
>> /sys/devices..../<device>->driver object keep this relationship, with
>> the <class entity>->device getting you to the <device> object.
>>
>>>> /sys/class/fc/fc_host/
>>>> /sys/class/fc/fc_remote_ports
>>>> /sys/class/fc/fcoe_host ( Initiator )
>>>> /sys/class/fc/fcoe_remote_ports ( Target )
>>>>
>> yes, something like this - although fcoe_remote_ports won't exist
>> (they
>> are no different from a fc_remote_port). And I don't see a
>> distinction
>> between an fc_host and and fcoe_host - we'd have one object (fc
>> port), a
>> couple of base attributes which are generic for it including one that
>> says "type" (is either "fc" or "fcoe), and then attributes which are
>> either fc-specific, or fcoe-specific. Optionally, we could make these
>> type-specific attributes a subdirectory/subclass.
>>
>> Looking at the device tree is probably a better representation:
>>
>> - There would be a base object, under the pci function, for the fc
>> port.
>>
>> - Under the fc port, we'd likely have a fabric entry point object - a
>> fcfport.
>> Theoretically, there could be many of these (and do multiple FCF's to
>> the same fabric count as different fcfports, or are they "paths"
>> under
>> a single fcfport ?). For now, assume there's only 1, which may be FC
>> (inherent in an Fx_port) or a FCOE FCF. There would be a generic
>> set of
>> attributes, including a "type", and then type-specific attributes
>> as well.
>>
>> - Under the fcfport, we'd have fcfabric objects. These would have
>> attributes such as fabric name, switchname, vfid header values,
>> fabric
>> e_d_tov/r_a_tov, etc values and so on. From this object on,
>> everything
>> should be generic FC and fcoe shouldn't change anything.
>>
>> Note: FCF's with different vlan tags would be considered different
>> fcfports each with a single fcfabric object underneath, while a
>> single
>> FCF with VFID headers is a single fcfport, with multiple fcfabric
>> objects under it.
>>
>> - Under the fcfabric object, we would then have a fcvport object.
>> This
>> object corresponds to each N_Port_ID we have on the the object (thus
>> FLOGI's and FDISCs with the fabric).
>>
>> - Under the fcvport object, we then have fc4 protocol roles for the
>> n_port_id, such as fcpinitiator (taking the place of the old
>> fc_host) or
>> fctgt (for a target mode role) or ficon or..., as well as objects for
>> each of the fc_remote_ports
>>
>> - Under the fcpinitiator role, we would have the scsi_host object
>>
>> - Under the scsi_host object, we would have scsi_target objects. We
>> would need to figure out how to cross-link (in some manner) between
>> the
>> scsi_target object and the fc_remote_port object.
>>
>> - Under the scsi_target object, we would have the scsi device
>> objects,
>> and so on...
>>
>>
>> Which leads to a device tree something like the following:
>>
>> /sys/devices/.../pci.../fcportX/fcfportY/fcfabricZ/fcvportA/fcrportB
>> /sys/devices/.../pci.../fcportX/fcfportY/fcfabricZ/fcvportA/
>> fcpinitC/hostD/target<H:C:T>/<H:C:T:L>/...
>>
>> hmmm... if fcoe, there's probably some kind of object that either
>> sits
>> at the same level as fcportX, or more likely, inbet6ween the pci
>> function object and the fcportX object, that reflects the shared
>> NIC port.
>>
>> (and I'm open to renaming the above classes to make more sense).
>
> Thanks for the detailed explanation. This definitely gives better
> picture than existing one.
> Any plans when to support this change?
>
>>
>>
>>>> Please comment whether I understood it right?
>>>>
>>>> This makes things lot clear for me. This conveys better
>>>> distinction
>>>> between FC and FCoE adapters present in the system. The FC and FCoE
>>>> specific details can be kept under respective directory and links
>>>> can be created to map their relations.
>>>>
>>>>
>> Yes. It is heavy, and at some point we need to figure out if
>> illustrating the topology in the device tree is actually worth it.
>>
>> Note that I'm not seeing separate object based on fc or fcoe, but
>> rather
>> a single offset, with a varying set of attributes. We could change
>> this
>> and make the varying set a subdirectory if it's clearer.
>>
>>>> As stated by you, the scope of these changes looks bit heavy. I
>>>> feel, it will be good to add these changes incrementally. Keeping
>>>> long term perspective, is it a good idea to add FCoE related
>>>> information as a separate directory parallel to fc_host in /sys/
>>>> class directory. I feel this leads to easier migration when FC is
>>>> opened up as a bus in future.
>>>>
>> Right now - I'm partial to a subdirectory under the fc_host that is
>> titled "fcoe", that has fcoe-specific attributes for the port.
>>
>
> I think in such a case, both fc and fcoe should be a separate sub-
> directories under fc_host. A new attribute called host_type should
> be added to mention which sub-directory to refer. The scis hostX
> will be present either in fcoe or fc directory based on host_type
> attribute.
>
> /sys/class/fc_host/fcoeX/ - This will have scsi_host and FCoE port
> specific information
> /sys/class/fc_host/fcX/ - This will have scsi_host and FC port
> specific information.
> /sys/class/fc_host/hostX_type - basically tells which sub-
> directories to refer for a particular host.
James,
These are other alternate options to the above:
1) To have separate hostX directory for fc and fcoe and keep rest of
the things same.
/sys/class/fc_hostX/ or /sys/class/fcoe_hostX depending upon the host
bus adapter type.
2) A subdirectory fcoe_hostX inside the fc_host directory in parallel
to scsi hostx directory to describe the fcoe host attributes
/sys/class/fc_host/hostX - SCSI attributes
/sys/class/fc_host/fcoe_hostX
Let me know your thoughts on this.
>
>
> Let me know what do u think about it.
>
> -- Giridhar.M.B
>
>
>
>
>
>
>> -- james
>>
>>
>>>> Thanks,
>>>> Giridhar.M.B
>>>>
>>>>
>>>> On Apr 30, 2009, at 10:00 AM, James Smart wrote:
>>>>
>>>>
>>>>> Giridhar,
>>>>>
>>>>> Yes I saw it. I'm really a little unsure. At LSF, we talked
>>>>> about the
>>>>> need for FC to open up as a bus, thus change the way the topology
>>>>> objects are created, and FCOE affects that as the attributes are
>>>>> pre-role (role being initiator or tgt or whatever) but where you
>>>>> are
>>>>> placing them is in a role-based location. So, if we say yes, as
>>>>> the set
>>>>> is rather small, then it starts us off on a divergence from this
>>>>> longer-term view. Yes, we can survive, but I hate telling mgmt
>>>>> code
>>>>> to
>>>>> deal with both variants in the future.
>>>>>
>>>>> -- james
>>>>>
>>>>>
>>>>>
>>>>> Giridhar Malavali wrote:
>>>>>
>>>>>> Hi James,
>>>>>>
>>>>>> I had submitted the following patch last week. I did not hear any
>>>>>> thing from you or from the community. Just a friendly check to
>>>>>> find
>>>>>> out whether this patch looks fine?
>>>>>>
>>>>>> Thanks,
>>>>>> Giridhar.M.B
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> Begin forwarded message:
>>>>>>
>>>>>>
>>>>>>
>>>>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>>>> Date: April 17, 2009 6:01:30 PM PDT
>>>>>>> To: James Smart <James.Smart@Emulex.Com>
>>>>>>> Cc: linux-scsi@vger.kernel.org
>>>>>>> Subject: Patch - To display FCoE Host information and
>>>>>>> statistics in
>>>>>>> sysfs
>>>>>>>
>>>>>>> Hi All,
>>>>>>>
>>>>>>> Attached is the patch for displaying FCoE Host bus adapter
>>>>>>> information in sysfs.
>>>>>>>
>>>>>>> This patch adds fcoe directory (/sys/class/fc_host/hostX/fcoe)
>>>>>>> and a
>>>>>>> statistics sub-directory (/sys/class/fc_host/hostX/fcoe/
>>>>>>> statistics)
>>>>>>> to display FCoE host specific and it statistics respectively.
>>>>>>>
>>>>>>> A new port type definition called VF_PORT (Virtual Fabric
>>>>>>> Port) is
>>>>>>> added to identify the virtual link between ENode and the FCF
>>>>>>> switch.
>>>>>>> The contents in the fcoe and its sub-directory will be valid, if
>>>>>>> the
>>>>>>> FC port type is a VF_PORT.
>>>>>>>
>>>>>>> Following are the members added to display host specific
>>>>>>> information
>>>>>>> under /sys/class/fc_host/hostX/fcoe directory
>>>>>>>
>>>>>>> 1) enode_mac_address - The burnt in MAC address used by the
>>>>>>> FCoE
>>>>>>> adapter
>>>>>>> 2) vn_port_mac_address - The current MAC address used by the
>>>>>>> FCoE
>>>>>>> adapter. This can be same as ENode MAC address or different
>>>>>>> based
>>>>>>> on
>>>>>>> the MAC addressing mode.
>>>>>>> 3) vf_port_mac_address - MAC address of the FCF.
>>>>>>> 4) mac_addressing_mode - The type of FCoE MAC addressing mode
>>>>>>> FCF_SELECTED - The FCF selects either SPMA or FPMA
>>>>>>> addressing mode
>>>>>>> SPMA_ONLY - Only SPMA addressing mode is supported by
>>>>>>> ENode
>>>>>>> FPMA_ONLY - Only FPMA addressing mode is supported by
>>>>>>> ENode
>>>>>>> SPMA_PREFERRED - SPMA addressing mode is preferred by
>>>>>>> ENode
>>>>>>> FPMA_PREFERRED - FPMA addressing mode is preferred by
>>>>>>> ENode
>>>>>>> 5) vlan_id - The Vlan ID used
>>>>>>>
>>>>>>> Following counters are added to display host statistics
>>>>>>> information
>>>>>>> under /sys/class/fc_host/hostX/fcoe/statistics directory
>>>>>>>
>>>>>>> The statistics information displayed is similar to FC host
>>>>>>> statistics. The FCoE statistics carries the same naming as FC
>>>>>>> host
>>>>>>> statistics but the information displayed are mapped to related
>>>>>>> MAC
>>>>>>> statistics.
>>>>>>>
>>>>>>> This information is based on the proposal made to T11 for
>>>>>>> mapping
>>>>>>> FC LESB ( Link Error Status Block) to FC_BB_E. Please refer to
>>>>>>> FCoE
>>>>>>> Link Error Status Block (09-204v1.pdf) document at www.t11.org/index.html
>>>>>>> .
>>>>>>>
>>>>>>> The various counters are
>>>>>>>
>>>>>>> 1) Invalid_crc_count
>>>>>>> 2) Invalid_tx_word_count
>>>>>>> 3) link_failure_count
>>>>>>> 4) loss_of_signal_count
>>>>>>> 5) loss_of_sync_count
>>>>>>> 6) prim_seq_protocol_error.
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Giridhar.M.B
>>>>>>>
>>>>>>> From: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>>>> Date: Fri, 17 Apr 2009 15:51:42 -0700
>>>>>>> Subject: [PATCH 1/2] Addition of FCoE information to sysfs.
>>>>>>>
>>>>>>> The information contains both FCoE host specific as well as its
>>>>>>> statistics.
>>>>>>> The FCoE specific information are displayed under /sys/class/
>>>>>>> fc_host/
>>>>>>> hostX/fcoe
>>>>>>> and /sys/class/fc_host/hostX/fcoe/statistics directories
>>>>>>> respectively.
>>>>>>>
>>>>>>> Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
>>>>>>> ---
>>>>>>> drivers/scsi/scsi_transport_fc.c | 292 +++++++++++++++++++++++
>>>>>>> +++
>>>>>>> +++
>>>>>>> ++++-----
>>>>>>> include/scsi/scsi_transport_fc.h | 53 +++++++
>>>>>>> 2 files changed, 311 insertions(+), 34 deletions(-)
>>>>>>>
>>>>>>> diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/
>>>>>>> scsi_transport_fc.c
>>>>>>> index a152f89..55e3757 100644
>>>>>>> --- a/drivers/scsi/scsi_transport_fc.c
>>>>>>> +++ b/drivers/scsi/scsi_transport_fc.c
>>>>>>> @@ -44,6 +44,18 @@ static int fc_vport_setup(struct Scsi_Host
>>>>>>> *shost, int channel,
>>>>>>> struct device *pdev, struct fc_vport_identifiers *ids,
>>>>>>> struct fc_vport **vport);
>>>>>>>
>>>>>>> +/* Kobjects for /sys/class/fc_host/hostx/fcoe
>>>>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>>>>> + */
>>>>>>> +struct kobject *private_fcoe_kobj;
>>>>>>> +struct kobject *private_fcoe_stat_kobj;
>>>>>>> +
>>>>>>> +/* Kobject attributes defined for /sys/class/fc_host/hostx/fcoe
>>>>>>> + * and /sys/class/fc_host/hostx/fcoe/statistics objects
>>>>>>> + */
>>>>>>> +#define
>>>>>>> FCOE_KOBJECT_ATTR(_prefix,_name,_mode,_show,_store) \
>>>>>>> +struct kobj_attribute kobject_attr_##_prefix##_##_name
>>>>>>> = \
>>>>>>> + __ATTR(_name,_mode,_show,_store)
>>>>>>> /*
>>>>>>> * Redefine so that we can have same named attributes in the
>>>>>>> * sdev/starget/host objects.
>>>>>>> @@ -97,6 +109,7 @@ static struct {
>>>>>>> { FC_PORTTYPE_LPORT, "LPort (private loop)" },
>>>>>>> { FC_PORTTYPE_PTP, "Point-To-Point (direct nport
>>>>>>> connection)" },
>>>>>>> { FC_PORTTYPE_NPIV, "NPIV VPORT" },
>>>>>>> + { FC_PORTTYPE_VFPORT, "VFPort (Virtual fabric
>>>>>>> via
>>>>>>> point-to-
>>>>>>> point)" },
>>>>>>> };
>>>>>>> fc_enum_name_search(port_type, fc_port_type, fc_port_type_names)
>>>>>>> #define FC_PORTTYPE_MAX_NAMELEN 50
>>>>>>> @@ -414,12 +427,6 @@ static int fc_host_setup(struct
>>>>>>> transport_container *tc, struct device *dev,
>>>>>>> return 0;
>>>>>>> }
>>>>>>>
>>>>>>> -static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>>>>> - "fc_host",
>>>>>>> - fc_host_setup,
>>>>>>> - NULL,
>>>>>>> - NULL);
>>>>>>> -
>>>>>>> /*
>>>>>>> * Setup and Remove actions for remote ports are handled
>>>>>>> * in the service functions below.
>>>>>>> @@ -617,34 +624,6 @@ send_vendor_fail:
>>>>>>> }
>>>>>>> EXPORT_SYMBOL(fc_host_post_vendor_event);
>>>>>>>
>>>>>>> -
>>>>>>> -
>>>>>>> -static __init int fc_transport_init(void)
>>>>>>> -{
>>>>>>> - int error;
>>>>>>> -
>>>>>>> - atomic_set(&fc_event_seq, 0);
>>>>>>> -
>>>>>>> - error = transport_class_register(&fc_host_class);
>>>>>>> - if (error)
>>>>>>> - return error;
>>>>>>> - error = transport_class_register(&fc_vport_class);
>>>>>>> - if (error)
>>>>>>> - return error;
>>>>>>> - error = transport_class_register(&fc_rport_class);
>>>>>>> - if (error)
>>>>>>> - return error;
>>>>>>> - return transport_class_register(&fc_transport_class);
>>>>>>> -}
>>>>>>> -
>>>>>>> -static void __exit fc_transport_exit(void)
>>>>>>> -{
>>>>>>> - transport_class_unregister(&fc_transport_class);
>>>>>>> - transport_class_unregister(&fc_rport_class);
>>>>>>> - transport_class_unregister(&fc_host_class);
>>>>>>> - transport_class_unregister(&fc_vport_class);
>>>>>>> -}
>>>>>>> -
>>>>>>> /*
>>>>>>> * FC Remote Port Attribute Management
>>>>>>> */
>>>>>>> @@ -1655,6 +1634,224 @@ static struct attribute_group
>>>>>>> fc_statistics_group = {
>>>>>>> .attrs = fc_statistics_attrs,
>>>>>>> };
>>>>>>>
>>>>>>> +/*
>>>>>>> + * FCoE Host attributes
>>>>>>> + */
>>>>>>> +
>>>>>>> +/* Convert mac_addressing_mode values to ascii string name */
>>>>>>> +static struct {
>>>>>>> + enum fcoe_mac_addressing_mode value;
>>>>>>> + char *name;
>>>>>>> +} mac_addressing_mode_names [] = {
>>>>>>> + { FCF_SELECTED, "FCF selcted - Either FPMA
>>>>>>> or SPMA" },
>>>>>>> + { SPMA_ONLY, "SPMA only - ENode
>>>>>>> Solicites SPMA only support" },
>>>>>>> + { FPMA_ONLY, "FPMA only - ENode
>>>>>>> Solicites SPMA only support" },
>>>>>>> + { SPMA_PREFERRED, "SPMA preferred - ENode
>>>>>>> Solicites SPMA
>>>>>>> preferred support" },
>>>>>>> + { FPMA_PREFERRED, "FPMA preferred - ENode
>>>>>>> Solicites FPMA
>>>>>>> preferred support" },
>>>>>>> +};
>>>>>>> +fc_enum_name_search(mac_addressing_mode,
>>>>>>> fcoe_mac_addressing_mode,
>>>>>>> mac_addressing_mode_names)
>>>>>>> +#define FCOE_MAC_ADDR_MODE_MAX_NAMELEN 80
>>>>>>> +
>>>>>>> +/* Show MAC address members of the FCoE host attributes */
>>>>>>> +static ssize_t
>>>>>>> +fcoe_host_attrs_show(const struct kobject *kobj, char *buf,
>>>>>>> unsigned long offset)
>>>>>>> +{
>>>>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>>>> + struct fcoe_host_attrs *stats;
>>>>>>> + ssize_t ret = -ENOENT;
>>>>>>> +
>>>>>>> + if (i->f->get_fcoe_host) {
>>>>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>>>>> + if (stats)
>>>>>>> + ret = snprintf(buf, 20, "0x%x\n",
>>>>>>> + *(u16 *)(((u8 *) stats) + offset));
>>>>>>> + }
>>>>>>> + return ret;
>>>>>>> +}
>>>>>>> +
>>>>>>> +/* generate a read-only FCoE Host attribute */
>>>>>>> +#define
>>>>>>> fcoe_host_attrs
>>>>>>> (name) \
>>>>>>> +static ssize_t show_fcoe_host_attrs_##name(struct kobject
>>>>>>> *cd, \
>>>>>>> + struct kobj_attribute
>>>>>>> *attr, \
>>>>>>> + char
>>>>>>> *buf) \
>>>>>>> +
>>>>>>> { \
>>>>>>> + return fcoe_host_attrs_show(cd,
>>>>>>> buf, \
>>>>>>> + offsetof(struct fcoe_host_attrs,
>>>>>>> name)); \
>>>>>>> +
>>>>>>> } \
>>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>>>> show_fcoe_host_attrs_##name, NULL)
>>>>>>> +
>>>>>>> +fcoe_host_attrs(vlan_id);
>>>>>>> +
>>>>>>> +#define fcoe_host_attrs_enum_attr(title,
>>>>>>> maxlen) \
>>>>>>> +static ssize_t show_fcoe_host_attrs_##title(struct kobject
>>>>>>> *kobj, \
>>>>>>> + struct kobj_attribute
>>>>>>> *attr, \
>>>>>>> + char
>>>>>>> *buf) \
>>>>>>> +
>>>>>>> { \
>>>>>>> + struct device *dev = kobj_to_dev(kobj-
>>>>>>>
>>>>>>>> parent); \
>>>>>>>>
>>>>>>> + struct Scsi_Host *shost =
>>>>>>> transport_class_to_shost(dev); \
>>>>>>> + struct fc_internal *i = to_fc_internal(shost-
>>>>>>>
>>>>>>>> transportt); \
>>>>>>>>
>>>>>>> + struct fcoe_host_attrs
>>>>>>> *stats; \
>>>>>>> + ssize_t ret = -
>>>>>>> EINVAL; \
>>>>>>> + const char
>>>>>>> *name; \
>>>>>>> +
>>>>>>> \
>>>>>>> + if (i->f->get_fcoe_host)
>>>>>>> { \
>>>>>>> + stats = (i->f->get_fcoe_host)
>>>>>>> (shost); \
>>>>>>> + if(stats)
>>>>>>> { \
>>>>>>> + name = get_fc_##title##_name(stats-
>>>>>>>
>>>>>>>> mac_addressing_mode); \
>>>>>>>>
>>>>>>> + if
>>>>>>> (name) \
>>>>>>> + ret= snprintf(buf, maxlen, "%s\n",
>>>>>>> name); \
>>>>>>> +
>>>>>>> } \
>>>>>>> +
>>>>>>> } \
>>>>>>> + return
>>>>>>> ret
>>>>>>> ; \
>>>>>>> +
>>>>>>> } \
>>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, title, S_IRUGO,
>>>>>>> show_fcoe_host_attrs_##title, NULL)
>>>>>>> +
>>>>>>> +fcoe_host_attrs_enum_attr(mac_addressing_mode,
>>>>>>> FCOE_MAC_ADDR_MODE_MAX_NAMELEN);
>>>>>>> +
>>>>>>> +static ssize_t
>>>>>>> +fcoe_mac_addr_show(const struct kobject *kobj, char *buf,
>>>>>>> unsigned
>>>>>>> long offset)
>>>>>>> +{
>>>>>>> + struct device *dev = kobj_to_dev(kobj->parent);
>>>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>>>> + struct fcoe_host_attrs *stats;
>>>>>>> + ssize_t ret = -ENOENT;
>>>>>>> +
>>>>>>> + if (offset > sizeof(struct fcoe_host_attrs))
>>>>>>> + WARN_ON(1);
>>>>>>> +
>>>>>>> + if (i->f->get_fcoe_host) {
>>>>>>> + stats = (i->f->get_fcoe_host)(shost);
>>>>>>> + if (stats) {
>>>>>>> + u8 *macaddr = (u8*)(((u8 *) stats) +
>>>>>>> offset);
>>>>>>> + ret = sysfs_format_mac(buf, macaddr, 6);
>>>>>>> + }
>>>>>>> + }
>>>>>>> + return ret;
>>>>>>> +}
>>>>>>> +
>>>>>>> +/* generate a read-only FCoE MAC address host attribute */
>>>>>>> +#define
>>>>>>> fcoe_mac_addr(name) \
>>>>>>> +static ssize_t show_fcoe_mac_addr_##name(struct kobject
>>>>>>> *cd, \
>>>>>>> + struct kobj_attribute
>>>>>>> *attr, \
>>>>>>> + char
>>>>>>> *buf) \
>>>>>>> +
>>>>>>> { \
>>>>>>> + return fcoe_mac_addr_show(cd,
>>>>>>> buf, \
>>>>>>> + offsetof(struct fcoe_host_attrs,
>>>>>>> name)); \
>>>>>>> +
>>>>>>> } \
>>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>>>> show_fcoe_mac_addr_##name, NULL)
>>>>>>> +
>>>>>>> +fcoe_mac_addr(enode_mac_address);
>>>>>>> +fcoe_mac_addr(vn_port_mac_address);
>>>>>>> +fcoe_mac_addr(vf_port_mac_address);
>>>>>>> +
>>>>>>> +static struct attribute *fcoe_host_attrs[] = {
>>>>>>> + &kobject_attr_fcoe_enode_mac_address.attr,
>>>>>>> + &kobject_attr_fcoe_vn_port_mac_address.attr,
>>>>>>> + &kobject_attr_fcoe_vf_port_mac_address.attr,
>>>>>>> + &kobject_attr_fcoe_vlan_id.attr,
>>>>>>> + &kobject_attr_fcoe_mac_addressing_mode.attr,
>>>>>>> + NULL
>>>>>>> +};
>>>>>>> +
>>>>>>> +static struct attribute_group fcoe_host_attrs_group = {
>>>>>>> + .attrs = fcoe_host_attrs,
>>>>>>> +};
>>>>>>> +
>>>>>>> +/* Read FCoE host statistics */
>>>>>>> +static ssize_t
>>>>>>> +fcoe_host_statistics_show(const struct kobject *kobj, char
>>>>>>> *buf,
>>>>>>> unsigned long offset)
>>>>>>> +{
>>>>>>> + struct device *dev = kobj_to_dev(kobj->parent->parent);
>>>>>>> + struct Scsi_Host *shost = transport_class_to_shost(dev);
>>>>>>> + struct fc_internal *i = to_fc_internal(shost->transportt);
>>>>>>> + struct fcoe_host_statistics *stats;
>>>>>>> + ssize_t ret = -ENOENT;
>>>>>>> +
>>>>>>> + if (offset > sizeof(struct fcoe_host_statistics) ||
>>>>>>> + offset % sizeof(u64) != 0)
>>>>>>> + WARN_ON(1);
>>>>>>> +
>>>>>>> + if (i->f->get_fcoe_host_stats) {
>>>>>>> + stats = (i->f->get_fcoe_host_stats)(shost);
>>>>>>> + if (stats)
>>>>>>> + ret = snprintf(buf, 20, "0x%llx\n",
>>>>>>> + (unsigned long long)*(u64 *)(((u8 *)
>>>>>>> stats) + offset));
>>>>>>> + }
>>>>>>> + return ret;
>>>>>>> +}
>>>>>>> +
>>>>>>> +/* generate a read-only FCoE host staistics attribute */
>>>>>>> +#define
>>>>>>> fcoe_host_statistics
>>>>>>> (name) \
>>>>>>> +static ssize_t show_fcoe_host_statistics_##name(struct kobject
>>>>>>> *cd, \
>>>>>>> + struct kobj_attribute
>>>>>>> *attr, \
>>>>>>> + char
>>>>>>> *buf) \
>>>>>>> +
>>>>>>> { \
>>>>>>> + return fcoe_host_statistics_show(cd,
>>>>>>> buf, \
>>>>>>> + offsetof(struct fcoe_host_statistics,
>>>>>>> name)); \
>>>>>>> +
>>>>>>> } \
>>>>>>> +static FCOE_KOBJECT_ATTR(fcoe, name, S_IRUGO,
>>>>>>> show_fcoe_host_statistics_##name, NULL)
>>>>>>> +
>>>>>>> +fcoe_host_statistics(link_failure_count);
>>>>>>> +fcoe_host_statistics(loss_of_sync_count);
>>>>>>> +fcoe_host_statistics(loss_of_signal_count);
>>>>>>> +fcoe_host_statistics(prim_seq_protocol_err_count);
>>>>>>> +fcoe_host_statistics(invalid_tx_word_count);
>>>>>>> +fcoe_host_statistics(invalid_crc_count);
>>>>>>> +
>>>>>>> +static struct attribute *fcoe_host_statistics[] = {
>>>>>>> + &kobject_attr_fcoe_link_failure_count.attr,
>>>>>>> + &kobject_attr_fcoe_loss_of_sync_count.attr,
>>>>>>> + &kobject_attr_fcoe_loss_of_signal_count.attr,
>>>>>>> + &kobject_attr_fcoe_prim_seq_protocol_err_count.attr,
>>>>>>> + &kobject_attr_fcoe_invalid_tx_word_count.attr,
>>>>>>> + &kobject_attr_fcoe_invalid_crc_count.attr,
>>>>>>> + NULL
>>>>>>> +};
>>>>>>> +
>>>>>>> +static struct attribute_group fcoe_host_statistics_group = {
>>>>>>> + .attrs = fcoe_host_statistics,
>>>>>>> +};
>>>>>>> +
>>>>>>> +static int fc_host_configure(struct transport_container *tc,
>>>>>>> struct
>>>>>>> device *dev,
>>>>>>> + struct device *cdev)
>>>>>>> +{
>>>>>>> + int error;
>>>>>>> +
>>>>>>> + /* Create a fcoe sub-directory under /sys/class/fc_host/
>>>>>>> hostX/ */
>>>>>>> + private_fcoe_kobj = kobject_create_and_add("fcoe", &cdev-
>>>>>>>
>>>>>>>> kobj);
>>>>>>>>
>>>>>>> + if (private_fcoe_kobj) {
>>>>>>> + /* Create the files associated with this kobject
>>>>>>> */
>>>>>>> + error = sysfs_create_group(private_fcoe_kobj,
>>>>>>> &fcoe_host_attrs_group);
>>>>>>> + if (error) {
>>>>>>> + kobject_put(private_fcoe_kobj);
>>>>>>> + goto error;
>>>>>>> + }
>>>>>>> + }
>>>>>>> +
>>>>>>> + /* Create a statistics sub-directory under /sys/class/
>>>>>>> fc_host/
>>>>>>> hostX/fcoe/ */
>>>>>>> + private_fcoe_stat_kobj =
>>>>>>> kobject_create_and_add("statistics",
>>>>>>> private_fcoe_kobj);
>>>>>>> + if (private_fcoe_stat_kobj) {
>>>>>>> + /* Create the files associated with this kobject
>>>>>>> */
>>>>>>> + error = sysfs_create_group(private_fcoe_stat_kobj,
>>>>>>> &fcoe_host_statistics_group);
>>>>>>> + if (error) {
>>>>>>> + kobject_put(private_fcoe_stat_kobj);
>>>>>>> + goto error;
>>>>>>> + }
>>>>>>> + }
>>>>>>> +
>>>>>>> +error:
>>>>>>> + return 0;
>>>>>>> +}
>>>>>>> +
>>>>>>> +static DECLARE_TRANSPORT_CLASS(fc_host_class,
>>>>>>> + "fc_host",
>>>>>>> + fc_host_setup,
>>>>>>> + NULL,
>>>>>>> + fc_host_configure);
>>>>>>> +
>>>>>>>
>>>>>>> /* Host Vport Attributes */
>>>>>>>
>>>>>>> @@ -3342,6 +3539,33 @@ fc_vport_sched_delete(struct work_struct
>>>>>>> *work)
>>>>>>> vport->channel, stat);
>>>>>>> }
>>>>>>>
>>>>>>> +static __init int fc_transport_init(void)
>>>>>>> +{
>>>>>>> + int error;
>>>>>>> +
>>>>>>> + atomic_set(&fc_event_seq, 0);
>>>>>>> +
>>>>>>> + error = transport_class_register(&fc_host_class);
>>>>>>> + if (error)
>>>>>>> + return error;
>>>>>>> + error = transport_class_register(&fc_vport_class);
>>>>>>> + if (error)
>>>>>>> + return error;
>>>>>>> + error = transport_class_register(&fc_rport_class);
>>>>>>> + if (error)
>>>>>>> + return error;
>>>>>>> + return transport_class_register(&fc_transport_class);
>>>>>>> +}
>>>>>>> +
>>>>>>> +static void __exit fc_transport_exit(void)
>>>>>>> +{
>>>>>>> + transport_class_unregister(&fc_transport_class);
>>>>>>> + transport_class_unregister(&fc_rport_class);
>>>>>>> + transport_class_unregister(&fc_host_class);
>>>>>>> + transport_class_unregister(&fc_vport_class);
>>>>>>> + kobject_put(private_fcoe_kobj);
>>>>>>> + kobject_put(private_fcoe_stat_kobj);
>>>>>>> +}
>>>>>>>
>>>>>>> /* Original Author: Martin Hicks */
>>>>>>> MODULE_AUTHOR("James Smart");
>>>>>>> diff --git a/include/scsi/scsi_transport_fc.h b/include/scsi/
>>>>>>> scsi_transport_fc.h
>>>>>>> index c9184f7..def2e23 100644
>>>>>>> --- a/include/scsi/scsi_transport_fc.h
>>>>>>> +++ b/include/scsi/scsi_transport_fc.h
>>>>>>> @@ -30,6 +30,8 @@
>>>>>>> #include <linux/sched.h>
>>>>>>> #include <scsi/scsi.h>
>>>>>>> #include <scsi/scsi_netlink.h>
>>>>>>> +#include <scsi/scsi_netlink.h>
>>>>>>> +#include <linux/etherdevice.h>
>>>>>>>
>>>>>>> struct scsi_transport_template;
>>>>>>>
>>>>>>> @@ -63,6 +65,7 @@ enum fc_port_type {
>>>>>>> FC_PORTTYPE_LPORT, /* (Private) Loop w/o FLPort
>>>>>>> */
>>>>>>> FC_PORTTYPE_PTP, /* Point to Point w/ another
>>>>>>> NPort */
>>>>>>> FC_PORTTYPE_NPIV, /* VPORT based on NPIV */
>>>>>>> + FC_PORTTYPE_VFPORT, /* Virtual Fabric PORT */
>>>>>>> };
>>>>>>>
>>>>>>>
>>>>>>> @@ -428,6 +431,53 @@ struct fc_host_statistics {
>>>>>>> u64 fcp_output_megabytes;
>>>>>>> };
>>>>>>>
>>>>>>> +/*
>>>>>>> + * FCoE Local Port (Host) Attributes
>>>>>>> + *
>>>>>>> + * Fixed attributes are not expected to change. The driver is
>>>>>>> + * expected to set these values after successfully calling
>>>>>>> scsi_add_host().
>>>>>>> + * The transport fully manages all get functions w/o driver
>>>>>>> interaction.
>>>>>>> + *
>>>>>>> + * Dynamic attributes are expected to change. The driver
>>>>>>> participates
>>>>>>> + * in all get/set operations via functions provided by the
>>>>>>> driver.
>>>>>>> + *
>>>>>>> + */
>>>>>>> +struct fcoe_host_attrs {
>>>>>>> + /* Fixed attributes */
>>>>>>> + uint8_t enode_mac_address[ETH_ALEN];
>>>>>>> +
>>>>>>> + /* Dynamic attributes */
>>>>>>> + uint8_t vn_port_mac_address[ETH_ALEN];
>>>>>>> + uint8_t vf_port_mac_address[ETH_ALEN];
>>>>>>> + u16 vlan_id;
>>>>>>> + u16 mac_addressing_mode;
>>>>>>> +};
>>>>>>> +
>>>>>>> +/* FCoE Statistics - Following proposal for FC_BB_E FC Link
>>>>>>> error
>>>>>>> status block representation
>>>>>>> + * from T11/09-204v0 guidelines
>>>>>>> + */
>>>>>>> +struct fcoe_host_statistics {
>>>>>>> + /* Port statistics */
>>>>>>> + u64 link_failure_count;
>>>>>>> + u64 loss_of_sync_count;
>>>>>>> + u64 loss_of_signal_count;
>>>>>>> + u64 prim_seq_protocol_err_count;
>>>>>>> + u64 invalid_tx_word_count;
>>>>>>> + u64 invalid_crc_count;
>>>>>>> +};
>>>>>>> +
>>>>>>> +
>>>>>>> +/*
>>>>>>> + * mac_addressing_mode: If you alter this, you also need to
>>>>>>> alter
>>>>>>> + * scsi_transport_fc.c (for the ascii descriptions).
>>>>>>> + */
>>>>>>> +enum fcoe_mac_addressing_mode {
>>>>>>> + FCF_SELECTED,
>>>>>>> + SPMA_ONLY,
>>>>>>> + FPMA_ONLY,
>>>>>>> + SPMA_PREFERRED,
>>>>>>> + FPMA_PREFERRED,
>>>>>>> +};
>>>>>>>
>>>>>>> /*
>>>>>>> * FC Event Codes - Polled and Async, following FC HBAAPI v2.0
>>>>>>> guidelines
>>>>>>> @@ -600,6 +650,9 @@ struct fc_function_template {
>>>>>>> struct fc_host_statistics * (*get_fc_host_stats)(struct
>>>>>>> Scsi_Host *);
>>>>>>> void (*reset_fc_host_stats)(struct Scsi_Host *);
>>>>>>>
>>>>>>> + struct fcoe_host_attrs* (*get_fcoe_host)(struct
>>>>>>> Scsi_Host *);
>>>>>>> + struct fcoe_host_statistics* (*get_fcoe_host_stats)(struct
>>>>>>> Scsi_Host *);
>>>>>>> +
>>>>>>> int (*issue_fc_host_lip)(struct Scsi_Host *);
>>>>>>>
>>>>>>> void (*dev_loss_tmo_callbk)(struct fc_rport *);
>>>>>>> --
>>>>>>> 1.6.2.rc1.30.gd43c
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>
>>>
>>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-07-06 21:14 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-04-18 1:01 Patch - To display FCoE Host information and statistics in sysfs Giridhar Malavali
[not found] ` <0DC3E4D6-2BD9-4A1B-9A70-5395FEAC1F04@qlogic.com>
[not found] ` <49F9D93A.9070906@emulex.com>
2009-05-05 7:07 ` Giridhar Malavali
2009-06-02 17:34 ` Giridhar Malavali
2009-06-03 15:24 ` James Smart
2009-06-24 23:13 ` Giridhar Malavali
2009-07-06 21:14 ` Giridhar Malavali
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.