[PATCH 2/4] vhost: restore device information in log messages

[Xia, Chenbo]

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Wednesday, June 29, 2022 9:45 PM
> To: Xia, Chenbo <chenbo.xia at intel.com>
> Cc: dev at dpdk.org; maxime.coquelin at redhat.com; Wang, YuanX
> <yuanx.wang at intel.com>; Ding, Xuan <xuan.ding at intel.com>;
> ferruh.yigit at xilinx.com
> Subject: Re: [PATCH 2/4] vhost: restore device information in log messages
> 
> On Wed, Jun 29, 2022 at 3:34 PM Xia, Chenbo <chenbo.xia at intel.com> wrote:
> >
> > Hi David,
> >
> > > -----Original Message-----
> > > From: David Marchand <david.marchand at redhat.com>
> > > Sent: Monday, June 27, 2022 5:27 PM
> > > To: dev at dpdk.org
> > > Cc: maxime.coquelin at redhat.com; Xia, Chenbo <chenbo.xia at intel.com>;
> Wang,
> > > YuanX <yuanx.wang at intel.com>; Ding, Xuan <xuan.ding at intel.com>
> > > Subject: [PATCH 2/4] vhost: restore device information in log messages
> > >
> > > device information in the log messages was dropped.
> > >
> > > Fixes: 52ade97e3641 ("vhost: fix physical address mapping")
> >
> > Should we add cc-stable tag in this case as it's 22.03 commit?
> > Last time when I asked Ferruh, he said although now we don't maintain
> > non-LTS release, we should still add this tag. Not sure this changed
> > or not.
> 
> No objection for leaks and other important fixes.

Good to know we are aligned :)

> Here, this patch is only fixing some log messages, so I did not see the
> need.

OK. Make sense to me

Thanks,
Chenbo

> 
> In any case, I am fine with both options and this can be added when
> the patch is applied by sub-tree maintainer.
> 
> >
> > For the patch:
> >
> > Reviewed-by: Chenbo Xia <chenbo.xia at intel.com>
> 
> Thanks Chenbo!
> 
> 
> --
> David Marchand