netdev - Re: [PATCH V2 net-next 1/2] qed: Add support for hardware offloaded FCoE.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LRH.2.00.1701301035450.7335@mvluser05.qlc.com>
Date:   Mon, 30 Jan 2017 10:53:16 -0800
From:   Arun Easi <arun.easi@...ium.com>
To:     Hannes Reinecke <hare@...e.de>
CC:     "Dupuis, Chad" <chad.dupuis@...ium.com>,
        <martin.petersen@...cle.com>, <linux-scsi@...r.kernel.org>,
        <fcoe-devel@...n-fcoe.org>, <netdev@...r.kernel.org>,
        <yuval.mintz@...ium.com>, <QLogic-Storage-Upstream@...ium.com>
Subject: Re: [PATCH V2 net-next 1/2] qed: Add support for hardware offloaded
 FCoE.

On Mon, 30 Jan 2017, 1:44am, Hannes Reinecke wrote:

> On 01/25/2017 09:33 PM, Dupuis, Chad wrote:
> > From: Arun Easi <arun.easi@...gic.com>
> > 
> > This adds the backbone required for the various HW initalizations
> > which are necessary for the FCoE driver (qedf) for QLogic FastLinQ
> > 4xxxx line of adapters - FW notification, resource initializations, etc.
> > 
> > Signed-off-by: Arun Easi <arun.easi@...ium.com>
> > Signed-off-by: Yuval Mintz <yuval.mintz@...ium.com>
> > ---
> >  drivers/net/ethernet/qlogic/Kconfig               |   3 +
> >  drivers/net/ethernet/qlogic/qed/Makefile          |   1 +
> >  drivers/net/ethernet/qlogic/qed/qed.h             |  11 +
> >  drivers/net/ethernet/qlogic/qed/qed_cxt.c         |  98 ++-
> >  drivers/net/ethernet/qlogic/qed/qed_cxt.h         |   3 +
> >  drivers/net/ethernet/qlogic/qed/qed_dcbx.c        |  13 +-
> >  drivers/net/ethernet/qlogic/qed/qed_dcbx.h        |   5 +-
> >  drivers/net/ethernet/qlogic/qed/qed_dev.c         | 205 ++++-
> >  drivers/net/ethernet/qlogic/qed/qed_dev_api.h     |  42 +
> >  drivers/net/ethernet/qlogic/qed/qed_fcoe.c        | 990 ++++++++++++++++++++++
> >  drivers/net/ethernet/qlogic/qed/qed_fcoe.h        |  52 ++
> >  drivers/net/ethernet/qlogic/qed/qed_hsi.h         | 781 ++++++++++++++++-
> >  drivers/net/ethernet/qlogic/qed/qed_hw.c          |   3 +
> >  drivers/net/ethernet/qlogic/qed/qed_ll2.c         |  25 +
> >  drivers/net/ethernet/qlogic/qed/qed_ll2.h         |   2 +-
> >  drivers/net/ethernet/qlogic/qed/qed_main.c        |   7 +
> >  drivers/net/ethernet/qlogic/qed/qed_mcp.c         |   3 +
> >  drivers/net/ethernet/qlogic/qed/qed_mcp.h         |   1 +
> >  drivers/net/ethernet/qlogic/qed/qed_reg_addr.h    |   8 +
> >  drivers/net/ethernet/qlogic/qed/qed_sp.h          |   4 +
> >  drivers/net/ethernet/qlogic/qed/qed_sp_commands.c |   3 +
> >  include/linux/qed/common_hsi.h                    |  10 +-
> >  include/linux/qed/fcoe_common.h                   | 715 ++++++++++++++++
> >  include/linux/qed/qed_fcoe_if.h                   | 145 ++++
> >  include/linux/qed/qed_if.h                        |  41 +-
> >  25 files changed, 3152 insertions(+), 19 deletions(-)
> >  create mode 100644 drivers/net/ethernet/qlogic/qed/qed_fcoe.c
> >  create mode 100644 drivers/net/ethernet/qlogic/qed/qed_fcoe.h
> >  create mode 100644 include/linux/qed/fcoe_common.h
> >  create mode 100644 include/linux/qed/qed_fcoe_if.h
> > 
> [ .. ]
> > diff --git a/drivers/net/ethernet/qlogic/qed/qed_dcbx.h b/drivers/net/ethernet/qlogic/qed/qed_dcbx.h
> > index d70300f..0fabe97 100644
> > --- a/drivers/net/ethernet/qlogic/qed/qed_dcbx.h
> > +++ b/drivers/net/ethernet/qlogic/qed/qed_dcbx.h
> > @@ -57,7 +57,6 @@ struct qed_dcbx_app_data {
> >  	u8 tc;			/* Traffic Class */
> >  };
> >  
> > -#ifdef CONFIG_DCB
> >  #define QED_DCBX_VERSION_DISABLED       0
> >  #define QED_DCBX_VERSION_IEEE           1
> >  #define QED_DCBX_VERSION_CEE            2
> > @@ -73,7 +72,6 @@ struct qed_dcbx_set {
> >  	struct qed_dcbx_admin_params config;
> >  	u32 ver_num;
> >  };
> > -#endif
> >  
> >  struct qed_dcbx_results {
> >  	bool dcbx_enabled;
> > @@ -97,9 +95,8 @@ struct qed_dcbx_info {
> >  	struct qed_dcbx_results results;
> >  	struct dcbx_mib operational;
> >  	struct dcbx_mib remote;
> > -#ifdef CONFIG_DCB
> >  	struct qed_dcbx_set set;
> > -#endif
> > +	struct qed_dcbx_get get;
> >  	u8 dcbx_cap;
> >  };
> >  
> Why did you remove the dependency on 'CONFIG_DCB'?

Thanks Hannes for the review.

The offloaded functions are not dependent on CONFIG_DCB; the DCB
functionality is handled by the firmware for them. The above '#ifdef'-ed 
part is shared by offloaded functions, and should be present regardless of 
the CONFIG_DCB setting.

Regards,
-Arun

> 
> Other than that:
> 
> Reviewed-by: Hannes Reinecke <hare@...e.com>