| Submitter | David Miller |
|---|---|
| Date | Nov. 18, 2010, 7:53 p.m. |
| Message ID | <20101118.115331.233701526.davem@davemloft.net> |
| Download | mbox | patch |
| Permalink | /patch/72140/ |
| State | Accepted |
| Delegated to: | David Miller |
| Headers | show |
Comments
Patch
diff --git a/drivers/atm/fore200e.c b/drivers/atm/fore200e.c index c8fc69c..c097619 100644 --- a/drivers/atm/fore200e.c +++ b/drivers/atm/fore200e.c @@ -92,7 +92,7 @@ #define FORE200E_INDEX(virt_addr, type, index) (&((type *)(virt_addr))[ index ]) -#define FORE200E_NEXT_ENTRY(index, modulo) (index = ++(index) % (modulo)) +#define FORE200E_NEXT_ENTRY(index, modulo) (index = ((index) + 1) % (modulo)) #if 1 #define ASSERT(expr) if (!(expr)) { \
GCC (rightfully) complains that: drivers/atm/fore200e.c:614:5: warning: operation on 'cmdq->head' may be undefined This is due to the FORE200E_NEXT_ENTRY macro, which essentially evaluates to: i = ++i % m Make it what's explicitly intended here which is: i = (i + 1) % m and the warning goes away. Signed-off-by: David S. Miller <davem@davemloft.net> --- drivers/atm/fore200e.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)