> -----Original Message----- > From: David Marchand <david.march...@redhat.com> > Sent: Wednesday, June 29, 2022 9:45 PM > To: Xia, Chenbo <chenbo....@intel.com> > Cc: dev@dpdk.org; maxime.coque...@redhat.com; Wang, YuanX > <yuanx.w...@intel.com>; Ding, Xuan <xuan.d...@intel.com>; > ferruh.yi...@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....@intel.com> wrote: > > > > Hi David, > > > > > -----Original Message----- > > > From: David Marchand <david.march...@redhat.com> > > > Sent: Monday, June 27, 2022 5:27 PM > > > To: dev@dpdk.org > > > Cc: maxime.coque...@redhat.com; Xia, Chenbo <chenbo....@intel.com>; > Wang, > > > YuanX <yuanx.w...@intel.com>; Ding, Xuan <xuan.d...@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....@intel.com> > > Thanks Chenbo! > > > -- > David Marchand