tree 82f95b267a611b09914fa0f8b028715e2a2f1aaf
parent ee912ccfa007351a62ba42bd60499769f6c02c1e
author Michael S. Tsirkin <mst@redhat.com> 1297787275 +0200
committer Aurelien Jarno <aurelien@aurel32.net> 1298211506 +0100

e1000: verify we have buffers, upfront

The spec says: Any descriptor with a non-zero status byte has been
processed by the hardware, and is ready to be handled by the software.

Thus, once we change a descriptor status to non-zero we should
never move the head backwards and try to reuse this
descriptor from hardware.

This actually happened with a multibuffer packet
that arrives when we don't have enough buffers.

Fix by checking that we have enough buffers upfront
so we never need to discard the packet midway through.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Alex Williamson <alex.williamson@redhat.com>
Acked-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>