diff mbox series

[v2,09/19] ipsec: fix missing header include

Message ID	20210115111052.16437-10-bruce.richardson@intel.com (mailing list archive)
State	Accepted, archived
Delegated to:	David Marchand
Headers	IronPort-SDR: QYjK401wMKfKSY6pknZBHarM5TfI5AZXX9sRWIT45ChxBM529qscwMf/Kt2NF4DuuQ3jdoX1gf FJJ1QM5yKDsw== IronPort-SDR: 3xORFrUzKoYDz8pGrPQmuFRdBtAQweBGFXIsF3Swwsz0ysMvRMI0O4LlvM4eyhEBCXWbyvTUby bjynN5T2KxSA== From: Bruce Richardson <bruce.richardson@intel.com> To: dev@dpdk.org Cc: david.marchand@redhat.com, Bruce Richardson <bruce.richardson@intel.com>, stable@dpdk.org, Konstantin Ananyev <konstantin.ananyev@intel.com>, Bernard Iremonger <bernard.iremonger@intel.com>, Vladimir Medvedkin <vladimir.medvedkin@intel.com>, Akhil Goyal <akhil.goyal@nxp.com> Date: Fri, 15 Jan 2021 11:10:41 +0000 Message-Id: <20210115111052.16437-10-bruce.richardson@intel.com> In-Reply-To: <20210115111052.16437-1-bruce.richardson@intel.com> References: <20210114110606.21142-1-bruce.richardson@intel.com> <20210115111052.16437-1-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v2 09/19] ipsec: fix missing header include Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	ensure headers have correct includes \| [v2,00/19] ensure headers have correct includes [v2,01/19] eal: fix missing header inclusion [v2,02/19] telemetry: fix missing header include [v2,03/19] ethdev: fix missing header include [v2,04/19] net: fix missing header include [v2,05/19] mbuf: fix missing header include [v2,06/19] bitratestats: fix missing header include [v2,07/19] rib: fix missing header includes [v2,08/19] vhost: fix missing header includes [v2,09/19] ipsec: fix missing header include [v2,10/19] fib: fix missing header includes [v2,11/19] table: fix missing header include [v2,12/19] pipeline: fix missing header includes [v2,13/19] metrics: fix variable declaration in header [v2,14/19] node: fix missing header include [v2,15/19] app: fix extra include paths for app builds [v2,16/19] app/chkincs: add chkincs app to verify headers [v2,17/19] eal: add missing include to mcslock [v2,18/19] eal/x86: add architecture-specific headers to chkincs [v2,19/19] test-meson-builds: add includes check to default x86 build

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Bruce Richardson Jan. 15, 2021, 11:10 a.m. UTC

  The rte_ipsec_sad.h header used the standard uintXX_t types, but did not
include stdint.h header for them.

Fixes: 401633d9c112 ("ipsec: add inbound SAD API")
Cc: stable@dpdk.org

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 lib/librte_ipsec/rte_ipsec_sad.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Ananyev, Konstantin Jan. 15, 2021, 11:35 a.m. UTC | #1

