Commit 6dc7d8c8 authored Mar 09, 2009 by Eilon Greenstein Committed by David S. Miller Mar 10, 2009

bnx2x: Casting page alignment



Adding a proper cast to the argument of PAGE_ALIGN macro so that the output
won't depend on its original type. Without this cast aligned value will be
truncated to the size of the argument type.

Reported-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Vladislav Zolotarov <vladz@broadcom.com>
Tested-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Eilon Greenstein <eilong@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent 916c775f

drivers/net/bnx2x.h

+1 −1

Original line number	Diff line number	Diff line
		@@ -152,7 +152,7 @@ struct sw_rx_page {
		#define PAGES_PER_SGE (1 << PAGES_PER_SGE_SHIFT)
		#define SGE_PAGE_SIZE PAGE_SIZE
		#define SGE_PAGE_SHIFT PAGE_SHIFT
		#define SGE_PAGE_ALIGN(addr) PAGE_ALIGN(addr)
		#define SGE_PAGE_ALIGN(addr) PAGE_ALIGN((typeof(PAGE_SIZE))addr)

		#define BCM_RX_ETH_PAYLOAD_ALIGN 64