Messages in this thread | | | Date | Tue, 1 Apr 2003 16:26:39 -0500 (EST) | From | "Richard B. Johnson" <> | Subject | Re: [PATCH] PATCH: dpt_i2o memory leak comments |
| |
On Tue, 1 Apr 2003, Randy.Dunlap wrote:
> | @@ -1318,7 +1318,9 @@ > | while(*status == 0){ > | if(time_after(jiffies,timeout)){ > | printk(KERN_WARNING"%s: IOP Reset Timeout\n",pHba->name); > | - kfree(status); > | + /* We loose 4 bytes of "status" here, but we cannot > | + free these because controller may awake and corrupt > | + those bytes at any time */ > s/loose/lose/ > > | @@ -1336,6 +1338,9 @@ > | } > | if(time_after(jiffies,timeout)){ > | printk(KERN_ERR "%s:Timeout waiting for IOP Reset.\n",pHba->name); > | + /* We loose 4 bytes of "status" here, but we cannot > | + free these because controller may awake and corrupt > | + those bytes at any time */ > s/loose/lose/ > > or is this a Brit vs. Amer difference? (not that I know of) > > -- > ~Randy > -
Loose means that something is rattling around, not connected, or not tied down. Lose is what happens on the Crap Tables (as above).
Cheers, Dick Johnson Penguin : Linux version 2.4.20 on an i686 machine (797.90 BogoMips). Why is the government concerned about the lunatic fringe? Think about it.
- 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/
| |