> -----Original Message-----
> From: Richardson, Bruce <bruce.richardson@intel.com>
> Sent: Friday, January 15, 2021 11:11 AM
> To: dev@dpdk.org
> Cc: david.marchand@redhat.com; Richardson, Bruce <bruce.richardson@intel.com>; stable@dpdk.org; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Iremonger, Bernard <bernard.iremonger@intel.com>; Medvedkin, Vladimir
> <vladimir.medvedkin@intel.com>; Akhil Goyal <akhil.goyal@nxp.com>
> Subject: [PATCH v2 09/19] ipsec: fix missing header include
> 
> The rte_ipsec_sad.h header used the standard uintXX_t types, but did not
> include stdint.h header for them.
> 
> Fixes: 401633d9c112 ("ipsec: add inbound SAD API")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
>  lib/librte_ipsec/rte_ipsec_sad.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/librte_ipsec/rte_ipsec_sad.h b/lib/librte_ipsec/rte_ipsec_sad.h
> index 3e67ab1e4b..66c703ff3a 100644
> --- a/lib/librte_ipsec/rte_ipsec_sad.h
> +++ b/lib/librte_ipsec/rte_ipsec_sad.h
> @@ -7,6 +7,7 @@
>  #define _RTE_IPSEC_SAD_H_
> 
>  #include <rte_compat.h>
> +#include <stdint.h>
> 
>  /**
>   * @file rte_ipsec_sad.h
> --

Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>

> 2.27.0

Vladimir Medvedkin Jan. 15, 2021, 11:37 a.m. UTC | #2

On 15/01/2021 11:10, Bruce Richardson wrote:
> The rte_ipsec_sad.h header used the standard uintXX_t types, but did not
> include stdint.h header for them.
> 
> Fixes: 401633d9c112 ("ipsec: add inbound SAD API")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
>   lib/librte_ipsec/rte_ipsec_sad.h | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/lib/librte_ipsec/rte_ipsec_sad.h b/lib/librte_ipsec/rte_ipsec_sad.h
> index 3e67ab1e4b..66c703ff3a 100644
> --- a/lib/librte_ipsec/rte_ipsec_sad.h
> +++ b/lib/librte_ipsec/rte_ipsec_sad.h
> @@ -7,6 +7,7 @@
>   #define _RTE_IPSEC_SAD_H_
>   
>   #include <rte_compat.h>
> +#include <stdint.h>
>   
>   /**
>    * @file rte_ipsec_sad.h
> 

Acked-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>

Vladimir Medvedkin Jan. 15, 2021, 11:45 a.m. UTC | #3

Hi Bruce,

Just a one nit, please put <stdint.h> before rte_ includes.

On 15/01/2021 11:37, Medvedkin, Vladimir wrote:
> 
> 
> On 15/01/2021 11:10, Bruce Richardson wrote:
>> The rte_ipsec_sad.h header used the standard uintXX_t types, but did not
>> include stdint.h header for them.
>>
>> Fixes: 401633d9c112 ("ipsec: add inbound SAD API")
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
>> ---
>>   lib/librte_ipsec/rte_ipsec_sad.h | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/librte_ipsec/rte_ipsec_sad.h 
>> b/lib/librte_ipsec/rte_ipsec_sad.h
>> index 3e67ab1e4b..66c703ff3a 100644
>> --- a/lib/librte_ipsec/rte_ipsec_sad.h
>> +++ b/lib/librte_ipsec/rte_ipsec_sad.h
>> @@ -7,6 +7,7 @@
>>   #define _RTE_IPSEC_SAD_H_
>>   #include <rte_compat.h>
>> +#include <stdint.h>
>>   /**
>>    * @file rte_ipsec_sad.h
>>
> 
> Acked-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
>

Bruce Richardson Jan. 15, 2021, 11:49 a.m. UTC | #4

On Fri, Jan 15, 2021 at 11:45:11AM +0000, Medvedkin, Vladimir wrote:
> Hi Bruce,
> 
> Just a one nit, please put <stdint.h> before rte_ includes.
>

Good point, I'll fix if I do a V3.
 
> On 15/01/2021 11:37, Medvedkin, Vladimir wrote:
> > 
> > 
> > On 15/01/2021 11:10, Bruce Richardson wrote:
> > > The rte_ipsec_sad.h header used the standard uintXX_t types, but did not
> > > include stdint.h header for them.
> > > 
> > > Fixes: 401633d9c112 ("ipsec: add inbound SAD API")
> > > Cc: stable@dpdk.org
> > > 
> > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> > > ---
> > >   lib/librte_ipsec/rte_ipsec_sad.h | 1 +
> > >   1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/lib/librte_ipsec/rte_ipsec_sad.h
> > > b/lib/librte_ipsec/rte_ipsec_sad.h
> > > index 3e67ab1e4b..66c703ff3a 100644
> > > --- a/lib/librte_ipsec/rte_ipsec_sad.h
> > > +++ b/lib/librte_ipsec/rte_ipsec_sad.h
> > > @@ -7,6 +7,7 @@
> > >   #define _RTE_IPSEC_SAD_H_
> > >   #include <rte_compat.h>
> > > +#include <stdint.h>
> > >   /**
> > >    * @file rte_ipsec_sad.h
> > > 
> > 
> > Acked-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
> > 
> 
> -- 
> Regards,
> Vladimir

diff mbox series

Patch

diff --git a/lib/librte_ipsec/rte_ipsec_sad.h b/lib/librte_ipsec/rte_ipsec_sad.h
index 3e67ab1e4b..66c703ff3a 100644
--- a/lib/librte_ipsec/rte_ipsec_sad.h
+++ b/lib/librte_ipsec/rte_ipsec_sad.h
@@ -7,6 +7,7 @@ 
 #define _RTE_IPSEC_SAD_H_
 
 #include <rte_compat.h>
+#include <stdint.h>
 
 /**
  * @file rte_ipsec_sad.h