Hi Maxime, >-----Original Message----- >From: Maxime Coquelin <maxime.coque...@redhat.com> >Sent: Friday, October 29, 2021 6:36 PM >To: Ding, Xuan <xuan.d...@intel.com>; dev@dpdk.org; Xia, Chenbo ><chenbo....@intel.com> >Cc: Hu, Jiayu <jiayu...@intel.com>; Burakov, Anatoly ><anatoly.bura...@intel.com> >Subject: Re: [PATCH v2] vhost: remove async dma map status > > > >On 10/27/21 12:00, Xuan Ding wrote: >> Async dma map status flag was added to prevent the unnecessary unmap >> when DMA devices bound to kernel driver. This brings maintenance cost >> for a lot of code. This patch removes the dma map status by using >> rte_errno instead. >> >> This patch relies on the following patch to fix a partial unmap check >> in vfio unmapping API. >> [1] https://www.mail-archive.com/dev@dpdk.org/msg226464.html >> >> Cc: anatoly.bura...@intel.com >> >> Signed-off-by: Xuan Ding <xuan.d...@intel.com> >> --- >> v2: >> * Fix a typo in commit log. >> --- >> lib/vhost/vhost.h | 3 -- >> lib/vhost/vhost_user.c | 70 ++++++++---------------------------------- >> 2 files changed, 13 insertions(+), 60 deletions(-) >> > > >Applied to dpdk-next-virtio/main with title fixed. >Please run check-git-log script next time.
Thanks for your fix. I ran the script first but got no warning... I will be more careful to check the format next time. BTW, should the CC be removed in the commit log? Regards, Xuan > >Thanks, >Maxime
