[dpdk-dev] [PATCH] lib/librte_vhost: fix bugs
Wodkowski, PawelX
pawelx.wodkowski at intel.com
Tue Apr 10 09:47:56 CEST 2018
> -----Original Message-----
> From: Maxime Coquelin [mailto:maxime.coquelin at redhat.com]
> Sent: Tuesday, April 10, 2018 9:44 AM
> To: Wodkowski, PawelX <pawelx.wodkowski at intel.com>; Zhang, Roy Fan
> <roy.fan.zhang at intel.com>; dev at dpdk.org
> Cc: jianjay.zhou at huawei.com; Tan, Jianfeng <jianfeng.tan at intel.com>
> Subject: Re: [PATCH] lib/librte_vhost: fix bugs
>
> Hi Pawel,
>
> On 04/10/2018 09:32 AM, Wodkowski, PawelX wrote:
> >> # all source are stored in SRCS-y
> >> SRCS-$(CONFIG_RTE_LIBRTE_VHOST) := fd_man.c iotlb.c socket.c vhost.c
> \
> >> diff --git a/lib/librte_vhost/vhost_crypto.c
> b/lib/librte_vhost/vhost_crypto.c
> >> index d84513e7f..2acfc908c 100644
> >> --- a/lib/librte_vhost/vhost_crypto.c
> >> +++ b/lib/librte_vhost/vhost_crypto.c
> >> @@ -381,7 +381,7 @@ vhost_crypto_create_sess(struct vhost_crypto
> >> *vcrypto,
> >> return;
> >> }
> >>
> >> - VC_LOG_DBG("Session (key %lu, session %p) created.",
> >> + VC_LOG_DBG("Session (key %llu, session %p) created.",
> >
> > Why 'llu' instead of PRIu64 (here and there)?
>
> You are right, but Fan actually fixed it in updated version (the initial
> patch has been split in 3):
> [PATCH] vhost/crypto: fix session id printf
>
Ohh, sorry. I need more coffee before starting review :)
> Cheers,
> Maxime
> > Pawel
> >
Pawel
More information about the dev
mailing list