H. Peter Anvin wrote:
quoted text
> Jeff Garzik wrote:
>>
>> Understood.
>>
>> I guess I am more annoyed that this stealth semantics change appears 
>> to have broken everything that depends on pci_iomap(), including 90%+ 
>> of all libata drivers, unless I am missing something.
>>
>> That one piece of code (pci_iomap) was correct under the old 
>> semantics, on x86 and elsewhere.  It's tested and working nicely, and 
>> depended upon by many drivers.
>>
> 
> That one piece of code has had no effective change.  Under both the old 
> and the new code, both branches functionally because ioremap_nocache(), 
> in one case because of MTRR and in one case because of PAT.

OK good...  libata uses it for controller registers exclusively, so that 
should be fine from an operational standpoint.

The lack of discussion is still a bit irksome...  at the very least I 
could have pointed out that lib/iomap.c wanted an update, and the 2.5 yo 
discussion could have resurfaced.

	Jeff



--
unsubscribe notice
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/