* Re: [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU [not found] ` <20120411155232.GA24211@n2100.arm.linux.org.uk> @ 2012-04-11 16:27 ` Russell King - ARM Linux 2012-04-11 17:26 ` David Brown 2012-04-11 17:38 ` Daniel Walker 0 siblings, 2 replies; 5+ messages in thread From: Russell King - ARM Linux @ 2012-04-11 16:27 UTC (permalink / raw) To: Nicolas Pitre, David Brown, Daniel Walker, Bryan Huntsman, linux-arm-msm Cc: Will Deacon, linux-arm-kernel On Wed, Apr 11, 2012 at 04:52:32PM +0100, Russell King - ARM Linux wrote: > On Wed, Apr 11, 2012 at 10:44:22AM -0400, Nicolas Pitre wrote: > > On Wed, 11 Apr 2012, Will Deacon wrote: > > > > > If a bank of memory spanning the 4GB boundary is added on a !CONFIG_LPAE > > > kernel then we will hang early during boot since the memory bank will > > > have wrapped around to zero. > > > > > > This patch truncates memory banks for !LPAE configurations when the end > > > address is not representable in 32 bits. > > > > > > Cc: Nicolas Pitre <nico@fluxnic.net> > > > Signed-off-by: Will Deacon <will.deacon@arm.com> > > > > Acked-by: Nicolas Pitre <nico@linaro.org> > > > > Now what if start = 1G and size = 5G? The size variable is an unsigned > > long, meaning that right now the size might be truncated to 1G. > > There's a solution to that which is quite easy to do: convert the bank > information to PFNs instead of addresses. That will probably eliminate > some corner cases with partial pages which would be desirable too. Of course, what prevents us doing that conversion sanely is all the shite platform code doing crap stuff like this: arch/arm/mach-msm/board-halibut.c: mi->bank[0].start = PHYS_OFFSET; arch/arm/mach-msm/board-halibut.c: mi->bank[0].size = (101*1024*1024); which I went through everything a few years ago and eliminated all this crap. It's back now. Sod it, we'll stick with the current 4GiB limited way as long as we have platform maintainers who do this kind of crappy hack. While here, I propose to delete these: arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].start = PHYS_OFFSET; arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].size = (219*1024*1024); arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].start = MSM_HIGHMEM_BASE; arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].node = PHYS_TO_NID(MSM_HIGHMEM_BASE); arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].size = MSM_HIGHMEM_SIZE; arch/arm/mach-msm/board-sapphire.c: mi->bank[0].start = PHYS_OFFSET; arch/arm/mach-msm/board-sapphire.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (84*1024*1024); arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); because they haven't been buildable since 7th May 2010 (that's 23 months ago), and no one has reported any build errors with them. They're only receiving updates from other sweeps and nothing more. This all means no one is even attempting to build this code. It's pointless having unbuildable code in the kernel, and it's nothing more than a useless maintanence burden. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU 2012-04-11 16:27 ` [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU Russell King - ARM Linux @ 2012-04-11 17:26 ` David Brown 2012-04-11 17:38 ` Daniel Walker 1 sibling, 0 replies; 5+ messages in thread From: David Brown @ 2012-04-11 17:26 UTC (permalink / raw) To: Russell King - ARM Linux Cc: Nicolas Pitre, Daniel Walker, Bryan Huntsman, linux-arm-msm, Will Deacon, linux-arm-kernel On Wed, Apr 11, 2012 at 05:27:30PM +0100, Russell King - ARM Linux wrote: > On Wed, Apr 11, 2012 at 04:52:32PM +0100, Russell King - ARM Linux wrote: > > On Wed, Apr 11, 2012 at 10:44:22AM -0400, Nicolas Pitre wrote: > > > On Wed, 11 Apr 2012, Will Deacon wrote: > > > > > > > If a bank of memory spanning the 4GB boundary is added on a !CONFIG_LPAE > > > > kernel then we will hang early during boot since the memory bank will > > > > have wrapped around to zero. > > > > > > > > This patch truncates memory banks for !LPAE configurations when the end > > > > address is not representable in 32 bits. > > > > > > > > Cc: Nicolas Pitre <nico@fluxnic.net> > > > > Signed-off-by: Will Deacon <will.deacon@arm.com> > > > > > > Acked-by: Nicolas Pitre <nico@linaro.org> > > > > > > Now what if start = 1G and size = 5G? The size variable is an unsigned > > > long, meaning that right now the size might be truncated to 1G. > > > > There's a solution to that which is quite easy to do: convert the bank > > information to PFNs instead of addresses. That will probably eliminate > > some corner cases with partial pages which would be desirable too. > > Of course, what prevents us doing that conversion sanely is all the > shite platform code doing crap stuff like this: > > arch/arm/mach-msm/board-halibut.c: mi->bank[0].start = PHYS_OFFSET; > arch/arm/mach-msm/board-halibut.c: mi->bank[0].size = (101*1024*1024); > > which I went through everything a few years ago and eliminated all this > crap. It's back now. Sod it, we'll stick with the current 4GiB limited > way as long as we have platform maintainers who do this kind of crappy > hack. I'm not sure there are even any working "halibut" targets (MSM7201 SURF). It wasn't a generally available target. I think the only in-use fish target is trout (HTC Dream). Although, it seems that every time I think this, someone will speak up about one of these targets. > While here, I propose to delete these: > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].start = PHYS_OFFSET; > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].size = (219*1024*1024); > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].start = MSM_HIGHMEM_BASE; > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].node = PHYS_TO_NID(MSM_HIGHMEM_BASE); > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].size = MSM_HIGHMEM_SIZE; > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].start = PHYS_OFFSET; > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (84*1024*1024); > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > > because they haven't been buildable since 7th May 2010 (that's 23 months > ago), and no one has reported any build errors with them. They're only > receiving updates from other sweeps and nothing more. This all means no > one is even attempting to build this code. It's pointless having > unbuildable code in the kernel, and it's nothing more than a useless > maintanence burden. I don't think these were ever built. They were never added to the Makefiles. Patch to follow. David -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU 2012-04-11 16:27 ` [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU Russell King - ARM Linux 2012-04-11 17:26 ` David Brown @ 2012-04-11 17:38 ` Daniel Walker 2012-04-11 17:40 ` Russell King - ARM Linux 1 sibling, 1 reply; 5+ messages in thread From: Daniel Walker @ 2012-04-11 17:38 UTC (permalink / raw) To: Russell King - ARM Linux Cc: Nicolas Pitre, David Brown, Bryan Huntsman, linux-arm-msm, Will Deacon, linux-arm-kernel On Wed, Apr 11, 2012 at 05:27:30PM +0100, Russell King - ARM Linux wrote: > Of course, what prevents us doing that conversion sanely is all the > shite platform code doing crap stuff like this: > > arch/arm/mach-msm/board-halibut.c: mi->bank[0].start = PHYS_OFFSET; > arch/arm/mach-msm/board-halibut.c: mi->bank[0].size = (101*1024*1024); > > which I went through everything a few years ago and eliminated all this > crap. It's back now. Sod it, we'll stick with the current 4GiB limited > way as long as we have platform maintainers who do this kind of crappy > hack. > > While here, I propose to delete these: > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].start = PHYS_OFFSET; > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].size = (219*1024*1024); > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].start = MSM_HIGHMEM_BASE; > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].node = PHYS_TO_NID(MSM_HIGHMEM_BASE); > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].size = MSM_HIGHMEM_SIZE; > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].start = PHYS_OFFSET; > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (84*1024*1024); > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > > because they haven't been buildable since 7th May 2010 (that's 23 months > ago), and no one has reported any build errors with them. They're only > receiving updates from other sweeps and nothing more. This all means no > one is even attempting to build this code. It's pointless having > unbuildable code in the kernel, and it's nothing more than a useless > maintanence burden. It can't be that hard to fix.. I'll look into cleaning it up. Daniel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU 2012-04-11 17:38 ` Daniel Walker @ 2012-04-11 17:40 ` Russell King - ARM Linux 2012-04-11 18:06 ` Daniel Walker 0 siblings, 1 reply; 5+ messages in thread From: Russell King - ARM Linux @ 2012-04-11 17:40 UTC (permalink / raw) To: Daniel Walker Cc: Nicolas Pitre, David Brown, Bryan Huntsman, linux-arm-msm, Will Deacon, linux-arm-kernel On Wed, Apr 11, 2012 at 10:38:50AM -0700, Daniel Walker wrote: > On Wed, Apr 11, 2012 at 05:27:30PM +0100, Russell King - ARM Linux wrote: > > While here, I propose to delete these: > > > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].start = PHYS_OFFSET; > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].size = (219*1024*1024); > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].start = MSM_HIGHMEM_BASE; > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].node = PHYS_TO_NID(MSM_HIGHMEM_BASE); > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].size = MSM_HIGHMEM_SIZE; > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].start = PHYS_OFFSET; > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (84*1024*1024); > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > > > > because they haven't been buildable since 7th May 2010 (that's 23 months > > ago), and no one has reported any build errors with them. They're only > > receiving updates from other sweeps and nothing more. This all means no > > one is even attempting to build this code. It's pointless having > > unbuildable code in the kernel, and it's nothing more than a useless > > maintanence burden. > > > It can't be that hard to fix.. I'll look into cleaning it up. What's the point of fixing code which isn't being used by anyone? As I said above, it's a maintanence burden and if its not being used it needs to be removed. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU 2012-04-11 17:40 ` Russell King - ARM Linux @ 2012-04-11 18:06 ` Daniel Walker 0 siblings, 0 replies; 5+ messages in thread From: Daniel Walker @ 2012-04-11 18:06 UTC (permalink / raw) To: Russell King - ARM Linux Cc: Nicolas Pitre, David Brown, Bryan Huntsman, linux-arm-msm, Will Deacon, linux-arm-kernel On Wed, Apr 11, 2012 at 06:40:24PM +0100, Russell King - ARM Linux wrote: > On Wed, Apr 11, 2012 at 10:38:50AM -0700, Daniel Walker wrote: > > On Wed, Apr 11, 2012 at 05:27:30PM +0100, Russell King - ARM Linux wrote: > > > While here, I propose to delete these: > > > > > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].start = PHYS_OFFSET; > > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[0].size = (219*1024*1024); > > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].start = MSM_HIGHMEM_BASE; > > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].node = PHYS_TO_NID(MSM_HIGHMEM_BASE); > > > arch/arm/mach-msm/board-mahimahi.c: mi->bank[1].size = MSM_HIGHMEM_SIZE; > > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].start = PHYS_OFFSET; > > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].node = PHYS_TO_NID(PHYS_OFFSET); > > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (84*1024*1024); > > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > > > arch/arm/mach-msm/board-sapphire.c: mi->bank[0].size = (101*1024*1024); > > > > > > because they haven't been buildable since 7th May 2010 (that's 23 months > > > ago), and no one has reported any build errors with them. They're only > > > receiving updates from other sweeps and nothing more. This all means no > > > one is even attempting to build this code. It's pointless having > > > unbuildable code in the kernel, and it's nothing more than a useless > > > maintanence burden. > > > > > > It can't be that hard to fix.. I'll look into cleaning it up. > > What's the point of fixing code which isn't being used by anyone? As I > said above, it's a maintanence burden and if its not being used it needs > to be removed. > Aren't there whole are sub-architectures that don't even build ? I think something that's a work in progress is fine, as long as someone (i.e. me) plans to get back to it at some point. I also said that _I_ would make it build .. So someone is going to use it, and it is going to build. Daniel ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-04-11 18:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1334154639-13869-1-git-send-email-will.deacon@arm.com>
[not found] ` <1334154639-13869-4-git-send-email-will.deacon@arm.com>
[not found] ` <alpine.LFD.2.02.1204111038130.16314@xanadu.home>
[not found] ` <20120411155232.GA24211@n2100.arm.linux.org.uk>
2012-04-11 16:27 ` [PATCH v2 3/4] ARM: mm: truncate memory banks to fit in 4GB space for classic MMU Russell King - ARM Linux
2012-04-11 17:26 ` David Brown
2012-04-11 17:38 ` Daniel Walker
2012-04-11 17:40 ` Russell King - ARM Linux
2012-04-11 18:06 ` Daniel Walker
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).