Revert "net: ifb error path loop fix"
authorLinus Torvalds <torvalds@woody.linux-foundation.org>
Tue, 30 Jan 2007 22:11:12 +0000 (14:11 -0800)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Tue, 30 Jan 2007 22:11:12 +0000 (14:11 -0800)
commitbcdddfb66cc998252d34758ce4109cedc0d24a5c
tree8cea3d0275dce00f12b8b4ac853fac6841a1c6b9
parent0f2452855d86901ba3766826ccb5606ea4e15ab9
Revert "net: ifb error path loop fix"

This reverts commit 0c0b3ae68ec93b1db5c637d294647d1cca0df763.

Quoth David:

  "Jeff, please revert

   It's wrong.  We had a lengthy analysis of this piece of code
   several months ago, and it is correct.

   Consider, if we run the loop and we get an error
   the following happens:

   1) attempt of ifb_init_one(i) fails, therefore we should
      not try to "ifb_free_one()" on "i" since it failed
   2) the loop iteration first increments "i", then it
      check for error

   Therefore we must decrement "i" twice before the first
   free during the cleanup.  One to "undo" the for() loop
   increment, and one to "skip" the ifb_init_one() case which
   failed."

Reported-by: David Miller <davem@davemloft.net>
Acked-by: Jeff Garzik <jgarzik@pobox.com>
Cc: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/net/ifb.c