Re: [PATCH]drivers/ata/libata-eh.c:1509 unneeded memset()

From: René Bolldorf
Date: Thu Jan 07 2010 - 15:40:17 EST


On 01/07/10 21:34, René Bolldorf wrote:
On 01/07/10 20:15, Jeff Garzik wrote:
I need a little bit more detail than an unqualified statement... Did
you audit all paths leading to this code point?

Yes, and my two systems running fine with the patch, no oops or panic's.

Sry forgot that:
/* initialize sense_buf with the error register,
* for the case where they are -not- overwritten
*/
sense_buf[0] = 0x70;
sense_buf[2] = dfl_sense_key;

So i think memset() is not needed and works very well without it.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/