[v1] eal: fix structure marker 4B type error
Checks
Commit Message
Use the 32bit typedef for structure 4B alignment marking.
Fixes: 2b393160a480 ("eal: introduce structure markers")
Cc: stable@dpdk.org
Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
lib/librte_eal/common/include/rte_common.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On 03/08, Haiyue Wang wrote:
>Use the 32bit typedef for structure 4B alignment marking.
>
>Fixes: 2b393160a480 ("eal: introduce structure markers")
>Cc: stable@dpdk.org
>
>Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
>---
> lib/librte_eal/common/include/rte_common.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/lib/librte_eal/common/include/rte_common.h b/lib/librte_eal/common/include/rte_common.h
>index 4b5f3a31f..3662545d2 100644
>--- a/lib/librte_eal/common/include/rte_common.h
>+++ b/lib/librte_eal/common/include/rte_common.h
>@@ -344,7 +344,7 @@ __extension__ typedef uint8_t RTE_MARKER8[0];
> /** Marker for 2B alignment in a structure. */
> __extension__ typedef uint16_t RTE_MARKER16[0];
> /** Marker for 4B alignment in a structure. */
>-__extension__ typedef uint16_t RTE_MARKER32[0];
>+__extension__ typedef uint32_t RTE_MARKER32[0];
> /** Marker for 8B alignment in a structure. */
> __extension__ typedef uint64_t RTE_MARKER64[0];
>
>--
>2.25.1
>
Acked-by: Xiaolong Ye <xiaolong.ye@intel.com>
On Sun, Mar 8, 2020 at 10:57 AM Ye Xiaolong <xiaolong.ye@intel.com> wrote:
>
> On 03/08, Haiyue Wang wrote:
> >Use the 32bit typedef for structure 4B alignment marking.
> >
> >Fixes: 2b393160a480 ("eal: introduce structure markers")
> >Cc: stable@dpdk.org
> >
> >Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
> >---
> > lib/librte_eal/common/include/rte_common.h | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >diff --git a/lib/librte_eal/common/include/rte_common.h b/lib/librte_eal/common/include/rte_common.h
> >index 4b5f3a31f..3662545d2 100644
> >--- a/lib/librte_eal/common/include/rte_common.h
> >+++ b/lib/librte_eal/common/include/rte_common.h
> >@@ -344,7 +344,7 @@ __extension__ typedef uint8_t RTE_MARKER8[0];
> > /** Marker for 2B alignment in a structure. */
> > __extension__ typedef uint16_t RTE_MARKER16[0];
> > /** Marker for 4B alignment in a structure. */
> >-__extension__ typedef uint16_t RTE_MARKER32[0];
> >+__extension__ typedef uint32_t RTE_MARKER32[0];
> > /** Marker for 8B alignment in a structure. */
> > __extension__ typedef uint64_t RTE_MARKER64[0];
> >
> >--
> >2.25.1
> >
>
> Acked-by: Xiaolong Ye <xiaolong.ye@intel.com>
Acked-by: Jerin Jacob <jerinj@marvell.com>
On Sun, Mar 8, 2020 at 5:22 AM Haiyue Wang <haiyue.wang@intel.com> wrote:
> Use the 32bit typedef for structure 4B alignment marking.
>
> Fixes: 2b393160a480 ("eal: introduce structure markers")
> Cc: stable@dpdk.org
>
> Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
Acked-by: Xiaolong Ye <xiaolong.ye@intel.com>
Acked-by: Jerin Jacob <jerinj@marvell.com>
Applied, thanks.
@@ -344,7 +344,7 @@ __extension__ typedef uint8_t RTE_MARKER8[0];
/** Marker for 2B alignment in a structure. */
__extension__ typedef uint16_t RTE_MARKER16[0];
/** Marker for 4B alignment in a structure. */
-__extension__ typedef uint16_t RTE_MARKER32[0];
+__extension__ typedef uint32_t RTE_MARKER32[0];
/** Marker for 8B alignment in a structure. */
__extension__ typedef uint64_t RTE_MARKER64[0];