Re: [PATCH] x86 microcode: don't check the size

[Shaohua Li <shaohua.li@intel.com>]

On Thu, 2006-09-07 at 00:18 -0400, Jeff Garzik wrote:
> Shaohua Li wrote:
> > IA32 manual says if micorcode update's size is 0, then the size is
> > default size (2048 bytes). But this doesn't suggest all microcode
> > update's size should be above 2048 bytes to me. We actually had a
> > microcode update whose size is 1024 bytes. The patch just removed the
> > check.
> > 
> > Signed-off-by: Shaohua Li <shaohua.li@intel.com>
> 
> Why not explicitly check for zero, rather than removing the questionable 
> less-than test?  The default size logic hasn't disappeared...
We just don't want the default size check. If the size is 0, we still
need use default size. The patch is just for those whose size isn't 0
but smaller than default size.

Thanks,
Shaohua
-
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/