diff options
author | Michael Chan <mchan@broadcom.com> | 2010-12-23 02:42:58 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-12-23 14:44:29 -0500 |
commit | 939b82e5bde56a98c72eccde2e3a88d32bffad4a (patch) | |
tree | e10e7ddaecef208d6fc0b11476a8943912663efc /drivers/net/cnic_if.h | |
parent | 8adc9240f98a816f7e9b3d93b9446a790110e062 (diff) |
cnic: Improve ->iscsi_nl_msg_send()
1. Change first parameter from cnic_dev to ulp_handle which is the hba
pointer. All other similar upcalls are using hba pointer. The callee
can then directly reference the hba without conversion.
2. Change return value from void to int so that an error code can be
passed back. This allows the operation to be retried.
Signed-off-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/cnic_if.h')
-rw-r--r-- | drivers/net/cnic_if.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/cnic_if.h b/drivers/net/cnic_if.h index 0dbeaec4f03a..33333e735f95 100644 --- a/drivers/net/cnic_if.h +++ b/drivers/net/cnic_if.h | |||
@@ -301,7 +301,7 @@ struct cnic_ulp_ops { | |||
301 | void (*cm_abort_complete)(struct cnic_sock *); | 301 | void (*cm_abort_complete)(struct cnic_sock *); |
302 | void (*cm_remote_close)(struct cnic_sock *); | 302 | void (*cm_remote_close)(struct cnic_sock *); |
303 | void (*cm_remote_abort)(struct cnic_sock *); | 303 | void (*cm_remote_abort)(struct cnic_sock *); |
304 | void (*iscsi_nl_send_msg)(struct cnic_dev *dev, u32 msg_type, | 304 | int (*iscsi_nl_send_msg)(void *ulp_ctx, u32 msg_type, |
305 | char *data, u16 data_size); | 305 | char *data, u16 data_size); |
306 | struct module *owner; | 306 | struct module *owner; |
307 | atomic_t ref_count; | 307 | atomic_t ref_count; |