H. Peter Anvin wrote: > Yinghai Lu wrote: >>>> >>> That doesn't quite look right; for one thing it doesn't handle the >>> (admittedly somewhat unlikely) case of end pointing to the end of the >>> address space. >>> >>> Something like: >>> >>> if (start > (resource_size_t)end-1) >>> continue; >>> > >> + if (start > (resource_size_t)end) >> continue; > > Erk... thinko on my part. Should have been (resource_size_t)-1. > how about start is already 32M ? you will insert blank one... 64bit, we could expand range that is above 4g to be aligned. YH -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html