Thread (18 messages) flat view 18 messages, 4 authors, 2016-02-23
STALE3821d

[PATCH] ARM: Allow MULTIPLATFORM to select XIP

From: Chris.Brandt@renesas.com (Chris Brandt)
Date: 2016-02-19 16:28:06

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help