Re: [PATCH] PATCH: dpt_i2o memory leak comments

Randy.Dunlap (rddunlap@osdl.org)
Tue, 1 Apr 2003 13:15:04 +0000


| @@ -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
-
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/