Re: [PATCH] PATCH: dpt_i2o memory leak comments

From: Randy.Dunlap (rddunlap@osdl.org)
Date: Tue Apr 01 2003 - 08:15:04 EST


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



This archive was generated by hypermail 2b29 : Mon Apr 07 2003 - 22:00:14 EST