[LUCID 2/2] [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

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

[LUCID 2/2] [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

Manoj Iyer

Config changes to set [Config] Added CONFIG_SR_REPORT_TIME_LIMIT=NNN where
it will print suspend/resume report of any driver/device that takes more
than NNN msec to resume.

The following changes since commit
a913ef9ba3b23aee8db094d5ab5804777c697ef1:
   Andy Whitcroft (1):
         UBUNTU: Ubuntu-2.6.32-13.18

are available in the git repository at:


ssh://zinc.canonical.com/srv/kernel.ubuntu.com/git/manjo/ubuntu-lucid.git
srtime

Manoj Iyer (2):
       UBUNTU: SAUCE: PM report driver and device suspend/resume times.
       UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

  debian.master/config/config.common.ubuntu |    1 +
  drivers/base/power/main.c                 |   35
+++++++++++++++++++++++++++++
  2 files changed, 36 insertions(+), 0 deletions(-)

From 795589e57107f73015bc64a9f880b824c79df1fd Mon Sep 17 00:00:00 2001
From: Manoj Iyer <[hidden email]>
Date: Thu, 11 Feb 2010 13:38:08 -0600
Subject: [PATCH] UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

To print driver/device suspend/resume time greater than NNN msec
use CONFIG_SR_REPORT_TIME_LIMIT=NNN

Signed-off-by: Manoj Iyer <[hidden email]>
---
  debian.master/config/config.common.ubuntu |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index 94e6614..017b821 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -3711,6 +3711,7 @@ CONFIG_SPLIT_PTLOCK_CPUS=4
  CONFIG_SQUASHFS=m
  # CONFIG_SQUASHFS_EMBEDDED is not set
  CONFIG_SQUASHFS_FRAGMENT_CACHE_SIZE=3
+# CONFIG_SR_REPORT_TIME_LIMIT is not set
  CONFIG_SSB=m
  # CONFIG_SSB_DEBUG is not set
  CONFIG_SSB_DRIVER_PCICORE=y
--
1.6.3.3


Cheers
--- manjo

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

Re: [LUCID 2/2] [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

Amit Kucheria-6
On 10 Feb 11, Manoy Iyer wrote:

>
> Config changes to set [Config] Added CONFIG_SR_REPORT_TIME_LIMIT=NNN where
> it will print suspend/resume report of any driver/device that takes more
> than NNN msec to resume.
>
> The following changes since commit
> a913ef9ba3b23aee8db094d5ab5804777c697ef1:
>    Andy Whitcroft (1):
>          UBUNTU: Ubuntu-2.6.32-13.18
>
> are available in the git repository at:
>
>
> ssh://zinc.canonical.com/srv/kernel.ubuntu.com/git/manjo/ubuntu-lucid.git
> srtime
>
> Manoj Iyer (2):
>        UBUNTU: SAUCE: PM report driver and device suspend/resume times.
>        UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
>
>   debian.master/config/config.common.ubuntu |    1 +
>   drivers/base/power/main.c                 |   35
> +++++++++++++++++++++++++++++
>   2 files changed, 36 insertions(+), 0 deletions(-)
>
> From 795589e57107f73015bc64a9f880b824c79df1fd Mon Sep 17 00:00:00 2001
> From: Manoj Iyer <[hidden email]>
> Date: Thu, 11 Feb 2010 13:38:08 -0600
> Subject: [PATCH] UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
>
> To print driver/device suspend/resume time greater than NNN msec
> use CONFIG_SR_REPORT_TIME_LIMIT=NNN
>
> Signed-off-by: Manoj Iyer <[hidden email]>
> ---
>   debian.master/config/config.common.ubuntu |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
> index 94e6614..017b821 100644
> --- a/debian.master/config/config.common.ubuntu
> +++ b/debian.master/config/config.common.ubuntu
> @@ -3711,6 +3711,7 @@ CONFIG_SPLIT_PTLOCK_CPUS=4
>   CONFIG_SQUASHFS=m
>   # CONFIG_SQUASHFS_EMBEDDED is not set
>   CONFIG_SQUASHFS_FRAGMENT_CACHE_SIZE=3
> +# CONFIG_SR_REPORT_TIME_LIMIT is not set
>   CONFIG_SSB=m
>   # CONFIG_SSB_DEBUG is not set
>   CONFIG_SSB_DRIVER_PCICORE=y
> --

Did you run updateconfigs after you mushed-in CONFIG_SR_REPORT_TIME_LIMIT
into config.common.ubuntu?

There should be a Kconfig option for this that I don't see in either this
patch or the previous one.

NACK.

/Amit

--
----------------------------------------------------------------------
Amit Kucheria, Kernel Engineer || [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: [LUCID 2/2] [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

Andy Whitcroft-3
In reply to this post by Manoj Iyer
On Thu, Feb 11, 2010 at 01:57:13PM -0600, Manoj Iyer wrote:

>
> Config changes to set [Config] Added CONFIG_SR_REPORT_TIME_LIMIT=NNN where
> it will print suspend/resume report of any driver/device that takes more
> than NNN msec to resume.
>
> The following changes since commit
> a913ef9ba3b23aee8db094d5ab5804777c697ef1:
>    Andy Whitcroft (1):
>          UBUNTU: Ubuntu-2.6.32-13.18
>
> are available in the git repository at:
>
>
> ssh://zinc.canonical.com/srv/kernel.ubuntu.com/git/manjo/ubuntu-lucid.git
> srtime
>
> Manoj Iyer (2):
>        UBUNTU: SAUCE: PM report driver and device suspend/resume times.
>        UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
>
>   debian.master/config/config.common.ubuntu |    1 +
>   drivers/base/power/main.c                 |   35
> +++++++++++++++++++++++++++++
>   2 files changed, 36 insertions(+), 0 deletions(-)
>
> From 795589e57107f73015bc64a9f880b824c79df1fd Mon Sep 17 00:00:00 2001
> From: Manoj Iyer <[hidden email]>
> Date: Thu, 11 Feb 2010 13:38:08 -0600
> Subject: [PATCH] UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
>
> To print driver/device suspend/resume time greater than NNN msec
> use CONFIG_SR_REPORT_TIME_LIMIT=NNN
>
> Signed-off-by: Manoj Iyer <[hidden email]>
> ---
>   debian.master/config/config.common.ubuntu |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
> index 94e6614..017b821 100644
> --- a/debian.master/config/config.common.ubuntu
> +++ b/debian.master/config/config.common.ubuntu
> @@ -3711,6 +3711,7 @@ CONFIG_SPLIT_PTLOCK_CPUS=4
>   CONFIG_SQUASHFS=m
>   # CONFIG_SQUASHFS_EMBEDDED is not set
>   CONFIG_SQUASHFS_FRAGMENT_CACHE_SIZE=3
> +# CONFIG_SR_REPORT_TIME_LIMIT is not set
>   CONFIG_SSB=m
>   # CONFIG_SSB_DEBUG is not set
>   CONFIG_SSB_DRIVER_PCICORE=y

It is not clear that this would remain here if you were to run fdr
updateconfigs with this patch set as it does not contain a Kconfig
fragment.  I think you may have forgotten to add a file somewhere.

-apw

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

Re: [LUCID 2/2] [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

Manoj Iyer
In reply to this post by Amit Kucheria-6

What is the correct method of adding config options to
config/config.common.ubuntu? create a Kconfig in drivers/ and run
updateconfig ?


Cheers
--- manjo

On Fri, 12 Feb 2010, Amit Kucheria wrote:

> On 10 Feb 11, Manoy Iyer wrote:
>>
>> Config changes to set [Config] Added CONFIG_SR_REPORT_TIME_LIMIT=NNN where
>> it will print suspend/resume report of any driver/device that takes more
>> than NNN msec to resume.
>>
>> The following changes since commit
>> a913ef9ba3b23aee8db094d5ab5804777c697ef1:
>>    Andy Whitcroft (1):
>>          UBUNTU: Ubuntu-2.6.32-13.18
>>
>> are available in the git repository at:
>>
>>
>> ssh://zinc.canonical.com/srv/kernel.ubuntu.com/git/manjo/ubuntu-lucid.git
>> srtime
>>
>> Manoj Iyer (2):
>>        UBUNTU: SAUCE: PM report driver and device suspend/resume times.
>>        UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
>>
>>   debian.master/config/config.common.ubuntu |    1 +
>>   drivers/base/power/main.c                 |   35
>> +++++++++++++++++++++++++++++
>>   2 files changed, 36 insertions(+), 0 deletions(-)
>>
>> From 795589e57107f73015bc64a9f880b824c79df1fd Mon Sep 17 00:00:00 2001
>> From: Manoj Iyer <[hidden email]>
>> Date: Thu, 11 Feb 2010 13:38:08 -0600
>> Subject: [PATCH] UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
>>
>> To print driver/device suspend/resume time greater than NNN msec
>> use CONFIG_SR_REPORT_TIME_LIMIT=NNN
>>
>> Signed-off-by: Manoj Iyer <[hidden email]>
>> ---
>>   debian.master/config/config.common.ubuntu |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
>> index 94e6614..017b821 100644
>> --- a/debian.master/config/config.common.ubuntu
>> +++ b/debian.master/config/config.common.ubuntu
>> @@ -3711,6 +3711,7 @@ CONFIG_SPLIT_PTLOCK_CPUS=4
>>   CONFIG_SQUASHFS=m
>>   # CONFIG_SQUASHFS_EMBEDDED is not set
>>   CONFIG_SQUASHFS_FRAGMENT_CACHE_SIZE=3
>> +# CONFIG_SR_REPORT_TIME_LIMIT is not set
>>   CONFIG_SSB=m
>>   # CONFIG_SSB_DEBUG is not set
>>   CONFIG_SSB_DRIVER_PCICORE=y
>> --
>
> Did you run updateconfigs after you mushed-in CONFIG_SR_REPORT_TIME_LIMIT
> into config.common.ubuntu?
>
> There should be a Kconfig option for this that I don't see in either this
> patch or the previous one.
>
> NACK.
>
> /Amit
>
> --
> ----------------------------------------------------------------------
> Amit Kucheria, Kernel Engineer || [hidden email]
> ----------------------------------------------------------------------
>
> --
> kernel-team mailing list
> [hidden email]
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>

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

Re: [LUCID 2/2] [Config] Added CONFIG_SR_REPORT_TIME_LIMIT

Amit Kucheria-6
On 10 Feb 12, Manoy Iyer wrote:
>
> What is the correct method of adding config options to
> config/config.common.ubuntu? create a Kconfig in drivers/ and run
> updateconfig ?
>

Add a CONFIG_FOO entry to kernel/power/Kconfig and make it enabled
with a default value.

Look at how BLK_DEV_RAM_COUNT is done in drivers/block/Kconfig

/Amit

>
> On Fri, 12 Feb 2010, Amit Kucheria wrote:
>
> >On 10 Feb 11, Manoy Iyer wrote:
> >>
> >>Config changes to set [Config] Added CONFIG_SR_REPORT_TIME_LIMIT=NNN where
> >>it will print suspend/resume report of any driver/device that takes more
> >>than NNN msec to resume.
> >>
> >>The following changes since commit
> >>a913ef9ba3b23aee8db094d5ab5804777c697ef1:
> >>   Andy Whitcroft (1):
> >>         UBUNTU: Ubuntu-2.6.32-13.18
> >>
> >>are available in the git repository at:
> >>
> >>
> >>ssh://zinc.canonical.com/srv/kernel.ubuntu.com/git/manjo/ubuntu-lucid.git
> >>srtime
> >>
> >>Manoj Iyer (2):
> >>       UBUNTU: SAUCE: PM report driver and device suspend/resume times.
> >>       UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
> >>
> >>  debian.master/config/config.common.ubuntu |    1 +
> >>  drivers/base/power/main.c                 |   35
> >>+++++++++++++++++++++++++++++
> >>  2 files changed, 36 insertions(+), 0 deletions(-)
> >>
> >>From 795589e57107f73015bc64a9f880b824c79df1fd Mon Sep 17 00:00:00 2001
> >>From: Manoj Iyer <[hidden email]>
> >>Date: Thu, 11 Feb 2010 13:38:08 -0600
> >>Subject: [PATCH] UBUNTU: [Config] Added CONFIG_SR_REPORT_TIME_LIMIT
> >>
> >>To print driver/device suspend/resume time greater than NNN msec
> >>use CONFIG_SR_REPORT_TIME_LIMIT=NNN
> >>
> >>Signed-off-by: Manoj Iyer <[hidden email]>
> >>---
> >>  debian.master/config/config.common.ubuntu |    1 +
> >>  1 files changed, 1 insertions(+), 0 deletions(-)
> >>
> >>diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
> >>index 94e6614..017b821 100644
> >>--- a/debian.master/config/config.common.ubuntu
> >>+++ b/debian.master/config/config.common.ubuntu
> >>@@ -3711,6 +3711,7 @@ CONFIG_SPLIT_PTLOCK_CPUS=4
> >>  CONFIG_SQUASHFS=m
> >>  # CONFIG_SQUASHFS_EMBEDDED is not set
> >>  CONFIG_SQUASHFS_FRAGMENT_CACHE_SIZE=3
> >>+# CONFIG_SR_REPORT_TIME_LIMIT is not set
> >>  CONFIG_SSB=m
> >>  # CONFIG_SSB_DEBUG is not set
> >>  CONFIG_SSB_DRIVER_PCICORE=y
> >>--
> >
> >Did you run updateconfigs after you mushed-in CONFIG_SR_REPORT_TIME_LIMIT
> >into config.common.ubuntu?
> >
> >There should be a Kconfig option for this that I don't see in either this
> >patch or the previous one.
> >
> >NACK.
> >
> >/Amit
> >
> >--
> >----------------------------------------------------------------------
> >Amit Kucheria, Kernel Engineer || [hidden email]
> >----------------------------------------------------------------------
> >
> >--
> >kernel-team mailing list
> >[hidden email]
> >https://lists.ubuntu.com/mailman/listinfo/kernel-team
> >

--
----------------------------------------------------------------------
Amit Kucheria, Kernel Engineer || [hidden email]
----------------------------------------------------------------------

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