[virtio-dev] Re: [virtio-comment] Re: [virtio-dev] Re: [virtio-comment] [PATCH v3 1/1] virtio-ism: introduce new device virtio-ism

From: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Wen Gu <guwen@linux.alibaba.com>,
	Halil Pasic <pasic@linux.ibm.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,
	Alexandra Winter <wintera@linux.ibm.com>
Subject: [virtio-dev] Re: [virtio-comment] Re: [virtio-dev] Re: [virtio-comment] [PATCH v3 1/1] virtio-ism: introduce new device virtio-ism
Date: Mon, 10 Apr 2023 09:47:45 +0800	[thread overview]
Message-ID: <1681091265.2051864-1-xuanzhuo@linux.alibaba.com> (raw)
In-Reply-To: <20230407070820-mutt-send-email-mst@kernel.org>

On Fri, 7 Apr 2023 07:13:13 -0400, "Michael S. Tsirkin" <mst@redhat.com> wrote:
> On Fri, Apr 07, 2023 at 11:22:15AM +0800, Xuan Zhuo wrote:
> > On Wed, 5 Apr 2023 08:52:14 -0400, "Michael S. Tsirkin" <mst@redhat.com> 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).
> >
> >
> > We don't want to support migration. At least we don't want to support it for the
> > time being. Because there are indeed many problems. I think Migration is not
> > necessary for a new Virtio device.
> >
> > Thanks.
>
> The specific implementation does not matter much. At the spec level we
> strive to make interfaces generic so they can be reused down the road,
> rather than having to invent new ones for each use-case.
> Maybe we can come up with a way that let devices choose either
> an existing one with a SEID or a new one with a UUID?

Yes. I agree this.

From the perspective of Virtio SPEC, we can provide some information for SMC. As
long as these requirements are reasonable. At present, @Wen is working in this.

Thanks.

>
>
> --
> MST
>

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