diff options
| author | Brett Creeley <brett.creeley@intel.com> | 2019-06-26 12:20:23 +0300 |
|---|---|---|
| committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2019-07-31 23:40:52 +0300 |
| commit | a1e9968593234c179fcb3e71875897ae585c8362 (patch) | |
| tree | 0a5118fa25d0fa5238cba8c054d8b507989ed9a2 /lib/test-string_helpers.c | |
| parent | 56923ab66467c2edbd1ff97240ff14805e0b1cdc (diff) | |
| download | linux-a1e9968593234c179fcb3e71875897ae585c8362.tar.xz | |
ice: Remove duplicate code in ice_alloc_rx_bufs
Currently if the call to ice_alloc_mapped_page() fails we jump to the
no_buf label, possibly call ice_release_rx_desc(), and return true
indicating that there is more work to do. In the success case we just
fall out of the while loop, possibly call ice_alloc_mapped_page(), and
return false saying we exhausted cleaned_count. This flow can be
improved by breaking if ice_alloc_mapped_page() fails and then the flow
outside of the while loop is the same for the failure and success case.
Signed-off-by: Brett Creeley <brett.creeley@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'lib/test-string_helpers.c')
0 files changed, 0 insertions, 0 deletions