On Thu, 2 Feb 2023, Linus Torvalds wrote: > So I think that commit should be reverted as broken, and then people > should *maybe* look at something like this (intentionally whitespace > damaged, and people should *really* think about what the > MTRR_TYPE_INVALID case should be - returning UC- is probably what is Not fully understanding what your proposal really does, I got curious and applied it to v6.2-rc6 with 90b926e68f50 (upstream) reverted. And it boots, and the disks are there, and the "ioremap error" is gone, but now I've got strange memory allocation errors, for like really small operations (I wanted to capture dmesg): $ dmesg -t | xz -9ec | base64 xz: (stdin): Out of memory With v6.2-rc6 (vanilla, and also booted under Xen) there's no problem allocating much more memory than that, so something is still not right. Patch applied for reference, but as you said: nobody should apply this :-) More details: - https://nerdbynature.de/bits/usb_v6.1.8/dmesg.6.1.8.xen - https://nerdbynature.de/bits/usb_v6.1.8/dmesg_xen-6.1.8_MTRR_TYPE_INVALID.txt - https://nerdbynature.de/bits/usb_v6.1.8/meminfo_xen-6.1.8.txt - https://nerdbynature.de/bits/usb_v6.1.8/meminfo_xen-6.1.8_MTRR_TYPE_INVALID.txt Thanks, Christian. -- BOFH excuse #448: vi needs to be upgraded to vii