* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
@ 2015-10-19 10:48 Alim Akhtar
2015-10-19 11:48 ` Javier Martinez Canillas
2015-10-19 13:22 ` Arnd Bergmann
0 siblings, 2 replies; 8+ messages in thread
From: Alim Akhtar @ 2015-10-19 10:48 UTC (permalink / raw)
To: linux-arm-kernel
CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
be used with defconfig.
Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
---
Every time I build exynos_defconfig, I need to manually change RAM_SIZE
to match my ramdisk image size. I am not sure what is the best ramdisk size
might be, but bumping it to 64K might be a reasonable one.
arch/arm/configs/exynos_defconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
index 3349713e6c63..e692aaa2cb8c 100644
--- a/arch/arm/configs/exynos_defconfig
+++ b/arch/arm/configs/exynos_defconfig
@@ -52,7 +52,7 @@ CONFIG_CMA_SIZE_MBYTES=64
CONFIG_BLK_DEV_LOOP=y
CONFIG_BLK_DEV_CRYPTOLOOP=y
CONFIG_BLK_DEV_RAM=y
-CONFIG_BLK_DEV_RAM_SIZE=8192
+CONFIG_BLK_DEV_RAM_SIZE=65536
CONFIG_SCSI=y
CONFIG_BLK_DEV_SD=y
CONFIG_CHR_DEV_SG=y
--
1.7.10.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
2015-10-19 10:48 [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K Alim Akhtar
@ 2015-10-19 11:48 ` Javier Martinez Canillas
2015-10-19 11:51 ` Russell King - ARM Linux
2015-10-19 11:55 ` Alim Akhtar
2015-10-19 13:22 ` Arnd Bergmann
1 sibling, 2 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2015-10-19 11:48 UTC (permalink / raw)
To: linux-arm-kernel
Hello Alim,
On 10/19/2015 12:48 PM, Alim Akhtar wrote:
> CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
> smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
> be used with defconfig.
>
> Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
> ---
I agree that 8KiB is too small and that should be bumped, I'm also
not sure what the best size would be but 64KiB sounds reasonable
to me. So for this patch:
Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
> Every time I build exynos_defconfig, I need to manually change RAM_SIZE
> to match my ramdisk image size. I am not sure what is the best ramdisk size
> might be, but bumping it to 64K might be a reasonable one.
>
In case you are interested, I got some stats by doing grep on the ARM
defconfigs using this [0]. Of course the sizes depends on the platform
but just to have an idea about what sizes are more popular than others.
SIZE COUNT
1024 17
8192 43
10240 1
16384 20
20000 25
32768 36
65536 15
[0]: http://hastebin.com/buqukimede.bash
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
^ permalink raw reply [flat|nested] 8+ messages in thread
* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
2015-10-19 11:48 ` Javier Martinez Canillas
@ 2015-10-19 11:51 ` Russell King - ARM Linux
2015-10-19 11:53 ` Alim Akhtar
2015-10-19 12:06 ` Javier Martinez Canillas
2015-10-19 11:55 ` Alim Akhtar
1 sibling, 2 replies; 8+ messages in thread
From: Russell King - ARM Linux @ 2015-10-19 11:51 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Oct 19, 2015 at 01:48:13PM +0200, Javier Martinez Canillas wrote:
> Hello Alim,
>
> On 10/19/2015 12:48 PM, Alim Akhtar wrote:
> > CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
> > smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
> > be used with defconfig.
> >
> > Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
> > ---
>
> I agree that 8KiB is too small and that should be bumped, I'm also
> not sure what the best size would be but 64KiB sounds reasonable
> to me. So for this patch:
It's _not_ 8KiB or 64KiB.
config BLK_DEV_RAM_SIZE
int "Default RAM disk size (kbytes)"
It's 8MiB or 64MiB - the value you place here is in units of 1024 bytes.
--
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
^ permalink raw reply [flat|nested] 8+ messages in thread* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
2015-10-19 11:51 ` Russell King - ARM Linux
@ 2015-10-19 11:53 ` Alim Akhtar
2015-10-19 12:06 ` Javier Martinez Canillas
1 sibling, 0 replies; 8+ messages in thread
From: Alim Akhtar @ 2015-10-19 11:53 UTC (permalink / raw)
To: linux-arm-kernel
Hi Russell,
On 10/19/2015 05:21 PM, Russell King - ARM Linux wrote:
> On Mon, Oct 19, 2015 at 01:48:13PM +0200, Javier Martinez Canillas wrote:
>> Hello Alim,
>>
>> On 10/19/2015 12:48 PM, Alim Akhtar wrote:
>>> CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
>>> smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
>>> be used with defconfig.
>>>
>>> Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
>>> ---
>>
>> I agree that 8KiB is too small and that should be bumped, I'm also
>> not sure what the best size would be but 64KiB sounds reasonable
>> to me. So for this patch:
>
> It's _not_ 8KiB or 64KiB.
>
> config BLK_DEV_RAM_SIZE
> int "Default RAM disk size (kbytes)"
>
> It's 8MiB or 64MiB - the value you place here is in units of 1024 bytes.
>
Thanks for correcting, will correct it.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
2015-10-19 11:51 ` Russell King - ARM Linux
2015-10-19 11:53 ` Alim Akhtar
@ 2015-10-19 12:06 ` Javier Martinez Canillas
1 sibling, 0 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2015-10-19 12:06 UTC (permalink / raw)
To: linux-arm-kernel
Hello Russell,
On 10/19/2015 01:51 PM, Russell King - ARM Linux wrote:
> On Mon, Oct 19, 2015 at 01:48:13PM +0200, Javier Martinez Canillas wrote:
>> Hello Alim,
>>
>> On 10/19/2015 12:48 PM, Alim Akhtar wrote:
>>> CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
>>> smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
>>> be used with defconfig.
>>>
>>> Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
>>> ---
>>
>> I agree that 8KiB is too small and that should be bumped, I'm also
>> not sure what the best size would be but 64KiB sounds reasonable
>> to me. So for this patch:
>
> It's _not_ 8KiB or 64KiB.
>
> config BLK_DEV_RAM_SIZE
> int "Default RAM disk size (kbytes)"
>
> It's 8MiB or 64MiB - the value you place here is in units of 1024 bytes.
>
Err, right. I got confused by the patch's commit message, sorry about
that and thanks for the clarification.
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
^ permalink raw reply [flat|nested] 8+ messages in thread
* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
2015-10-19 11:48 ` Javier Martinez Canillas
2015-10-19 11:51 ` Russell King - ARM Linux
@ 2015-10-19 11:55 ` Alim Akhtar
1 sibling, 0 replies; 8+ messages in thread
From: Alim Akhtar @ 2015-10-19 11:55 UTC (permalink / raw)
To: linux-arm-kernel
Hi Javier,
On 10/19/2015 05:18 PM, Javier Martinez Canillas wrote:
> Hello Alim,
>
> On 10/19/2015 12:48 PM, Alim Akhtar wrote:
>> CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
>> smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
>> be used with defconfig.
>>
>> Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
>> ---
>
> I agree that 8KiB is too small and that should be bumped, I'm also
> not sure what the best size would be but 64KiB sounds reasonable
> to me. So for this patch:
>
> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
>
Thanks
>> Every time I build exynos_defconfig, I need to manually change RAM_SIZE
>> to match my ramdisk image size. I am not sure what is the best ramdisk size
>> might be, but bumping it to 64K might be a reasonable one.
>>
>
> In case you are interested, I got some stats by doing grep on the ARM
> defconfigs using this [0]. Of course the sizes depends on the platform
> but just to have an idea about what sizes are more popular than others.
>
> SIZE COUNT
> 1024 17
> 8192 43
> 10240 1
> 16384 20
> 20000 25
> 32768 36
> 65536 15
>
This is informative.
> [0]: http://hastebin.com/buqukimede.bash
>
> Best regards,
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
2015-10-19 10:48 [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K Alim Akhtar
2015-10-19 11:48 ` Javier Martinez Canillas
@ 2015-10-19 13:22 ` Arnd Bergmann
2015-10-21 4:56 ` Alim Akhtar
1 sibling, 1 reply; 8+ messages in thread
From: Arnd Bergmann @ 2015-10-19 13:22 UTC (permalink / raw)
To: linux-arm-kernel
On Monday 19 October 2015 16:18:35 Alim Akhtar wrote:
> Spam Status: CRM114
> CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
> smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
> be used with defconfig.
>
> Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
> ---
> Every time I build exynos_defconfig, I need to manually change RAM_SIZE
> to match my ramdisk image size. I am not sure what is the best ramdisk size
> might be, but bumping it to 64K might be a reasonable one.
Is there still a reason to use ramdisks these days? If 8MB is not
enough to have a useful initrd, that might mean that everyone else
uses initramfs and you should just do the same thing?
Then we can also remove this option here.
Arnd
^ permalink raw reply [flat|nested] 8+ messages in thread
* [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K
2015-10-19 13:22 ` Arnd Bergmann
@ 2015-10-21 4:56 ` Alim Akhtar
0 siblings, 0 replies; 8+ messages in thread
From: Alim Akhtar @ 2015-10-21 4:56 UTC (permalink / raw)
To: linux-arm-kernel
Hi Arnd
On 10/19/2015 06:52 PM, Arnd Bergmann wrote:
> On Monday 19 October 2015 16:18:35 Alim Akhtar wrote:
>> Spam Status: CRM114
>> CONFIG_BLK_DEV_RAM_SIZE is currently set to 8K, which is a bit on the
>> smaller side, lets bump it up to 64K so that a bigger RAM_DISK can
>> be used with defconfig.
>>
>> Signed-off-by: Alim Akhtar <alim.akhtar@samsung.com>
>> ---
>> Every time I build exynos_defconfig, I need to manually change RAM_SIZE
>> to match my ramdisk image size. I am not sure what is the best ramdisk size
>> might be, but bumping it to 64K might be a reasonable one.
>
> Is there still a reason to use ramdisks these days? If 8MB is not
> enough to have a useful initrd, that might mean that everyone else
> uses initramfs and you should just do the same thing?
>
There is not specific reason to still use ramdisks, but I feel till we
remove this config, lets modify it to be used by few like me.
I am ok with either way.
Thanks
> Then we can also remove this option here.
>
> Arnd
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-10-21 4:56 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-19 10:48 [RFC PATCH] ARM: exynos_defconfig: Increase CONFIG_BLK_DEV_RAM_SIZE to 64K Alim Akhtar
2015-10-19 11:48 ` Javier Martinez Canillas
2015-10-19 11:51 ` Russell King - ARM Linux
2015-10-19 11:53 ` Alim Akhtar
2015-10-19 12:06 ` Javier Martinez Canillas
2015-10-19 11:55 ` Alim Akhtar
2015-10-19 13:22 ` Arnd Bergmann
2015-10-21 4:56 ` Alim Akhtar
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).