Re: [PATCH] PATCH: dpt_i2o memory leak comments

[Richard B. Johnson]

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.