On 11/4/21 11:19, Jiayu Hu wrote:
For the packet using multiple descriptors, the flag of first descriptor cannot be updated until finishing updating the flag of remaining descriptors. However, if the packet's descriptor index is wrapped, the first descriptor's flag is updated earlier than the others.
The commit message is ambiguous IMHO, it is not clear whether it describes the bug or the fix. I understand it describes the problem, as the first descriptor flags always have to be updated last, but it please reword it to make it clear what iu the issue, and how this patch fixes it.
Fixes: 873e8dad6f49 ("vhost: support packed ring in async datapath") Cc: sta...@dpdk.org
If you think it is worth backporting it to stable, please take care of preparing the backport as I suspect it will not apply cleanly.
Signed-off-by: Jiayu Hu <jiayu...@intel.com> --- lib/vhost/virtio_net.c | 122 ++++++++++++++++++----------------------- 1 file changed, 54 insertions(+), 68 deletions(-)