search for: virtq_desc_f_used

...ring_packed.desc[last_used].flags); > > + avail = flags & VRING_DESC_F_AVAIL(1); > > + used = flags & VRING_DESC_F_USED(1); > > + > > + return avail == used; > > +} > > This looks interesting, spec said: > > " > Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an > available descriptor and > equal for a used descriptor. > Note that this observation is mostly useful for sanity-checking as these are > necessary but not sufficient > conditions - for example, all descriptors are zero-initialized. To detect > used and...

...ring_packed.desc[last_used].flags); > > + avail = flags & VRING_DESC_F_AVAIL(1); > > + used = flags & VRING_DESC_F_USED(1); > > + > > + return avail == used; > > +} > > This looks interesting, spec said: > > " > Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an > available descriptor and > equal for a used descriptor. > Note that this observation is mostly useful for sanity-checking as these are > necessary but not sufficient > conditions - for example, all descriptors are zero-initialized. To detect > used and...

...DESC_F_AVAIL(1); > > > > + used = flags & VRING_DESC_F_USED(1); > > > > + > > > > + return avail == used; > > > > +} > > > This looks interesting, spec said: > > > > > > " > > > Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an > > > available descriptor and > > > equal for a used descriptor. > > > Note that this observation is mostly useful for sanity-checking as these are > > > necessary but not sufficient > > > conditions - for example, all descript...

...DESC_F_AVAIL(1); > > > > + used = flags & VRING_DESC_F_USED(1); > > > > + > > > > + return avail == used; > > > > +} > > > This looks interesting, spec said: > > > > > > " > > > Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an > > > available descriptor and > > > equal for a used descriptor. > > > Note that this observation is mostly useful for sanity-checking as these are > > > necessary but not sufficient > > > conditions - for example, all descript...

...ed].flags); >>> + avail = flags & VRING_DESC_F_AVAIL(1); >>> + used = flags & VRING_DESC_F_USED(1); >>> + >>> + return avail == used; >>> +} >> This looks interesting, spec said: >> >> " >> Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an >> available descriptor and >> equal for a used descriptor. >> Note that this observation is mostly useful for sanity-checking as these are >> necessary but not sufficient >> conditions - for example, all descriptors are zero-initialized. To d...

...virtio16_to_cpu(vq->vq.vdev, > + vq->vring_packed.desc[last_used].flags); > + avail = flags & VRING_DESC_F_AVAIL(1); > + used = flags & VRING_DESC_F_USED(1); > + > + return avail == used; > +} This looks interesting, spec said: " Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an available descriptor and equal for a used descriptor. Note that this observation is mostly useful for sanity-checking as these are necessary but not sufficient conditions - for example, all descriptors are zero-initialized. To detect used and available descriptors it is...

...; + used = flags & VRING_DESC_F_USED(1); > > > > > + > > > > > + return avail == used; > > > > > +} > > > > This looks interesting, spec said: > > > > > > > > " > > > > Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an > > > > available descriptor and > > > > equal for a used descriptor. > > > > Note that this observation is mostly useful for sanity-checking as these are > > > > necessary but not sufficient > > > > conditions -...

..._USED(1); > > > > > > + > > > > > > + return avail == used; > > > > > > +} > > > > > This looks interesting, spec said: > > > > > > > > > > " > > > > > Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an > > > > > available descriptor and > > > > > equal for a used descriptor. > > > > > Note that this observation is mostly useful for sanity-checking as these are > > > > > necessary but not sufficient > > &gt...

..._USED(1); > > > > > > + > > > > > > + return avail == used; > > > > > > +} > > > > > This looks interesting, spec said: > > > > > > > > > > " > > > > > Thus VIRTQ_DESC_F_AVAIL and VIRTQ_DESC_F_USED bits are different for an > > > > > available descriptor and > > > > > equal for a used descriptor. > > > > > Note that this observation is mostly useful for sanity-checking as these are > > > > > necessary but not sufficient > > &gt...

Hi, I have a concern about the portability of offloading the new VIRTIO packed ring format to hardware. According to the PCIe rev 2.0, paragraph 2.4.2. Update Ordering and Granularity Observed by a Read Transaction" " if a host CPU writes a QWORD to host memory, a Requester reading that QWORD from host memory may observe a portion of the QWORD updated and another portion of it

Hello everyone, This RFC implements packed ring support for virtio driver. The code was tested with DPDK vhost (testpmd/vhost-PMD) implemented by Jens at http://dpdk.org/ml/archives/dev/2018-January/089417.html Minor changes are needed for the vhost code, e.g. to kick the guest. TODO: - Refinements and bug fixes; - Split into small patches; - Test indirect descriptor support; - Test/fix event

Hello everyone, This RFC implements packed ring support for virtio driver. The code was tested with DPDK vhost (testpmd/vhost-PMD) implemented by Jens at http://dpdk.org/ml/archives/dev/2018-January/089417.html Minor changes are needed for the vhost code, e.g. to kick the guest. TODO: - Refinements and bug fixes; - Split into small patches; - Test indirect descriptor support; - Test/fix event