> -----Original Message----- > From: Ananyev, Konstantin > Sent: Wednesday, January 17, 2018 11:34 AM > To: Nicolau, Radu <radu.nico...@intel.com>; dev@dpdk.org > Cc: Lu, Wenzhuo <wenzhuo...@intel.com>; Zhao, XinfengX > <xinfengx.z...@intel.com>; De Lara Guarch, Pablo > <pablo.de.lara.gua...@intel.com> > Subject: RE: [PATCH] net/ixgbe: check if security capabilities are enabled by > HW > > Hi Radu, > > > -----Original Message----- > > From: Nicolau, Radu > > Sent: Wednesday, January 17, 2018 11:19 AM > > To: dev@dpdk.org > > Cc: Lu, Wenzhuo <wenzhuo...@intel.com>; Ananyev, Konstantin > > <konstantin.anan...@intel.com>; Zhao, XinfengX > > <xinfengx.z...@intel.com>; De Lara Guarch, Pablo > > <pablo.de.lara.gua...@intel.com>; Nicolau, Radu > > <radu.nico...@intel.com> > > Subject: [PATCH] net/ixgbe: check if security capabilities are enabled > > by HW > > > > Check if the security enable bits are not fused before setting offload > > capabilities for security > > In theory dev_info_get() - could be called at any stage of device > configuration or even when RX/TX is active. > Do you really want to assert SECRXCTRL at that moment? > Probably better to do this only once at init time and then just use some > stored value? > Konstantin >
Yes, that's true. I will send a v2