Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1441276 > unrolled thread

Re: [PATCH v9 0/7] ACPI / processor_idle: Add ACPI v6.0 LPI support

Started bySudeep Holla <sudeep.holla@arm.com>
First post2016-07-12 13:50 +0200
Last post2016-07-14 03:20 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v9 0/7] ACPI / processor_idle: Add ACPI v6.0 LPI support Sudeep Holla <sudeep.holla@arm.com> - 2016-07-12 13:50 +0200
    Re: [PATCH v9 0/7] ACPI / processor_idle: Add ACPI v6.0 LPI support "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-12 14:10 +0200
      Re: [PATCH v9 0/7] ACPI / processor_idle: Add ACPI v6.0 LPI support Sudeep Holla <sudeep.holla@arm.com> - 2016-07-12 15:00 +0200
        Re: [PATCH v9 0/7] ACPI / processor_idle: Add ACPI v6.0 LPI support "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-14 03:20 +0200

#1441276 — Re: [PATCH v9 0/7] ACPI / processor_idle: Add ACPI v6.0 LPI support

FromSudeep Holla <sudeep.holla@arm.com>
Date2016-07-12 13:50 +0200
SubjectRe: [PATCH v9 0/7] ACPI / processor_idle: Add ACPI v6.0 LPI support
Message-ID<rU4ml-4l8-5@gated-at.bofh.it>

On 08/07/16 18:07, Sudeep Holla wrote:
> ACPI 6.0 introduced LPI(Low Power Idle) states that provides an alternate
> method to describe processor idle states. It extends the specification
> to allow the expression of idle states like C-states selectable by the
> OSPM when a processor goes idle, but may affect more than one processor,
> and may affect other system components.
>
> LPI extensions leverages the processor container device(again introduced
> in ACPI 6.0) allowing to express which parts of the system are affected
> by a given LPI state. It defines the local power states for each node
> in a hierarchical processor topology. The OSPM can use _LPI object to
> select a local power state for each level of processor hierarchy in the
> system. They used to produce a composite power state request that is
> presented to the platform by the OSPM.
>
> Since multiple processors affect the idle state for any non-leaf hierarchy
> node, coordination of idle state requests between the processors is
> required. ACPI supports two different coordination schemes: Platform
> coordinated and  OS initiated.
>

I was hoping to get this in v4.8 now that merge window is
delayed/extended if you have no further comments on this series.

-- 
Regards,
Sudeep

[toc] | [next] | [standalone]


#1441299

From"Rafael J. Wysocki" <rjw@rjwysocki.net>
Date2016-07-12 14:10 +0200
Message-ID<rU4FI-4He-39@gated-at.bofh.it>
In reply to#1441276
On Tuesday, July 12, 2016 12:42:06 PM Sudeep Holla wrote:
> 
> On 08/07/16 18:07, Sudeep Holla wrote:
> > ACPI 6.0 introduced LPI(Low Power Idle) states that provides an alternate
> > method to describe processor idle states. It extends the specification
> > to allow the expression of idle states like C-states selectable by the
> > OSPM when a processor goes idle, but may affect more than one processor,
> > and may affect other system components.
> >
> > LPI extensions leverages the processor container device(again introduced
> > in ACPI 6.0) allowing to express which parts of the system are affected
> > by a given LPI state. It defines the local power states for each node
> > in a hierarchical processor topology. The OSPM can use _LPI object to
> > select a local power state for each level of processor hierarchy in the
> > system. They used to produce a composite power state request that is
> > presented to the platform by the OSPM.
> >
> > Since multiple processors affect the idle state for any non-leaf hierarchy
> > node, coordination of idle state requests between the processors is
> > required. ACPI supports two different coordination schemes: Platform
> > coordinated and  OS initiated.
> >
> 
> I was hoping to get this in v4.8 now that merge window is
> delayed/extended if you have no further comments on this series.

I'll get to it in the next couple of days.  If it looks all good and there are
no comments, I'll queue it up.

Thanks,
Rafael

[toc] | [prev] | [next] | [standalone]


