[mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

classic Classic list List threaded Threaded
11 messages Options
Reply | Threaded
Open this post in threaded view
|

[mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Eric Miao-4
commit b224640b7e8b201c679dab961de9b36dc832803a
Author: Eric Miao <[hidden email]>
Date:   Tue Apr 20 10:06:27 2010 +0800

     UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

     BugLink: http://bugs.launchpad.net/bugs/563679

     +CONFIG_BLK_DEV_RAM_COUNT=16
     +CONFIG_BLK_DEV_RAM_SIZE=65536

     Signed-off-by: Eric Miao <[hidden email]>

diff --git a/debian.mvl-dove/config/config.common.ubuntu
b/debian.mvl-dove/config/config.common.ubuntu
index 6f859b4..6d15247 100644
--- a/debian.mvl-dove/config/config.common.ubuntu
+++ b/debian.mvl-dove/config/config.common.ubuntu
@@ -265,8 +265,8 @@ CONFIG_BLK_DEV_LOOP=y
  CONFIG_BLK_DEV_MD=m
  CONFIG_BLK_DEV_NBD=m
  CONFIG_BLK_DEV_RAM=y
-CONFIG_BLK_DEV_RAM_COUNT=1
-CONFIG_BLK_DEV_RAM_SIZE=4096
+CONFIG_BLK_DEV_RAM_COUNT=16
+CONFIG_BLK_DEV_RAM_SIZE=65536
  CONFIG_BLK_DEV_SD=y
  CONFIG_BLK_DEV_SR=m
  # CONFIG_BLK_DEV_SR_VENDOR is not set

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Tim Gardner-2
On 04/19/2010 09:29 PM, Eric Miao wrote:

> commit b224640b7e8b201c679dab961de9b36dc832803a
> Author: Eric Miao<[hidden email]>
> Date:   Tue Apr 20 10:06:27 2010 +0800
>
>       UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536
>
>       BugLink: http://bugs.launchpad.net/bugs/563679
>
>       +CONFIG_BLK_DEV_RAM_COUNT=16
>       +CONFIG_BLK_DEV_RAM_SIZE=65536
>
>       Signed-off-by: Eric Miao<[hidden email]>
>
> diff --git a/debian.mvl-dove/config/config.common.ubuntu
> b/debian.mvl-dove/config/config.common.ubuntu
> index 6f859b4..6d15247 100644
> --- a/debian.mvl-dove/config/config.common.ubuntu
> +++ b/debian.mvl-dove/config/config.common.ubuntu
> @@ -265,8 +265,8 @@ CONFIG_BLK_DEV_LOOP=y
>    CONFIG_BLK_DEV_MD=m
>    CONFIG_BLK_DEV_NBD=m
>    CONFIG_BLK_DEV_RAM=y
> -CONFIG_BLK_DEV_RAM_COUNT=1
> -CONFIG_BLK_DEV_RAM_SIZE=4096
> +CONFIG_BLK_DEV_RAM_COUNT=16
> +CONFIG_BLK_DEV_RAM_SIZE=65536
>    CONFIG_BLK_DEV_SD=y
>    CONFIG_BLK_DEV_SR=m
>    # CONFIG_BLK_DEV_SR_VENDOR is not set
>

Assuming testing has shown this to work:

Acked-by: Tim Gardner <[hidden email]>

--
Tim Gardner [hidden email]

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Andy Whitcroft-3
In reply to this post by Eric Miao-4
On Tue, Apr 20, 2010 at 11:29:54AM +0800, Eric Miao wrote:

> commit b224640b7e8b201c679dab961de9b36dc832803a
> Author: Eric Miao <[hidden email]>
> Date:   Tue Apr 20 10:06:27 2010 +0800
>
>     UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536
>
>     BugLink: http://bugs.launchpad.net/bugs/563679
>
>     +CONFIG_BLK_DEV_RAM_COUNT=16
>     +CONFIG_BLK_DEV_RAM_SIZE=65536
>
>     Signed-off-by: Eric Miao <[hidden email]>
>
> diff --git a/debian.mvl-dove/config/config.common.ubuntu
> b/debian.mvl-dove/config/config.common.ubuntu
> index 6f859b4..6d15247 100644
> --- a/debian.mvl-dove/config/config.common.ubuntu
> +++ b/debian.mvl-dove/config/config.common.ubuntu
> @@ -265,8 +265,8 @@ CONFIG_BLK_DEV_LOOP=y
>  CONFIG_BLK_DEV_MD=m
>  CONFIG_BLK_DEV_NBD=m
>  CONFIG_BLK_DEV_RAM=y
> -CONFIG_BLK_DEV_RAM_COUNT=1
> -CONFIG_BLK_DEV_RAM_SIZE=4096
> +CONFIG_BLK_DEV_RAM_COUNT=16
> +CONFIG_BLK_DEV_RAM_SIZE=65536
>  CONFIG_BLK_DEV_SD=y
>  CONFIG_BLK_DEV_SR=m
>  # CONFIG_BLK_DEV_SR_VENDOR is not set

Looks good.

Acked-by: Andy Whitcroft <[hidden email]>

-apw

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[APPLIED] [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Andy Whitcroft-3
In reply to this post by Eric Miao-4
Applied to Lucid.

-apw

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Scott James Remnant-2
In reply to this post by Eric Miao-4
On Tue, 2010-04-20 at 11:29 +0800, Eric Miao wrote:

>      +CONFIG_BLK_DEV_RAM_COUNT=16
>
This will have a significant boot performance impact; since now 16 block
devices will have to be created and probed, instead of one.

Scott
--
Scott James Remnant
[hidden email]

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (205 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Colin Ian King-2
In reply to this post by Eric Miao-4
On Tue, 2010-04-20 at 11:29 +0800, Eric Miao wrote:

> commit b224640b7e8b201c679dab961de9b36dc832803a
> Author: Eric Miao <[hidden email]>
> Date:   Tue Apr 20 10:06:27 2010 +0800
>
>      UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536
>
>      BugLink: http://bugs.launchpad.net/bugs/563679
>
>      +CONFIG_BLK_DEV_RAM_COUNT=16
>      +CONFIG_BLK_DEV_RAM_SIZE=65536
>
>      Signed-off-by: Eric Miao <[hidden email]>
>
> diff --git a/debian.mvl-dove/config/config.common.ubuntu
> b/debian.mvl-dove/config/config.common.ubuntu
> index 6f859b4..6d15247 100644
> --- a/debian.mvl-dove/config/config.common.ubuntu
> +++ b/debian.mvl-dove/config/config.common.ubuntu
> @@ -265,8 +265,8 @@ CONFIG_BLK_DEV_LOOP=y
>   CONFIG_BLK_DEV_MD=m
>   CONFIG_BLK_DEV_NBD=m
>   CONFIG_BLK_DEV_RAM=y
> -CONFIG_BLK_DEV_RAM_COUNT=1
> -CONFIG_BLK_DEV_RAM_SIZE=4096
> +CONFIG_BLK_DEV_RAM_COUNT=16
> +CONFIG_BLK_DEV_RAM_SIZE=65536
>   CONFIG_BLK_DEV_SD=y
>   CONFIG_BLK_DEV_SR=m
>   # CONFIG_BLK_DEV_SR_VENDOR is not set
>
Why are we also increasing the count from 1 to 16?


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Eric Miao-4
On Thu, Apr 22, 2010 at 4:35 AM, Colin Ian King
<[hidden email]> wrote:

> On Tue, 2010-04-20 at 11:29 +0800, Eric Miao wrote:
>> commit b224640b7e8b201c679dab961de9b36dc832803a
>> Author: Eric Miao <[hidden email]>
>> Date:   Tue Apr 20 10:06:27 2010 +0800
>>
>>      UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536
>>
>>      BugLink: http://bugs.launchpad.net/bugs/563679
>>
>>      +CONFIG_BLK_DEV_RAM_COUNT=16
>>      +CONFIG_BLK_DEV_RAM_SIZE=65536
>>
>>      Signed-off-by: Eric Miao <[hidden email]>
>>
>> diff --git a/debian.mvl-dove/config/config.common.ubuntu
>> b/debian.mvl-dove/config/config.common.ubuntu
>> index 6f859b4..6d15247 100644
>> --- a/debian.mvl-dove/config/config.common.ubuntu
>> +++ b/debian.mvl-dove/config/config.common.ubuntu
>> @@ -265,8 +265,8 @@ CONFIG_BLK_DEV_LOOP=y
>>   CONFIG_BLK_DEV_MD=m
>>   CONFIG_BLK_DEV_NBD=m
>>   CONFIG_BLK_DEV_RAM=y
>> -CONFIG_BLK_DEV_RAM_COUNT=1
>> -CONFIG_BLK_DEV_RAM_SIZE=4096
>> +CONFIG_BLK_DEV_RAM_COUNT=16
>> +CONFIG_BLK_DEV_RAM_SIZE=65536
>>   CONFIG_BLK_DEV_SD=y
>>   CONFIG_BLK_DEV_SR=m
>>   # CONFIG_BLK_DEV_SR_VENDOR is not set
>>
> Why are we also increasing the count from 1 to 16?
>

Loic pointed out the potential issue of RAM_SIZE being too small to cause
d-i initrds to fail.

The DEV_RAM_COUNT is actually sync'ed with x86, so it will benefit both
ARM and x86 if this can be reduced.

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Oliver Grawert
In reply to this post by Scott James Remnant-2
hi,
Am Mittwoch, den 21.04.2010, 11:43 -0700 schrieb Scott James Remnant:
> On Tue, 2010-04-20 at 11:29 +0800, Eric Miao wrote:
>
> >      +CONFIG_BLK_DEV_RAM_COUNT=16
> >
> This will have a significant boot performance impact; since now 16 block
> devices will have to be created and probed, instead of one.
since the patch is supposed to add initrd (vs. initramfs) support i
wonder why we actually need more than one here, bumping the size for one
device should be totally enough to make the usecase work.

ciao
        oli

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (205 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Andy Whitcroft-3
In reply to this post by Scott James Remnant-2
On Wed, Apr 21, 2010 at 11:43:00AM -0700, Scott James Remnant wrote:
> On Tue, 2010-04-20 at 11:29 +0800, Eric Miao wrote:
>
> >      +CONFIG_BLK_DEV_RAM_COUNT=16
> >
> This will have a significant boot performance impact; since now 16 block
> devices will have to be created and probed, instead of one.

This change was for the mvl-dove branch only, and mearly brings it into
line with the master branch.  Is this something we should be reviewing
across the board?

-apw

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Loïc Minier-17
In reply to this post by Scott James Remnant-2
On Wed, Apr 21, 2010, Scott James Remnant wrote:
> >      +CONFIG_BLK_DEV_RAM_COUNT=16
> >
> This will have a significant boot performance impact; since now 16 block
> devices will have to be created and probed, instead of one.

 Sounds like we need to revisit this for all configs; this thread was
 mostly about bringing dove on par with the other configs.

--
Loïc Minier

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [mvl-dove] UBUNTU: [Config] increase CONFIG_BLK_DEV_RAM_SIZE to 65536

Scott James Remnant-2
In reply to this post by Andy Whitcroft-3
On Thu, 2010-04-22 at 08:19 +0100, Andy Whitcroft wrote:

> On Wed, Apr 21, 2010 at 11:43:00AM -0700, Scott James Remnant wrote:
> > On Tue, 2010-04-20 at 11:29 +0800, Eric Miao wrote:
> >
> > >      +CONFIG_BLK_DEV_RAM_COUNT=16
> > >
> > This will have a significant boot performance impact; since now 16 block
> > devices will have to be created and probed, instead of one.
>
> This change was for the mvl-dove branch only, and mearly brings it into
> line with the master branch.  Is this something we should be reviewing
> across the board?
>
Right, I was just warning that it might affect that branch - one assumes
it was =1 for a reason.

I think I'd rather just see on-demand allocation of RAM and LOOP devices
rather than fixed numbers in the kernel config ;-)

Scott
--
Scott James Remnant
[hidden email]

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (205 bytes) Download Attachment