Discussion of the implementations of VIRTIO specification
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
Cc: Halil Pasic <pasic@linux.ibm.com>,
	virtio-dev@lists.oasis-open.org, hans@linux.alibaba.com,
	herongguang@linux.alibaba.com, zmlcc@linux.alibaba.com,
	dust.li@linux.alibaba.com, tonylu@linux.alibaba.com,
	zhenzao@linux.alibaba.com, helinguo@linux.alibaba.com,
	gerry@linux.alibaba.com, Jan Kiszka <jan.kiszka@siemens.com>,
	wintera@linux.ibm.com, kgraul@linux.ibm.com,
	wenjia@linux.ibm.com, jaka@linux.ibm.com, hca@linux.ibm.com,
	twinkler@linux.ibm.com, raspl@linux.ibm.com,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Jason Wang <jasowang@redhat.com>
Subject: Re: [virtio-dev] [PATCH v2 1/1] virtio-ism: introduce new device virtio-ism
Date: Thu, 12 Jan 2023 15:30:58 +0100	[thread overview]
Message-ID: <87r0vzddul.fsf@redhat.com> (raw)
In-Reply-To: <1673524110.7417235-2-xuanzhuo@linux.alibaba.com>

On Thu, Jan 12 2023, Xuan Zhuo <xuanzhuo@linux.alibaba.com> wrote:

> On Thu, 12 Jan 2023 09:42:05 +0100, Cornelia Huck <cohuck@redhat.com> wrote:
>> On Thu, Jan 12 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
>>
>> > On Thu, Jan 12, 2023 at 10:01:25AM +0800, Jason Wang wrote:
>> >> On Wed, Jan 11, 2023 at 11:11 PM Halil Pasic <pasic@linux.ibm.com> wrote:
>> >> >
>> >> > On Wed, 11 Jan 2023 19:08:53 +0800
>> >> > Xuan Zhuo <xuanzhuo@linux.alibaba.com> wrote:
>> >> >
>> >> > > > > +ISM(Internal Shared Memory) device provides the ability to share memory between
>> >> > > > > +different VMs launched from the same entity.
>> >> > > >
>> >> > > > Launched by instead of from? Maybe introduce a catchy name for the
>> >> > > > "entity that launched the VMs" and prevent oversimplification by
>> >> > > > explaining any shortcomings of the name if any in one place. Host would
>> >> > > > be one candidate, VMM another.
>> >> > >
>> >> > > Cornelia Huck wrote:
>> >> > >
>> >> > >       Is there a way to avoid the term "host" (throughout this document)?
>> >> > >       IIUC, you need the uniqueness within the scope of the entity that
>> >> > >       launches the different instances that get shared access to the regions
>> >> > >       (which could conceivably a unit of hardware?)
>> >> > >
>> >> > > And I think she is right, so I am trying to remove the term HOST.
>> >> > >
>> >> > > Do you have better opinions? I think VMM is not particularly suitable.
>> >
>> > I think fundamentally from spec POV memory is shared between devices.
>> > How sharing is accomplished guest does not care so neither should the
>> > spec. Can some RDMA tricks be used for synchronisation behind the
>> > scenes? Maybe, the spec does not care. But we can give an example.
>> >
>> > So something like:
>> >
>> > 	An ISM(Internal Shared Memory) device provides the ability to
>> > 	access memory shared between multiple devices. This allows low-overhead
>> > 	communication in presence of such memory. For example, memory can be
>> > 	shared with guests of multiple virtual machines running on the same
>> > 	host, with each virtual machine including an ISM device and with
>> > 	the guests using the ISM devices to access the shared memory.
>> >
>> > what do others think?
>>
>> I like that: we don't want to talk about hosts/VMMs/etc. as we
>> fundamentally deal with devices and drivers, but sharing between guests
>> is of course the obvious use case.
>>
>> I'm just wondering how best to express the uniqueness scope, is it per
>> (ISM) device?
>
> No, each vm has at least one separate device. The devices in a host form
> an uniqueness scope.

Should we call it a 'group', then? A host would be an example of such a
group.


---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org


  reply	other threads:[~2023-01-12 14:31 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-23  8:13 [PATCH v2 0/1] introduce virtio-ism: internal shared memory device Xuan Zhuo
2022-12-23  8:13 ` [PATCH v2 1/1] virtio-ism: introduce new device virtio-ism Xuan Zhuo
2023-01-10 22:34   ` [virtio-dev] " Halil Pasic
2023-01-11 11:08     ` Xuan Zhuo
2023-01-11 15:11       ` Halil Pasic
2023-01-12  2:01         ` Jason Wang
2023-01-12  6:56           ` Michael S. Tsirkin
2023-01-12  8:42             ` Cornelia Huck
2023-01-12 11:48               ` Xuan Zhuo
2023-01-12 14:30                 ` Cornelia Huck [this message]
2023-01-12 15:41                   ` Halil Pasic
2023-01-12 16:07                     ` Cornelia Huck
2023-01-13  1:58                     ` Xuan Zhuo
2023-01-13  2:29                       ` Jason Wang
2023-01-13  6:24                         ` Xuan Zhuo
2023-01-13 12:00                           ` Michael S. Tsirkin
2023-01-16  2:10                             ` Xuan Zhuo
2023-01-19 12:30                               ` Alexandra Winter
2023-01-28  7:42                                 ` Xuan Zhuo
2023-01-12 11:47             ` Xuan Zhuo
2023-01-12 12:15             ` Halil Pasic
2023-01-11 15:22       ` Halil Pasic
2023-01-12 11:57         ` Xuan Zhuo
2023-01-11 15:30       ` Halil Pasic
2023-01-12 12:03         ` Xuan Zhuo
2023-01-11 20:46       ` Halil Pasic
2023-01-12 12:23         ` Xuan Zhuo
2023-01-11 21:12       ` Halil Pasic
2023-01-12  7:01         ` Michael S. Tsirkin
2023-01-12 12:31         ` Xuan Zhuo
2023-01-20 13:06           ` Halil Pasic
2023-01-12 12:40         ` Xuan Zhuo
2023-02-05 12:30         ` Michael S. Tsirkin
2023-02-06  2:15           ` Xuan Zhuo
2023-01-25 12:55 ` [PATCH v2 0/1] introduce virtio-ism: internal shared memory device Wenjia Zhang
2023-03-01  9:34   ` Tony Lu
2023-03-01  9:34     ` [virtio-dev] " Tony Lu

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=87r0vzddul.fsf@redhat.com \
    --to=cohuck@redhat.com \
    --cc=dust.li@linux.alibaba.com \
    --cc=gerry@linux.alibaba.com \
    --cc=hans@linux.alibaba.com \
    --cc=hca@linux.ibm.com \
    --cc=helinguo@linux.alibaba.com \
    --cc=herongguang@linux.alibaba.com \
    --cc=jaka@linux.ibm.com \
    --cc=jan.kiszka@siemens.com \
    --cc=jasowang@redhat.com \
    --cc=kgraul@linux.ibm.com \
    --cc=mst@redhat.com \
    --cc=pasic@linux.ibm.com \
    --cc=raspl@linux.ibm.com \
    --cc=tonylu@linux.alibaba.com \
    --cc=twinkler@linux.ibm.com \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=wenjia@linux.ibm.com \
    --cc=wintera@linux.ibm.com \
    --cc=xuanzhuo@linux.alibaba.com \
    --cc=zhenzao@linux.alibaba.com \
    --cc=zmlcc@linux.alibaba.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