Re: [regression, bisected] fails to boot on Dell Optiplex 760 withVT-d enabled

[Faidon Liambotis]

Chris Wright wrote:
> AFAIK, this platform is broken when VT-d is enabled.
> Can you add this patch to a broken kernel and retest?
> Please boot with 'debug' on the kernel command line.
Thanks but David was faster, we've already established that

http://git.infradead.org/iommu-2.6.git/commit/0815565a

which is, as far as I can understand, very similar to your patch, indeed
fixes the issue.

> +	if (ver == (u32)-1 || iommu->cap == (u64)-1 || iommu->ecap == (u64)-1) {
As an observation, David's patch is checking if *both* iommu->cap and
iommu->ecap are -1, while yours checks for either. You're also checking
for ver. Obviously, I have no idea which of the two is more correct.

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