Rene Herman wrote: > How is it for efficiency? I thought it was for correctness. > romsignature is using probe_kernel_adress() while all other accesses > to the ROMs there aren't. > > If nothing else, anyone reading that code is likely to ask himself the > very same question -- why the one, and not the others.
Well, I was wondering about all the uses of __get_user; why not probe_kernel_address() everywhere? I think its reasonable to assume that if the signature is mapped and correct, then everything else is mapped. That's certainly the case for Xen, which is why I added it. If you think this is unclear, then I think a comment to explain this rather than code changes is the appropriate fix. J - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/