All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ivshmem: fix for modified mempool struct
@ 2016-07-01 16:26 Ferruh Yigit
  2016-07-01 16:37 ` Hunt, David
  2016-07-01 17:32 ` Thomas Monjalon
  0 siblings, 2 replies; 3+ messages in thread
From: Ferruh Yigit @ 2016-07-01 16:26 UTC (permalink / raw)
  To: dev; +Cc: Anatoly Burakov

struct rte_mempool changed its "ring" field to "pool_data"

"ring" field is accessed by ivshmem library, and updated to "pool_data"

This patch fixes the compile error:

== Build lib/librte_ivshmem
  CC rte_ivshmem.o
.../lib/librte_ivshmem/rte_ivshmem.c:
 In function 'add_mempool_to_metadata':
.../lib/librte_ivshmem/rte_ivshmem.c:584:32:
 error: 'const struct rte_mempool' has no member named 'ring'
  return add_ring_to_metadata(mp->ring, config);
                                ^~

Fixes: 449c49b93a6b ("mempool: support handler operations")
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 lib/librte_ivshmem/rte_ivshmem.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/librte_ivshmem/rte_ivshmem.c b/lib/librte_ivshmem/rte_ivshmem.c
index 5c83920..c26edb6 100644
--- a/lib/librte_ivshmem/rte_ivshmem.c
+++ b/lib/librte_ivshmem/rte_ivshmem.c
@@ -581,7 +581,7 @@ add_mempool_to_metadata(const struct rte_mempool *mp,
 	}
 
 	/* mempool consists of memzone and ring */
-	return add_ring_to_metadata(mp->ring, config);
+	return add_ring_to_metadata(mp->pool_data, config);
 }
 
 int
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ivshmem: fix for modified mempool struct
  2016-07-01 16:26 [PATCH] ivshmem: fix for modified mempool struct Ferruh Yigit
@ 2016-07-01 16:37 ` Hunt, David
  2016-07-01 17:32 ` Thomas Monjalon
  1 sibling, 0 replies; 3+ messages in thread
From: Hunt, David @ 2016-07-01 16:37 UTC (permalink / raw)
  To: dev



On 1/7/2016 5:26 PM, Ferruh Yigit wrote:
> struct rte_mempool changed its "ring" field to "pool_data"
>
> "ring" field is accessed by ivshmem library, and updated to "pool_data"
>
> This patch fixes the compile error:
>
> == Build lib/librte_ivshmem
>    CC rte_ivshmem.o
> .../lib/librte_ivshmem/rte_ivshmem.c:
>   In function 'add_mempool_to_metadata':
> .../lib/librte_ivshmem/rte_ivshmem.c:584:32:
>   error: 'const struct rte_mempool' has no member named 'ring'
>    return add_ring_to_metadata(mp->ring, config);
>                                  ^~
>
> Fixes: 449c49b93a6b ("mempool: support handler operations")
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>   lib/librte_ivshmem/rte_ivshmem.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/librte_ivshmem/rte_ivshmem.c b/lib/librte_ivshmem/rte_ivshmem.c
> index 5c83920..c26edb6 100644
> --- a/lib/librte_ivshmem/rte_ivshmem.c
> +++ b/lib/librte_ivshmem/rte_ivshmem.c
> @@ -581,7 +581,7 @@ add_mempool_to_metadata(const struct rte_mempool *mp,
>   	}
>   
>   	/* mempool consists of memzone and ring */
> -	return add_ring_to_metadata(mp->ring, config);
> +	return add_ring_to_metadata(mp->pool_data, config);
>   }
>   
>   int

Acked-by: David Hunt <david.hunt@intel.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ivshmem: fix for modified mempool struct
  2016-07-01 16:26 [PATCH] ivshmem: fix for modified mempool struct Ferruh Yigit
  2016-07-01 16:37 ` Hunt, David
@ 2016-07-01 17:32 ` Thomas Monjalon
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2016-07-01 17:32 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: dev, Anatoly Burakov

2016-07-01 17:26, Ferruh Yigit:
> struct rte_mempool changed its "ring" field to "pool_data"
> 
> "ring" field is accessed by ivshmem library, and updated to "pool_data"
> 
> This patch fixes the compile error:
> 
> == Build lib/librte_ivshmem
>   CC rte_ivshmem.o
> .../lib/librte_ivshmem/rte_ivshmem.c:
>  In function 'add_mempool_to_metadata':
> .../lib/librte_ivshmem/rte_ivshmem.c:584:32:
>  error: 'const struct rte_mempool' has no member named 'ring'
>   return add_ring_to_metadata(mp->ring, config);
>                                 ^~
> 
> Fixes: 449c49b93a6b ("mempool: support handler operations")
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-07-01 17:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-01 16:26 [PATCH] ivshmem: fix for modified mempool struct Ferruh Yigit
2016-07-01 16:37 ` Hunt, David
2016-07-01 17:32 ` Thomas Monjalon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.