From: Jason Wang <jasowang@redhat.com>
To: Randy Dunlap <rdunlap@infradead.org>,
mst@redhat.com, linux-kernel@vger.kernel.org,
kvm@vger.kernel.org, virtualization@lists.linux-foundation.org,
netdev@vger.kernel.org
Cc: tiwei.bie@intel.com, jgg@mellanox.com,
maxime.coquelin@redhat.com, cunming.liang@intel.com,
zhihong.wang@intel.com, rob.miller@broadcom.com,
xiao.w.wang@intel.com, haotian.wang@sifive.com,
lingshan.zhu@intel.com, eperezma@redhat.com, lulu@redhat.com,
parav@mellanox.com, kevin.tian@intel.com, stefanha@redhat.com,
hch@infradead.org, aadam@redhat.com, jiri@mellanox.com,
shahafs@mellanox.com, hanand@xilinx.com, mhabets@solarflare.com
Subject: Re: [PATCH V3 1/5] vhost: factor out IOTLB
Date: Thu, 20 Feb 2020 13:39:39 +0800 [thread overview]
Message-ID: <b348cb04-3b5b-1a11-a4c5-4b05d31e88c0@redhat.com> (raw)
In-Reply-To: <61d64892-ce77-3e86-acb8-a49679fc0047@infradead.org>
On 2020/2/20 下午12:04, Randy Dunlap wrote:
> On 2/19/20 7:56 PM, Jason Wang wrote:
>> This patch factors out IOTLB into a dedicated module in order to be
>> reused by other modules like vringh. User may choose to enable the
>> automatic retiring by specifying VHOST_IOTLB_FLAG_RETIRE flag to fit
>> for the case of vhost device IOTLB implementation.
>>
>> Signed-off-by: Jason Wang <jasowang@redhat.com>
>> ---
>> MAINTAINERS | 1 +
>> drivers/vhost/Kconfig | 7 ++
>> drivers/vhost/Makefile | 2 +
>> drivers/vhost/net.c | 2 +-
>> drivers/vhost/vhost.c | 221 +++++++++++-------------------------
>> drivers/vhost/vhost.h | 36 ++----
>> drivers/vhost/vhost_iotlb.c | 171 ++++++++++++++++++++++++++++
>> include/linux/vhost_iotlb.h | 45 ++++++++
>> 8 files changed, 304 insertions(+), 181 deletions(-)
>> create mode 100644 drivers/vhost/vhost_iotlb.c
>> create mode 100644 include/linux/vhost_iotlb.h
>>
> Hi,
> Sorry if you have gone over this previously:
Thanks for the review, it's really helpful.
>
>> diff --git a/drivers/vhost/Kconfig b/drivers/vhost/Kconfig
>> index 3d03ccbd1adc..eef634ff9a6e 100644
>> --- a/drivers/vhost/Kconfig
>> +++ b/drivers/vhost/Kconfig
>> @@ -36,6 +36,7 @@ config VHOST_VSOCK
>>
>> config VHOST
>> tristate
>> + select VHOST_IOTLB
>> ---help---
>> This option is selected by any driver which needs to access
>> the core of vhost.
>> @@ -54,3 +55,9 @@ config VHOST_CROSS_ENDIAN_LEGACY
>> adds some overhead, it is disabled by default.
>>
>> If unsure, say "N".
>> +
>> +config VHOST_IOTLB
>> + tristate
>> + default m
> "default m" should not be needed. Just make whatever needs it select it.
Yes, will fix.
Thanks
>
>> + help
>> + Generic IOTLB implementation for vhost and vringh.
>
next prev parent reply other threads:[~2020-02-20 5:40 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-20 3:56 [PATCH V3 0/5] vDPA support Jason Wang
2020-02-20 3:56 ` [PATCH V3 1/5] vhost: factor out IOTLB Jason Wang
2020-02-20 4:04 ` Randy Dunlap
2020-02-20 5:39 ` Jason Wang [this message]
2020-02-20 3:56 ` [PATCH V3 2/5] vringh: IOTLB support Jason Wang
2020-02-20 3:56 ` [PATCH V3 3/5] vDPA: introduce vDPA bus Jason Wang
2020-02-20 4:06 ` Randy Dunlap
2020-02-20 5:40 ` Jason Wang
2020-02-20 3:56 ` [PATCH V3 4/5] virtio: introduce a vDPA based transport Jason Wang
2020-02-20 4:07 ` Randy Dunlap
2020-02-20 5:40 ` Jason Wang
2020-02-20 3:56 ` [PATCH V3 5/5] vdpasim: vDPA device simulator Jason Wang
2020-02-20 4:09 ` Randy Dunlap
2020-02-20 5:41 ` Jason Wang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b348cb04-3b5b-1a11-a4c5-4b05d31e88c0@redhat.com \
--to=jasowang@redhat.com \
--cc=aadam@redhat.com \
--cc=cunming.liang@intel.com \
--cc=eperezma@redhat.com \
--cc=hanand@xilinx.com \
--cc=haotian.wang@sifive.com \
--cc=hch@infradead.org \
--cc=jgg@mellanox.com \
--cc=jiri@mellanox.com \
--cc=kevin.tian@intel.com \
--cc=kvm@vger.kernel.org \
--cc=lingshan.zhu@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lulu@redhat.com \
--cc=maxime.coquelin@redhat.com \
--cc=mhabets@solarflare.com \
--cc=mst@redhat.com \
--cc=netdev@vger.kernel.org \
--cc=parav@mellanox.com \
--cc=rdunlap@infradead.org \
--cc=rob.miller@broadcom.com \
--cc=shahafs@mellanox.com \
--cc=stefanha@redhat.com \
--cc=tiwei.bie@intel.com \
--cc=virtualization@lists.linux-foundation.org \
--cc=xiao.w.wang@intel.com \
--cc=zhihong.wang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).