Hi! I note that chaining INDIRECT descriptors with NEXT currently is broken in lguest, because current ring index gets overwritten. It does not matter with current virtio in guest, but I think it's worth fixing: for example for when we have trouble locating a request in a physically contigious region. Makes sense? -- MST
On Mon, 5 Oct 2009 01:07:34 am Michael S. Tsirkin wrote:> Hi! > I note that chaining INDIRECT descriptors with NEXT > currently is broken in lguest, because current > ring index gets overwritten.I agree this should be fixed, but not quite sure what you're referring to. I could force indirect and reproduce it, but I figure asking you for details would be more efficient :) Thanks! Rusty.
Possibly Parallel Threads
- INDIRECT and NEXT
- [PATCH net-next v2 3/5] virtio_ring: add packed ring support
- [PATCH net-next v2 3/5] virtio_ring: add packed ring support
- [PATCH] virtio-ring: Use threshold for switching to indirect descriptors
- [PATCH] virtio-ring: Use threshold for switching to indirect descriptors