All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
Cc: Alexandra Winter <wintera@linux.ibm.com>,
	virtio-comment@lists.oasis-open.org
Subject: Re: Fwd: [virtio-comment] Re: [virtio-dev] Re: [virtio-comment] [PATCH v3 1/1] virtio-ism: introduce new device virtio-ism
Date: Fri, 7 Apr 2023 07:16:01 -0400	[thread overview]
Message-ID: <20230407071407-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <1680837925.8940198-3-xuanzhuo@linux.alibaba.com>

On Fri, Apr 07, 2023 at 11:25:25AM +0800, Xuan Zhuo wrote:
> On Wed, 5 Apr 2023 15:06:38 +0200, Alexandra Winter <wintera@linux.ibm.com> wrote:
> > Forwarding to virtio-comment@lists.oasis-open.org for completeness
> > as I cannot post to virtio-dev@lists.oasis-open.org
> >
> >
> > -------- Forwarded Message --------
> > Subject: [virtio-comment] Re: [virtio-dev] Re: [virtio-comment] [PATCH v3 1/1] virtio-ism: introduce new device virtio-ism
> > Date: Wed, 5 Apr 2023 15:02:57 +0200
> > From: Alexandra Winter <wintera@linux.ibm.com>
> > To: Michael S. Tsirkin <mst@redhat.com>
> > CC: Wen Gu <guwen@linux.alibaba.com>, Halil Pasic <pasic@linux.ibm.com>, Xuan Zhuo <xuanzhuo@linux.alibaba.com>, virtio-comment@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, cohuck@redhat.com, jasowang@redhat.com, Jan Kiszka <jan.kiszka@siemens.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, virtio-dev@lists.oasis-open.org
> >
> >
> >
> > On 05.04.23 14:52, Michael S. Tsirkin wrote:
> > > On Wed, Apr 05, 2023 at 02:39:53PM +0200, Alexandra Winter wrote:
> > >>
> > >>
> > >> On 24.03.23 05:03, Wen Gu wrote:
> > >>>
> > >>>
> > >>> On 2023/3/23 22:46, Halil Pasic wrote:
> > >>>
> > >>>> On Thu,  9 Feb 2023 11:30:56 +0800
> > >>>> Xuan Zhuo <xuanzhuo@linux.alibaba.com> wrote:
> > >>>>
> > >>
> > >> ...
> > >>
> > >>>
> > >>>> To get back to the things proposed here: the cdid is IMHO
> > >>>> a nice thing, and is functionally corresponding to the
> > >>>> (S)EID. But it is 16 byte wide, and I have no idea how
> > >>>> is it supposed to be used in the CLC handshake.
> > >>>>
> > >>>
> > >>> CLC handshake carry one SEID for all the SMC-D device. Considering
> > >>> coexistence with ISM, I am not sure whether we can change or increase
> > >>> the SEID.. cc Alexandra
> > >>>
> > >>> Thanks!
> > >>> Wen Gu
> > >>
> > >> As mentioned by others, discussions are ongoing.
> > >> It would be great, if we can agree on a way to use the existing CLC handshake
> > >> for SMC-D via virtio-ism and ism-loopback.
> > >> In that case SEID needs to be unique per hardware instance, cannot be increased and
> > >> can only be changed for x86 in a non-colliding way.
> > >>
> > >> An alternative would be to define new a SMC-D(?) protocol variant/version, where we
> > >> are free to define new fields (e.g. UUIDs).
> > >>
> > >> Alexandra
> > >
> > > Problem with tying to hardware is that it is blocking
> > > migration (which is a challenge with ism anyway, but still).
> >
> > Very true. My assumption is that _Internal_ Shared Memory
> > is limited to one instance of HW (as it is today).
> > So the 2 partners need to be on the same instance of HW.
> > So after migration of 1 partner, they should be able to detect
> > that internal SMC-D is no longer possible and switch to
> > SMC-R (remote) for the next connection.
> > (I have no idea how a connection could survive migration)
> 
> 
> I agree.
> 
> In terms of structure, there is no way to support migration. We can indeed
> provide some means to perceive the occurrence of migration to the upper
> users. Finally, the user (such as SMC) decides how to do it by itself.
> 
> Thanks.

I am not asking that this support be immediately included now.
We were discussing IDs, and experience shows it is a good idea
to support software-generated ones. Migration is one use-case
there could be others.


-- 
MST


This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/


  reply	other threads:[~2023-04-07 11:16 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-09  3:30 [PATCH v3 0/1] introduce virtio-ism: internal shared memory device Xuan Zhuo
2023-02-09  3:30 ` [PATCH v3 1/1] virtio-ism: introduce new device virtio-ism Xuan Zhuo
2023-02-09  3:35   ` [virtio-comment] " Parav Pandit
2023-02-09  3:36     ` Xuan Zhuo
2023-03-07 11:15   ` [virtio-comment] " Xuan Zhuo
2023-03-07 11:15     ` [virtio-dev] " Xuan Zhuo
2023-03-15 11:15     ` Xuan Zhuo
2023-03-15 11:15       ` [virtio-comment] " Xuan Zhuo
2023-03-23 14:46   ` [virtio-dev] Re: [virtio-comment] " Halil Pasic
2023-03-23 14:46     ` Halil Pasic
2023-03-24  3:08     ` [virtio-dev] " Xuan Zhuo
2023-03-24  3:08       ` Xuan Zhuo
2023-03-24  4:03     ` [virtio-dev] " Wen Gu
2023-04-05 12:39       ` [virtio-comment] " Alexandra Winter
2023-04-05 12:52         ` Michael S. Tsirkin
2023-04-05 12:52           ` [virtio-comment] " Michael S. Tsirkin
2023-04-05 13:02           ` Alexandra Winter
2023-04-05 13:06             ` Fwd: " Alexandra Winter
2023-04-07  3:25               ` Xuan Zhuo
2023-04-07 11:16                 ` Michael S. Tsirkin [this message]
2023-04-07  3:22           ` Xuan Zhuo
2023-04-07  3:22             ` [virtio-comment] " Xuan Zhuo
2023-04-07 11:13             ` [virtio-dev] " Michael S. Tsirkin
2023-04-07 11:13               ` Michael S. Tsirkin
2023-04-10  1:47               ` [virtio-dev] " Xuan Zhuo
2023-04-10  1:47                 ` Xuan Zhuo
2023-04-10  1:23             ` [virtio-dev] " Jason Wang
2023-04-10  1:23               ` Jason Wang
2023-04-10  1:53               ` [virtio-dev] " Xuan Zhuo
2023-04-10  1:53                 ` Xuan Zhuo
2023-04-10  2:04                 ` [virtio-dev] " Jason Wang
2023-04-10  2:04                   ` Jason Wang
2023-03-24  4:51   ` Parav Pandit
2023-03-24  4:51     ` Parav Pandit
2023-03-24  6:35     ` [virtio-dev] " Xuan Zhuo
2023-03-24  6:35       ` Xuan Zhuo
2023-03-24  9:10     ` [virtio-dev] " Michael S. Tsirkin
2023-03-24  9:10       ` Michael S. Tsirkin
2023-04-26  7:41   ` [virtio-dev] " Xuan Zhuo
2023-04-26  7:41     ` [virtio-comment] " Xuan Zhuo

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=20230407071407-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=virtio-comment@lists.oasis-open.org \
    --cc=wintera@linux.ibm.com \
    --cc=xuanzhuo@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.