Skip to content

Commit 954b54d

Browse files
arndbJeff Kirsher
authored andcommitted
ixgbevf: fix unused variable warning
The new ixgbevf_set_rx_buffer_len() function causes a harmless warnings in configurations with large page size: drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c: In function 'ixgbevf_set_rx_buffer_len': drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c:1758:15: error: unused variable 'max_frame' [-Werror=unused-variable] This rephrases the code so that the compiler can see the use of that variable, making it slightly easier to read in the process. Fixes: f15c5ba ("ixgbevf: add support for using order 1 pages to receive large frames") Signed-off-by: Arnd Bergmann <[email protected]> Tested-by: Andrew Bowers <[email protected]> Acked-by: Alexander Duyck <[email protected]> Signed-off-by: Jeff Kirsher <[email protected]>
1 parent e656805 commit 954b54d

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1766,12 +1766,12 @@ static void ixgbevf_set_rx_buffer_len(struct ixgbevf_adapter *adapter,
17661766

17671767
set_ring_build_skb_enabled(rx_ring);
17681768

1769-
#if (PAGE_SIZE < 8192)
1770-
if (max_frame <= IXGBEVF_MAX_FRAME_BUILD_SKB)
1771-
return;
1769+
if (PAGE_SIZE < 8192) {
1770+
if (max_frame <= IXGBEVF_MAX_FRAME_BUILD_SKB)
1771+
return;
17721772

1773-
set_ring_uses_large_buffer(rx_ring);
1774-
#endif
1773+
set_ring_uses_large_buffer(rx_ring);
1774+
}
17751775
}
17761776

17771777
/**

0 commit comments

Comments
 (0)