Answers inline.
Internal Use - Confidential > -----Original Message----- > From: Maxime Coquelin <maxime.coque...@redhat.com> > Sent: Monday, August 25, 2025 1:49 AM > To: Bathija, Pravin <pravin.bath...@dell.com>; Bruce Richardson > <bruce.richard...@intel.com>; David Marchand > <david.march...@redhat.com>; Chenbo Xia <chen...@nvidia.com> > Cc: dev@dpdk.org; pravin.m.bathija....@gmail.com; Thomas Monjalon > <tho...@monjalon.net> > Subject: Re: [PATCH 0/3] vhost_user: configure memory slots > > > [EXTERNAL EMAIL] > > Hi, > > Please try to fix your email client so that we can follow the replies inlined. Thank you for the input. I have made the necessary adjustments. Please let me know if there is anything missing. > > On 8/22/25 9:33 AM, Bathija, Pravin wrote: > > Dear Maxime, Bruce, David, Chenbo, > > > > I would like to follow up on the status of the patches and when we should > expect these to be merged into the mainline. > > > > Please let me know if I can help answer any questions you might have. > > > > Regards, > > Pravin > > > > > > Internal Use - Confidential > > -----Original Message----- > > From: Bathija, Pravin > > Sent: Thursday, August 21, 2025 7:48 PM > > To: Thomas Monjalon <tho...@monjalon.net> > > Cc: dev@dpdk.org; pravin.m.bathija....@gmail.com; Maxime Coquelin > > <maxime.coque...@redhat.com>; Chenbo Xia <chen...@nvidia.com>; David > > Marchand <david.march...@redhat.com>; Bruce Richardson > > <bruce.richard...@intel.com> > > Subject: RE: [PATCH 0/3] vhost_user: configure memory slots > > > > Comments inline: > > > > -----Original Message----- > > From: Thomas Monjalon <tho...@monjalon.net> > > Sent: Tuesday, August 19, 2025 4:36 AM > > To: Bathija, Pravin <pravin.bath...@dell.com> > > Cc: dev@dpdk.org; pravin.m.bathija....@gmail.com; Maxime Coquelin > > <maxime.coque...@redhat.com>; Chenbo Xia <chen...@nvidia.com>; David > > Marchand <david.march...@redhat.com>; Bruce Richardson > > <bruce.richard...@intel.com> > > Subject: Re: [PATCH 0/3] vhost_user: configure memory slots > > > > > > [EXTERNAL EMAIL] > > > > Comments inline > > > > 12/08/2025 04:33, Pravin M Bathija: > >> - add protocol feature support add and removal of memory > >> regions/slots dynamically. > >> - this patch set has been tested with multiple instances of fio > > > > What is "fio"? > > Fio is an open-source tool to test I/O workloads on disks and file systems. > > > >> connected via libblkio to spdk target backed unix domain socket. > >> - please apply this this patch-set to dpdk mainline as well as > >> version 25.03.0. > > > > As mentionned previously, we don't plan to maintain 25.03. > > I understand you need it for SPDK, > > but I think SPDK should rely only on LTS branches. > > Is there a specific reason why 25.03 is required in SPDK? > > > > When we do sub-module init on spdk master, it uses dpdk version 25.03 from > it's own sub-repo. > > Our changes for vhost have already been pushed to spdk master. We have > > also pushed the same dpdk changes in the spdk sub-repo "dpdk" which is > > version 25.03. I was just ensuring that there is uniformity in the 25.03 > codebase, whether it is on the spdk sub-repo or 25.03 within dpdk repo. > > Ok, still, it would be better for SPDK to uses LTS as we have no plans to add > extra stable branches. > > Also, your series may be considered as a feature and not a bug fix, so even if > 25.03 would be a stable branch (which it is not), it would not be backported > against it. Understood. I would request that we merge my changes into DPDK mainline before September 18 when the merge window closes for SPDK v25.09. > > > > > > >> v2: > >> - fixed warnings with e-mail address > >> - fixed warnings with typos > >> - there is a warning about __rte_packed_begin not being in pair with > >> __rte_packed_end which on reading the code turns out to be false. It > >> is an anomaly of how git-format generates the patch and how > >> check-patch checks it. > > > > When sending a v2, please use -v2 so it is reflected in the email subject. > > I will remember this for next time. Thank you for pointing that out. > > > >> Pravin M Bathija (3): > >> mailmap: add user > >> vhost_user: configure memory slots > >> vhost_user: support for memory regions > >> > >> .mailmap | 1 + > >> lib/vhost/rte_vhost.h | 4 + > >> lib/vhost/vhost_user.c | 325 > >> +++++++++++++++++++++++++++++++++++------ > >> lib/vhost/vhost_user.h | 10 ++ > >> 4 files changed, 296 insertions(+), 44 deletions(-) > > > > > >
