Jason Wang
2021-Jan-19 11:09 UTC
[PATCH linux-next v3 6/6] vdpa_sim_net: Add support for user supported devices
On 2021/1/15 ??2:27, Parav Pandit wrote:>>> With that mac, mtu as optional input fields provide the necessary flexibility >> for different stacks to take appropriate shape as they desire. >> >> >> Thanks for the clarification. I think we'd better document the above in the >> patch that introduces the mac setting from management API. > Yes. Will do. > Thanks.Adding Sean. Regarding to mac address setting. Do we plan to allow to modify mac address after the creation? It looks like Openstack wants this. Sean may share more information on this. Thanks
Parav Pandit
2021-Jan-20 03:21 UTC
[PATCH linux-next v3 6/6] vdpa_sim_net: Add support for user supported devices
> From: Jason Wang <jasowang at redhat.com> > Sent: Tuesday, January 19, 2021 4:39 PM > To: Parav Pandit <parav at nvidia.com>; Michael S. Tsirkin <mst at redhat.com> > Cc: virtualization at lists.linux-foundation.org; Eli Cohen <elic at nvidia.com>; > netdev at vger.kernel.org; Sean Mooney <smooney at redhat.com> > Subject: Re: [PATCH linux-next v3 6/6] vdpa_sim_net: Add support for user > supported devices > > > On 2021/1/15 ??2:27, Parav Pandit wrote: > >>> With that mac, mtu as optional input fields provide the necessary > >>> flexibility > >> for different stacks to take appropriate shape as they desire. > >> > >> > >> Thanks for the clarification. I think we'd better document the above > >> in the patch that introduces the mac setting from management API. > > Yes. Will do. > > Thanks. > > > Adding Sean. > > Regarding to mac address setting. Do we plan to allow to modify mac > address after the creation? It looks like Openstack wants this. >Mac address is exposed in the features so yes, it should be possible to modify it as part of features modify command. (in future). User needs to make sure that device is not attached to vhost or higher layer stack when device configuration layout is modified.> Sean may share more information on this. > > Thanks