#1441324

FromSudeep Holla <sudeep.holla@arm.com>
Date2016-07-12 15:00 +0200
Message-ID<rU5s5-51i-5@gated-at.bofh.it>
In reply to#1441299

On 12/07/16 13:09, Rafael J. Wysocki wrote:
> On Tuesday, July 12, 2016 12:42:06 PM Sudeep Holla wrote:
>>
>> On 08/07/16 18:07, Sudeep Holla wrote:
>>> ACPI 6.0 introduced LPI(Low Power Idle) states that provides an alternate
>>> method to describe processor idle states. It extends the specification
>>> to allow the expression of idle states like C-states selectable by the
>>> OSPM when a processor goes idle, but may affect more than one processor,
>>> and may affect other system components.
>>>
>>> LPI extensions leverages the processor container device(again introduced
>>> in ACPI 6.0) allowing to express which parts of the system are affected
>>> by a given LPI state. It defines the local power states for each node
>>> in a hierarchical processor topology. The OSPM can use _LPI object to
>>> select a local power state for each level of processor hierarchy in the
>>> system. They used to produce a composite power state request that is
>>> presented to the platform by the OSPM.
>>>
>>> Since multiple processors affect the idle state for any non-leaf hierarchy
>>> node, coordination of idle state requests between the processors is
>>> required. ACPI supports two different coordination schemes: Platform
>>> coordinated and  OS initiated.
>>>
>>
>> I was hoping to get this in v4.8 now that merge window is
>> delayed/extended if you have no further comments on this series.
>
> I'll get to it in the next couple of days.  If it looks all good and there are
> no comments, I'll queue it up.
>

Thanks for the update.

-- 
Regards,
Sudeep

[toc] | [prev] | [next] | [standalone]


#1442980

From"Rafael J. Wysocki" <rafael@kernel.org>
Date2016-07-14 03:20 +0200
Message-ID<rUDtM-2CC-11@gated-at.bofh.it>
In reply to#1441324
On Tue, Jul 12, 2016 at 2:59 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>
>
> On 12/07/16 13:09, Rafael J. Wysocki wrote:
>>
>> On Tuesday, July 12, 2016 12:42:06 PM Sudeep Holla wrote:
>>>
>>>
>>> On 08/07/16 18:07, Sudeep Holla wrote:
>>>>
>>>> ACPI 6.0 introduced LPI(Low Power Idle) states that provides an
>>>> alternate
>>>> method to describe processor idle states. It extends the specification
>>>> to allow the expression of idle states like C-states selectable by the
>>>> OSPM when a processor goes idle, but may affect more than one processor,
>>>> and may affect other system components.
>>>>
>>>> LPI extensions leverages the processor container device(again introduced
>>>> in ACPI 6.0) allowing to express which parts of the system are affected
>>>> by a given LPI state. It defines the local power states for each node
>>>> in a hierarchical processor topology. The OSPM can use _LPI object to
>>>> select a local power state for each level of processor hierarchy in the
>>>> system. They used to produce a composite power state request that is
>>>> presented to the platform by the OSPM.
>>>>
>>>> Since multiple processors affect the idle state for any non-leaf
>>>> hierarchy
>>>> node, coordination of idle state requests between the processors is
>>>> required. ACPI supports two different coordination schemes: Platform
>>>> coordinated and  OS initiated.
>>>>
>>>
>>> I was hoping to get this in v4.8 now that merge window is
>>> delayed/extended if you have no further comments on this series.
>>
>>
>> I'll get to it in the next couple of days.  If it looks all good and there
>> are
>> no comments, I'll queue it up.
>>
>
> Thanks for the update.

I've just looked at patch [2/7] and there still are things I'm not
particularly liking in it.

Apart from a bunch of minor stuff, acpi_processor_get_lpi_info() is
not my favorite.  The way it creates all of those temporary data
structures and then walks them recursively seems a bit excessive.

I need to think about it a bit more, so I'll send my comments on that
patch later this week.

Thanks,
Rafael

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web