Messages in this thread | | | Date | Tue, 1 Apr 2003 13:15:04 +0000 | From | "Randy.Dunlap" <> | Subject | Re: [PATCH] PATCH: dpt_i2o memory leak comments |
| |
| @@ -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/
| |