On Thu, Apr 16, 2020 at 10:40 AM Ferruh Yigit <ferruh.yi...@intel.com>
wrote:
> On 4/15/2020 9:18 AM, Venkat Duvvuru wrote:
> > From: Michael Wildt <michael.wi...@broadcom.com>
> >
> > - Add infrastructure support
> > - Add tf_core open session support
> >
> > Signed-off-by: Michael Wildt <michael.wi...@broadcom.com>
> > Reviewed-by: Randy Schacher <stuart.schac...@broadcom.com>
> > Reviewed-by: Ajit Kumar Khaparde <ajit.khapa...@broadcom.com>
> > Signed-off-by: Venkat Duvvuru <venkatkumar.duvv...@broadcom.com>
>
> <...>
>
> > +int
> > +tfp_calloc(struct tfp_calloc_parms *parms)
> > +{
> > + if (parms == NULL)
> > + return -EINVAL;
> > +
> > + parms->mem_va = rte_zmalloc("tf",
> > + (parms->nitems * parms->size),
> > + parms->alignment);
> > + if (parms->mem_va == NULL) {
> > + PMD_DRV_LOG(ERR, "Allocate failed mem_va\n");
> > + return -ENOMEM;
> > + }
> > +
> > + parms->mem_pa = (void
> *)((uintptr_t)rte_mem_virt2iova(parms->mem_va));
> > + if (parms->mem_pa == (void *)RTE_BAD_IOVA) {
>
> This is causing a warning for 32bit icc [1], becuase 64-bit value
> converted to
> 32-bit pointer. Can you do the same casting that has been done in one line
> above, like [2].
>
> [1]
> .../drivers/net/bnxt/tf_core/tfp.c(110): warning #2259: non-pointer
> conversion
> from "rte_iova_t={uint64_t={__uint64_t={unsigned long long}}}" to "void *"
> may
> lose significant bits
> if (parms->mem_pa == (void *)RTE_BAD_IOVA) {
> ^
> [2]
> - if (parms->mem_pa == (void *)RTE_BAD_IOVA) {
> + if (parms->mem_pa == (void *)((uintptr_t)RTE_BAD_IOVA)) {
>
Makes sense. Thanks
