tree fcb0bf9b49df17b7a4ddfc320339a05731421ab2
parent 8b9a47ffbf7e3ccc96f4bfe9fd067a8bec396a6b
author Catherine Sullivan <csully@google.com> 1602103156 -0700
committer David Awogbemila <awogbemila@google.com> 1621467517 +0000

gve: Fix bug in gve_rx_alloc/free_ring

In gve_rx_free_ring, now that we are setting the pagecount high, rather
than just freeing the page with put_page we need to make sure we
subtract off the bias first.

In gve_rx_alloc_ring, we need to unfill the ring if the allocation fails
to properly cleanup.

In gve_prefill_buffers, if a page allocation failed we weren't cleaning
up properly. In addition to freeing the page we need to reduce the page
count so we should be using free_buffer. We should also be actually
iterating over the buffers that were successfully alloced not using the
local page and addr vars.

BUG=b/184636169

Signed-off-by: Catherine Sullivan <csully@google.com>
Change-Id: Ic944a6f59ad522fb7fdd96a5e28d492bf855561a
Reviewed-on: https://cos-review.googlesource.com/c/third_party/kernel/+/14552
Reviewed-by: Oleksandr Tymoshenko <ovt@google.com>
Tested-by: Vaibhav Rustagi <vaibhavrustagi@google.com>