mbox series

[net,v2,0/2] ibmvnic: Bug fixes for queue descriptor processing

Message ID 1606763244-28111-1-git-send-email-tlfalcon@linux.ibm.com
Headers show
Series ibmvnic: Bug fixes for queue descriptor processing | expand

Message

Thomas Falcon Nov. 30, 2020, 7:07 p.m. UTC
This series resolves a few issues in the ibmvnic driver's
RX buffer and TX completion processing. The first patch
includes memory barriers to synchronize queue descriptor
reads. The second patch fixes a memory leak that could
occur if the device returns a TX completion with an error
code in the descriptor, in which case the respective socket
buffer and other relevant data structures may not be freed
or updated properly.

v2: Provide more detailed comments explaining specifically what
    reads are being ordered, suggested by Michael Ellerman

Thomas Falcon (2):
  ibmvnic: Ensure that SCRQ entry reads are correctly ordered
  ibmvnic: Fix TX completion error handling

 drivers/net/ethernet/ibm/ibmvnic.c | 22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

Comments

Jakub Kicinski Dec. 1, 2020, 1:12 a.m. UTC | #1
On Mon, 30 Nov 2020 13:07:22 -0600 Thomas Falcon wrote:
> This series resolves a few issues in the ibmvnic driver's

> RX buffer and TX completion processing. The first patch

> includes memory barriers to synchronize queue descriptor

> reads. The second patch fixes a memory leak that could

> occur if the device returns a TX completion with an error

> code in the descriptor, in which case the respective socket

> buffer and other relevant data structures may not be freed

> or updated properly.

> 

> v2: Provide more detailed comments explaining specifically what

>     reads are being ordered, suggested by Michael Ellerman


The commit hashes on fixes tags need to be at least 12 characters long,
please fix and repost.