Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-07 Thread Krzysztof Hałasa
Russell King - ARM Linux writes: > Okay, reverted. Thanks. -- Krzysztof Halasa Research Institute for Automation and Measurements PIAP Al. Jerozolimskie 202, 02-486 Warsaw, Poland -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@v

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-07 Thread Russell King - ARM Linux
On Tue, Jan 07, 2014 at 12:45:42PM -0500, Santosh Shilimkar wrote: > On Monday 06 January 2014 08:11 PM, Russell King - ARM Linux wrote: > > /-- > > | > > Right, so, with Assabet, which has CONFIG_DMA_ZONE=y and > > | > > CONFIG_ARM_PATCH_

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-07 Thread Santosh Shilimkar
On Monday 06 January 2014 08:11 PM, Russell King - ARM Linux wrote: > On Mon, Jan 06, 2014 at 06:42:13PM -0500, Santosh Shilimkar wrote: >> On Monday 06 January 2014 05:39 PM, Russell King - ARM Linux wrote: >>> On Mon, Jan 06, 2014 at 08:28:20PM +0100, Krzysztof Hałasa wrote: Russell, Santosh

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-06 Thread Russell King - ARM Linux
On Mon, Jan 06, 2014 at 06:42:13PM -0500, Santosh Shilimkar wrote: > On Monday 06 January 2014 05:39 PM, Russell King - ARM Linux wrote: > > On Mon, Jan 06, 2014 at 08:28:20PM +0100, Krzysztof Hałasa wrote: > >> Russell, Santosh, > >> > >> the unneeded commit causing regression is still in place. P

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-06 Thread Russell King - ARM Linux
On Mon, Jan 06, 2014 at 08:28:20PM +0100, Krzysztof Hałasa wrote: > Russell, Santosh, > > the unneeded commit causing regression is still in place. Please try to > compile an ARM kernel without CONFIG_ARM_PATCH_PHYS_VIRT and with > CONFIG_ZONE_DMA and see for yourself, if you don't believe me. >

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-06 Thread Santosh Shilimkar
On Monday 06 January 2014 05:39 PM, Russell King - ARM Linux wrote: > On Mon, Jan 06, 2014 at 08:28:20PM +0100, Krzysztof Hałasa wrote: >> Russell, Santosh, >> >> the unneeded commit causing regression is still in place. Please try to >> compile an ARM kernel without CONFIG_ARM_PATCH_PHYS_VIRT and

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-06 Thread Santosh Shilimkar
On Monday 06 January 2014 05:08 PM, Krzysztof Hałasa wrote: > Santosh Shilimkar writes: > >> I am afraid you didn't understood what the fix is if you say above. >> arm_dma_limit is broken without this fix for LPAE machines with >> memory starting 4 GB physical boundary. > > I wonder what CONFIG_

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-06 Thread Krzysztof Hałasa
Santosh Shilimkar writes: > I am afraid you didn't understood what the fix is if you say above. > arm_dma_limit is broken without this fix for LPAE machines with > memory starting 4 GB physical boundary. I wonder what CONFIG_ARM_PATCH_PHYS_VIRT and CONFIG_ZONE_DMA do they use? Of course, CONFIG

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-06 Thread Santosh Shilimkar
On Monday 06 January 2014 02:28 PM, Krzysztof Hałasa wrote: > Russell, Santosh, > > the unneeded commit causing regression is still in place. Please try to > compile an ARM kernel without CONFIG_ARM_PATCH_PHYS_VIRT and with > CONFIG_ZONE_DMA and see for yourself, if you don't believe me. > > Plea

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-06 Thread Krzysztof Hałasa
Russell, Santosh, the unneeded commit causing regression is still in place. Please try to compile an ARM kernel without CONFIG_ARM_PATCH_PHYS_VIRT and with CONFIG_ZONE_DMA and see for yourself, if you don't believe me. Please be aware that this commit fixes nothing, its only function is causing t

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2014-01-01 Thread Krzysztof Hałasa
Russell King - ARM Linux writes: > On Tue, Dec 31, 2013 at 12:20:56PM +0100, Krzysztof Hałasa wrote: >> arch/arm/mm/init.c: In function 'setup_dma_zone': >> arch/arm/mm/init.c:232:19: error: '__pv_phys_offset' undeclared (first use >> in this function) >> >> Reverting the following commit fixes

Re: Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2013-12-31 Thread Russell King - ARM Linux
On Tue, Dec 31, 2013 at 12:20:56PM +0100, Krzysztof Hałasa wrote: > arch/arm/mm/init.c: In function 'setup_dma_zone': > arch/arm/mm/init.c:232:19: error: '__pv_phys_offset' undeclared (first use in > this function) > > Reverting the following commit fixes it: Already fixed long before Christmas.

Regression (ARM) arch/arm/mm/init.c doesn't build without CONFIG_ARM_PATCH_PHYS_VIRT.

2013-12-31 Thread Krzysztof Hałasa
arch/arm/mm/init.c: In function 'setup_dma_zone': arch/arm/mm/init.c:232:19: error: '__pv_phys_offset' undeclared (first use in this function) Reverting the following commit fixes it: commit 787b0d5c1ca7ff24feb6f92e4c7f4410ee7d81a8 Author: Santosh Shilimkar Date: Mon Dec 2 20:29:12 2013 +0100