From: Ferruh Yigit <ferruh.yigit@intel.com> To: Yongseok Koh <yskoh@mellanox.com>, Shahaf Shuler <shahafs@mellanox.com> Cc: "dev@dpdk.org" <dev@dpdk.org>, "stable@dpdk.org" <stable@dpdk.org> Subject: Re: [dpdk-stable] [PATCH] net/mlx5: fix 32b compilation Date: Thu, 13 Jun 2019 14:29:40 +0100 Message-ID: <9b58271e-994d-d7d3-fd43-01a7f2105d23@intel.com> (raw) In-Reply-To: <10BD7CE0-1B5A-4756-955B-66754A9F2CF1@mellanox.com> On 6/12/2019 10:25 PM, Yongseok Koh wrote: > >> On Jun 11, 2019, at 10:32 PM, Shahaf Shuler <shahafs@mellanox.com> wrote: >> >> 32b Compilation output the below error: >> >> drivers/net/mlx5/mlx5_txq.c: In function 'mlx5_txq_ibv_new': >> error: format '%lx' expects argument of type 'long unsigned int', but >> argument 8 has type 'off_t' [-Werror=format=] >> DRV_LOG(DEBUG, "port %u: uar_mmap_offset 0x%lx", >> ^ > > off_t would have same size as pointer. > Changing lx to PRIxPTR would work? 'off_t' is no more same size as pointer, please check [1], otherwise what you said correct and code was working fine previously. Briefly, from build system we are asking/forcing off_t to be always 64-bit [1]: https://mails.dpdk.org/archives/dev/2019-June/133819.html > >> Fixes: 6bf10ab69be0 ("net/mlx5: support 32-bit systems") >> Cc: stable@dpdk.org >> >> Signed-off-by: Shahaf Shuler <shahafs@mellanox.com> >> --- >> drivers/net/mlx5/mlx5_txq.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c >> index ebb42cbffb..a421139e3c 100644 >> --- a/drivers/net/mlx5/mlx5_txq.c >> +++ b/drivers/net/mlx5/mlx5_txq.c >> @@ -566,8 +566,8 @@ mlx5_txq_ibv_new(struct rte_eth_dev *dev, uint16_t idx) >> txq_uar_init(txq_ctrl); >> if (qp.comp_mask & MLX5DV_QP_MASK_UAR_MMAP_OFFSET) { >> txq_ctrl->uar_mmap_offset = qp.uar_mmap_offset; >> - DRV_LOG(DEBUG, "port %u: uar_mmap_offset 0x%lx", >> - dev->data->port_id, txq_ctrl->uar_mmap_offset); >> + DRV_LOG(DEBUG, "port %u: uar_mmap_offset 0x%x", >> + dev->data->port_id, (int)(txq_ctrl->uar_mmap_offset)); >> } else { >> DRV_LOG(ERR, >> "port %u failed to retrieve UAR info, invalid" >> -- >> 2.12.0 >> >
next prev parent reply other threads:[~2019-06-13 13:29 UTC|newest] Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-06-12 5:32 Shahaf Shuler 2019-06-12 21:25 ` Yongseok Koh 2019-06-13 13:29 ` Ferruh Yigit [this message] 2019-06-13 13:35 ` Ferruh Yigit 2020-01-13 14:22 ` [dpdk-stable] [PATCH] net/mlx5: fix doorbell register offset type Viacheslav Ovsiienko 2020-01-13 14:29 ` Matan Azrad 2020-01-15 10:09 ` Raslan Darawsheh
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=9b58271e-994d-d7d3-fd43-01a7f2105d23@intel.com \ --to=ferruh.yigit@intel.com \ --cc=dev@dpdk.org \ --cc=shahafs@mellanox.com \ --cc=stable@dpdk.org \ --cc=yskoh@mellanox.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
patches for DPDK stable branches This inbox may be cloned and mirrored by anyone: git clone --mirror https://inbox.dpdk.org/stable/0 stable/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 stable stable/ https://inbox.dpdk.org/stable \ stable@dpdk.org public-inbox-index stable Example config snippet for mirrors. Newsgroup available over NNTP: nntp://inbox.dpdk.org/inbox.dpdk.stable AGPL code for this site: git clone https://public-inbox.org/public-inbox.git