Re: Linux-2.4.19pre3-ac5

CaT (cat@zip.com.au)
Fri, 22 Mar 2002 17:11:16 +1100


On Thu, Mar 21, 2002 at 09:23:36PM -0800, Andre Hedrick wrote:
> It is a BUG() check to see if there are cases where the interrupt handler
> is being set (re armed) while it is currently set for another event.
>
> if (HWGROUP(drive)->handler != NULL)
> BUG();
> ide_set_handler(drive, handler, timeout, expirey);
>
> If we are reloading the handler but it was set but something else , never
> called during a completion, and/or is dangling. It is a typo my bad :-(
>
> Edit and change it from "==" to "!="

Now I'm confused. Should that be != to ==? :) And every instance I find?

-- 
SOCCER PLAYER IN GENITAL-BITING SCANDAL  ---  "It was something between
friends that I thought would have no importance until this morning when
I got up and saw all  the commotion in the news,"  Gallardo told a news
conference. "It stunned me."
Reyes told Marca that he had "felt a slight pinch."
      -- http://www.azcentral.com/offbeat/articles/1129soccer29-ON.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/