On 19 Feb 2016, Arnd Bergmann wrote:
quoted
But, now I see that PHYS_OFFSET is used all over the place as a hard
coded #define, hence your comment "which cannot patch the kernel image
at boot time"
So, I retract my thought...it has to be configured at build-time
(unless of course you turn it into a global variable everywhere...which
might be an even bigger mess)
BTW, I've tried removing the patching in CONFIG_PHYS_VIRT and replaced it
with references to __pv_phys_pfn_offset, which surprisingly only grew
.text from 4901692 to 4904300 bytes, so the size overhead of doing this
would be close to zero.
Arnd
Cool. If you come up with a patch, I'll give it a try.
Overall, I'd rather side with "Let me enable stuff even though it might not work automatically" as opposed to "Don't let me do anything unless it's simple and safe" (which kind of reminds me of Apple products)
Chris