[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-29 Thread Jason Wang
On Wed, Jun 29, 2022 at 5:02 PM Michael S. Tsirkin wrote: > > On Wed, Jun 29, 2022 at 04:43:18PM +0800, Jason Wang wrote: > > On Tue, Jun 28, 2022 at 10:24 PM Michael S. Tsirkin wrote: > > > > > > On Thu, Jun 23, 2022 at 11:34:42AM +0800, Jason Wang wrote: > > > > On Thu, Jun 23, 2022 at 10:57

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-29 Thread Michael S. Tsirkin
On Wed, Jun 29, 2022 at 04:43:18PM +0800, Jason Wang wrote: > On Tue, Jun 28, 2022 at 10:24 PM Michael S. Tsirkin wrote: > > > > On Thu, Jun 23, 2022 at 11:34:42AM +0800, Jason Wang wrote: > > > On Thu, Jun 23, 2022 at 10:57 AM Parav Pandit wrote: > > > > > > > > > > > > > From: Jason Wang > >

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-29 Thread Jason Wang
On Tue, Jun 28, 2022 at 10:24 PM Michael S. Tsirkin wrote: > > On Thu, Jun 23, 2022 at 11:34:42AM +0800, Jason Wang wrote: > > On Thu, Jun 23, 2022 at 10:57 AM Parav Pandit wrote: > > > > > > > > > > From: Jason Wang > > > > Sent: Wednesday, June 22, 2022 10:42 PM > > > > > > [..] > > > > > Are

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-28 Thread Michael S. Tsirkin
On Thu, Jun 23, 2022 at 11:34:42AM +0800, Jason Wang wrote: > On Thu, Jun 23, 2022 at 10:57 AM Parav Pandit wrote: > > > > > > > From: Jason Wang > > > Sent: Wednesday, June 22, 2022 10:42 PM > > > > [..] > > > > Are you suggesting only feature bits through register-based transport > > > > or

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-22 Thread Jason Wang
On Thu, Jun 23, 2022 at 10:57 AM Parav Pandit wrote: > > > > From: Jason Wang > > Sent: Wednesday, June 22, 2022 10:42 PM > > [..] > > > Are you suggesting only feature bits through register-based transport or > > > all > > commands via register-based transport? > > > > For feature bits, if we

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-22 Thread Jason Wang
On Thu, Jun 23, 2022 at 10:07 AM Parav Pandit wrote: > > > > > From: Jason Wang > > Sent: Wednesday, June 22, 2022 9:27 PM > > > > 1. proposed new command doesn't demand exposing registers in the PCI > > > memory mapped area 2. Today AQ is perceived in to be PF, but it doesn't > > have to be.

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-22 Thread Jason Wang
On Mon, Jun 20, 2022 at 7:06 PM Parav Pandit wrote: > > > > > From: Michael S. Tsirkin > > Sent: Monday, June 20, 2022 6:00 AM > > > > On Tue, May 31, 2022 at 08:39:24PM +, Parav Pandit wrote: > > > > And as I said, you will need much more spec work to reach the level > > > > to which

[virtio-dev] Re: [virtio-comment] Re: [PATCH v5 2/7] Introduce admin command set

2022-06-20 Thread Michael S. Tsirkin
On Wed, May 18, 2022 at 05:16:13PM +0300, Max Gurtovoy wrote: > > On 5/18/2022 4:50 PM, Cornelia Huck wrote: > > On Wed, May 18 2022, Max Gurtovoy wrote: > > > > > On 5/15/2022 6:23 PM, Michael S. Tsirkin wrote: > > > > On Wed, Apr 27, 2022 at 01:58:19AM +0300, Max Gurtovoy wrote: > > > > >

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-20 Thread Michael S. Tsirkin
On Mon, Jun 20, 2022 at 05:19:26PM +, Parav Pandit wrote: > > > > From: Michael S. Tsirkin > > Sent: Monday, June 20, 2022 1:04 PM > > > > Max with transport I am not talking about admin queue specifically. > > The point Parav made is that he wants to avoid using memory writable > >

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-20 Thread Michael S. Tsirkin
On Mon, Jun 20, 2022 at 07:54:00PM +0300, Max Gurtovoy wrote: > > On 6/20/2022 7:46 PM, Michael S. Tsirkin wrote: > > On Mon, Jun 20, 2022 at 11:06:07AM +, Parav Pandit wrote: > > > > > > > From: Michael S. Tsirkin > > > > Sent: Monday, June 20, 2022 6:00 AM > > > > > > > > On Tue, May 31,

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-20 Thread Michael S. Tsirkin
On Mon, Jun 20, 2022 at 11:06:07AM +, Parav Pandit wrote: > > > > From: Michael S. Tsirkin > > Sent: Monday, June 20, 2022 6:00 AM > > > > On Tue, May 31, 2022 at 08:39:24PM +, Parav Pandit wrote: > > > > And as I said, you will need much more spec work to reach the level > > > > to

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-20 Thread Michael S. Tsirkin
On Mon, Jun 20, 2022 at 05:24:02AM -0400, Michael S. Tsirkin wrote: > With 1.2 release nearing completion I went back to see what's going on > with this wotk, and it looks like I forgot to reply to this question. > Sorry. > > > On Tue, May 31, 2022 at 08:39:24PM +, Parav Pandit wrote: > > >

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-06-20 Thread Michael S. Tsirkin
With 1.2 release nearing completion I went back to see what's going on with this wotk, and it looks like I forgot to reply to this question. Sorry. On Tue, May 31, 2022 at 08:39:24PM +, Parav Pandit wrote: > > And as I said, you will need much more spec work to reach the level to which > >

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-05-18 Thread Cornelia Huck
On Wed, May 18 2022, Max Gurtovoy wrote: > On 5/17/2022 2:48 PM, Michael S. Tsirkin wrote: >> On Mon, May 16, 2022 at 09:08:34PM +, Parav Pandit wrote: >>> Hi Michael, >>> From: Michael S. Tsirkin Sent: Sunday, May 15, 2022 11:24 AM >>> [..] >>> > +\subsection{VIRTIO ADMIN

[virtio-dev] Re: [virtio-comment] Re: [PATCH v5 2/7] Introduce admin command set

2022-05-18 Thread Cornelia Huck
On Wed, May 18 2022, Max Gurtovoy wrote: > On 5/15/2022 6:23 PM, Michael S. Tsirkin wrote: >> On Wed, Apr 27, 2022 at 01:58:19AM +0300, Max Gurtovoy wrote: >>> This command set is used for essential administrative and management >>> operations. >>> >>> Admin commands should be submitted to a

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-05-17 Thread Michael S. Tsirkin
On Mon, May 16, 2022 at 09:08:34PM +, Parav Pandit wrote: > Hi Michael, > > > From: Michael S. Tsirkin > > Sent: Sunday, May 15, 2022 11:24 AM > > [..] > > > > +\subsection{VIRTIO ADMIN DEVICE CAPS ACCEPT > > command}\label{sec:Basic > > > +Facilities of a Virtio Device / Admin command set

[virtio-dev] Re: [virtio-comment] RE: [PATCH v5 2/7] Introduce admin command set

2022-05-17 Thread Cornelia Huck
On Mon, May 16 2022, Parav Pandit wrote: > Hi Michael, > >> From: Michael S. Tsirkin >> Sent: Sunday, May 15, 2022 11:24 AM > > [..] > >> > +\subsection{VIRTIO ADMIN DEVICE CAPS ACCEPT >> command}\label{sec:Basic >> > +Facilities of a Virtio Device / Admin command set / VIRTIO ADMIN >> >