> >   struct snet_vq {
> >       /* VQ callback */
> >       struct vdpa_callback cb;
> > +     /* VQ initial state */
> > +     struct vdpa_vq_state inital_state;
> 
> 
> I guess you meant "initial_state" really here.

Yes, you're right, I'll fix it.

> Btw, who is used to initialize initial_state (especially the case for
> packed ring)?
> 
> In snet_set_vq_state(), only the value that is set by bus is checked but
> not the inital_state.

initial_state is only used to store the vq state received in snet_set_vq_state 
from the bus, and later on pass is to the DPU in snet_write_conf.

The name initial is used because this is the state we initialize the vqs in the 
DPU.

Thanks
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Reply via email to