On 06/13/2016 11:10 AM, Christian Ehrhardt wrote:
> Hmm,
> Hi again Oliver.
>
> I was too fast saying yes.
> I don't know what is different now but I clearly tested it wrong the
> first time.
> Now I get:
>
> CC rte_xen_lib.o
> /mnt/nvme/dpdk-16-07-pre-linking/drivers/net/xenvirt/rte_xen_lib.c:In
> function ?grant_gntalloc_mbuf_pool?:
> /mnt/nvme/dpdk-16-07-pre-linking/drivers/net/xenvirt/rte_xen_lib.c:441:6:error:
> ?mp? undeclared (first use in this function)
> if (mp->nb_mem_chunks != 1) {
> ^
> /mnt/nvme/dpdk-16-07-pre-linking/drivers/net/xenvirt/rte_xen_lib.c:441:6:note:
> each undeclared identifier is reported only once for each function it
> appears in
> /mnt/nvme/dpdk-16-07-pre-linking/drivers/net/xenvirt/rte_xen_lib.c:422:46:error:
> unused parameter ?mpool? [-Werror=unused-parameter]
> grant_gntalloc_mbuf_pool(struct rte_mempool *mpool, uint32_t pg_num,
> uint32_t *gref_arr, phys_addr_t *pa_arr, int mempool_idx)
> ^
> cc1: all warnings being treated as errors
>
>
> Not too hard, changing the mp to mpool on the two places the patch has
> inserted it gets it working.
>
Thanks, and sorry I did not test the compilation because I don't have
the xen libraries installed.
This shows that even reading the patch several times is less efficient
than a compiler ;)
I'll send a v2 soon.
