Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-03-16 Thread Pavel Machek
Hi! > > Defaults are not right solution; system should still work if I select > > non-default settings. Which you claim is not a case, but you don't see > > why you should fix it. > > If something is implemented using IPMI, then IPMI has to be there to > use it. Matthew's statement was: > >

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-03-16 Thread Pavel Machek
Hi! Defaults are not right solution; system should still work if I select non-default settings. Which you claim is not a case, but you don't see why you should fix it. If something is implemented using IPMI, then IPMI has to be there to use it. Matthew's statement was: For example,

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-03-13 Thread Corey Minyard
On 03/13/2014 03:38 AM, Pavel Machek wrote: > On Thu 2014-03-13 07:24:36, Matthew Garrett wrote: >> On Thu, 2014-03-13 at 08:22 +0100, Pavel Machek wrote: >>> On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: No. The power meter driver knows nothing about IPMI. It makes no IPMI calls.

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-03-13 Thread Pavel Machek
On Thu 2014-03-13 07:24:36, Matthew Garrett wrote: > On Thu, 2014-03-13 at 08:22 +0100, Pavel Machek wrote: > > On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: > > > No. The power meter driver knows nothing about IPMI. It makes no IPMI > > > calls. There's no requirement that a vendor implement

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-03-13 Thread Matthew Garrett
On Thu, 2014-03-13 at 08:22 +0100, Pavel Machek wrote: > On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: > > No. The power meter driver knows nothing about IPMI. It makes no IPMI > > calls. There's no requirement that a vendor implement it via IPMI. > > Yet you claim that IMPI is needed for

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-03-13 Thread Pavel Machek
On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: > On Thu, 2014-03-13 at 00:00 +0100, Pavel Machek wrote: > > On Tue 2014-02-18 23:15:08, Matthew Garrett wrote: > > > For example, if you load the ACPI power meter driver before you've > > > installed the ACPI IPMI driver you'll typically get

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-03-13 Thread Pavel Machek
On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: On Thu, 2014-03-13 at 00:00 +0100, Pavel Machek wrote: On Tue 2014-02-18 23:15:08, Matthew Garrett wrote: For example, if you load the ACPI power meter driver before you've installed the ACPI IPMI driver you'll typically get failures

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-03-13 Thread Matthew Garrett
On Thu, 2014-03-13 at 08:22 +0100, Pavel Machek wrote: On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: No. The power meter driver knows nothing about IPMI. It makes no IPMI calls. There's no requirement that a vendor implement it via IPMI. Yet you claim that IMPI is needed for that, and

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-03-13 Thread Pavel Machek
On Thu 2014-03-13 07:24:36, Matthew Garrett wrote: On Thu, 2014-03-13 at 08:22 +0100, Pavel Machek wrote: On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: No. The power meter driver knows nothing about IPMI. It makes no IPMI calls. There's no requirement that a vendor implement it via

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-03-13 Thread Corey Minyard
On 03/13/2014 03:38 AM, Pavel Machek wrote: On Thu 2014-03-13 07:24:36, Matthew Garrett wrote: On Thu, 2014-03-13 at 08:22 +0100, Pavel Machek wrote: On Wed 2014-03-12 23:22:49, Matthew Garrett wrote: No. The power meter driver knows nothing about IPMI. It makes no IPMI calls. There's no

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-03-12 Thread Matthew Garrett
On Thu, 2014-03-13 at 00:00 +0100, Pavel Machek wrote: > On Tue 2014-02-18 23:15:08, Matthew Garrett wrote: > > For example, if you load the ACPI power meter driver before you've > > installed the ACPI IPMI driver you'll typically get failures (most > > vendors implement it via IPMI). > > Would

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-03-12 Thread Pavel Machek
On Tue 2014-02-18 23:15:08, Matthew Garrett wrote: > On Wed, 2014-02-19 at 00:26 +0100, Rafael J. Wysocki wrote: > > On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: > > > > > > The ACPI IPMI driver implements IPMI operation region support for the ACPI > > > core. Systems that

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-03-12 Thread Pavel Machek
On Tue 2014-02-18 23:15:08, Matthew Garrett wrote: On Wed, 2014-02-19 at 00:26 +0100, Rafael J. Wysocki wrote: On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-03-12 Thread Matthew Garrett
On Thu, 2014-03-13 at 00:00 +0100, Pavel Machek wrote: On Tue 2014-02-18 23:15:08, Matthew Garrett wrote: For example, if you load the ACPI power meter driver before you've installed the ACPI IPMI driver you'll typically get failures (most vendors implement it via IPMI). Would the right

RE: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-23 Thread Zheng, Lv
Hi, > From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Matthew Garrett > Sent: Saturday, February 22, 2014 12:13 AM > > On Fri, 2014-02-21 at 02:17 +, Zheng, Lv wrote: > > > In fact there is a workaround solution I've posted here: > >

RE: [PATCH V2] Change ACPI IPMI support to default y

2014-02-23 Thread Zheng, Lv
Hi, From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Matthew Garrett Sent: Saturday, February 22, 2014 12:13 AM On Fri, 2014-02-21 at 02:17 +, Zheng, Lv wrote: In fact there is a workaround solution I've posted here:

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-21 Thread Corey Minyard
On 02/21/2014 09:51 AM, Matthew Garrett wrote: > On Fri, 2014-02-21 at 07:37 -0600, Corey Minyard wrote: > >> However, the basic problem is that hardware vendors produce hardware >> that sucks and then expect software to fix all the problems. Most IPMI >> interfaces don't have interrupts, so they

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-21 Thread Russ Anderson
On Fri, Feb 21, 2014 at 12:13:13AM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 17:59 -0600, Russ Anderson wrote: > > On Thu, Feb 20, 2014 at 11:09:42PM +, Matthew Garrett wrote: > > > On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: > > > > > > > > The ACPI spec requires IPMI

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-21 Thread Russ Anderson
On Fri, Feb 21, 2014 at 02:17:12AM +, Zheng, Lv wrote: > Hi, > > Sorry for interrupting you. > I have some information that may be helpful for your discussion. > Please find them in the inlined replies. > Well, I don't want to join the fight, just for your informations. :-) I don't want to

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-21 Thread Matthew Garrett
On Fri, 2014-02-21 at 02:17 +, Zheng, Lv wrote: > In fact there is a workaround solution I've posted here: > https://patchwork.kernel.org/patch/2831851/ > The updated version of this patch can be found at: > https://bugzilla.kernel.org/attachment.cgi?id=112611 > It is the acpi-ipmi13.patch

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-21 Thread Matthew Garrett
On Fri, 2014-02-21 at 07:37 -0600, Corey Minyard wrote: > However, the basic problem is that hardware vendors produce hardware > that sucks and then expect software to fix all the problems. Most IPMI > interfaces don't have interrupts, so they have to be polled. Then they > add important

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-21 Thread Corey Minyard
On 02/20/2014 03:00 PM, Matthew Garrett wrote: > On Thu, 2014-02-20 at 14:59 -0600, Russ Anderson wrote: >> On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: >>> On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: >>> This is also a problem for systems with functional BMCs.

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-21 Thread Corey Minyard
On 02/20/2014 03:00 PM, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:59 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: This is also a problem for systems with functional BMCs. Our large

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-21 Thread Matthew Garrett
On Fri, 2014-02-21 at 07:37 -0600, Corey Minyard wrote: However, the basic problem is that hardware vendors produce hardware that sucks and then expect software to fix all the problems. Most IPMI interfaces don't have interrupts, so they have to be polled. Then they add important interfaces

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-21 Thread Matthew Garrett
On Fri, 2014-02-21 at 02:17 +, Zheng, Lv wrote: In fact there is a workaround solution I've posted here: https://patchwork.kernel.org/patch/2831851/ The updated version of this patch can be found at: https://bugzilla.kernel.org/attachment.cgi?id=112611 It is the acpi-ipmi13.patch file.

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-21 Thread Russ Anderson
On Fri, Feb 21, 2014 at 02:17:12AM +, Zheng, Lv wrote: Hi, Sorry for interrupting you. I have some information that may be helpful for your discussion. Please find them in the inlined replies. Well, I don't want to join the fight, just for your informations. :-) I don't want to join

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-21 Thread Russ Anderson
On Fri, Feb 21, 2014 at 12:13:13AM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 17:59 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 11:09:42PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: The ACPI spec requires IPMI functionality

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-21 Thread Corey Minyard
On 02/21/2014 09:51 AM, Matthew Garrett wrote: On Fri, 2014-02-21 at 07:37 -0600, Corey Minyard wrote: However, the basic problem is that hardware vendors produce hardware that sucks and then expect software to fix all the problems. Most IPMI interfaces don't have interrupts, so they have to

RE: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Zheng, Lv
Hi, Sorry for interrupting you. I have some information that may be helpful for your discussion. Please find them in the inlined replies. Well, I don't want to join the fight, just for your informations. :-) > From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 17:59 -0600, Russ Anderson wrote: > On Thu, Feb 20, 2014 at 11:09:42PM +, Matthew Garrett wrote: > > On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: > > > > > > The ACPI spec requires IPMI functionality before a module loads at > > > boot time? And the kernel is

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 11:09:42PM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: > > On Thu, Feb 20, 2014 at 10:26:45PM +, Matthew Garrett wrote: > > > > Because I'm trying to ensure that the default behaviour of the kernel is > > > to *work*.

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Rafael J. Wysocki
On Thursday, February 20, 2014 02:14:58 PM Russ Anderson wrote: > On Tue, Feb 18, 2014 at 11:28:29AM -0500, Matthew Garrett wrote: > > The ACPI IPMI driver implements IPMI operation region support for the ACPI > > core. Systems that declare ACPI operation regions may reference them at any > >

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: > On Thu, Feb 20, 2014 at 10:26:45PM +, Matthew Garrett wrote: > > Because I'm trying to ensure that the default behaviour of the kernel is > > to *work*. Defaulting to having IPMI be modular means that the default > > behaviour of the

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Tue, Feb 18, 2014 at 11:28:29AM -0500, Matthew Garrett wrote: > The ACPI IPMI driver implements IPMI operation region support for the ACPI > core. Systems that declare ACPI operation regions may reference them at any > time, including during kernel initialisation. These accesses will fail >

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 10:26:45PM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 16:06 -0600, Russ Anderson wrote: > > On Thu, Feb 20, 2014 at 09:39:23PM +, Matthew Garrett wrote: > > > On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: > > > > > > > For some customers _any_

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 16:06 -0600, Russ Anderson wrote: > On Thu, Feb 20, 2014 at 09:39:23PM +, Matthew Garrett wrote: > > On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: > > > > > For some customers _any_ amount is significant, especially > > > on large clustered systems where the

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 09:39:23PM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: > > > For some customers _any_ amount is significant, especially > > on large clustered systems where the amount is multiplied > > by tens or hundreds of thousands of nodes.

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 15:49 -0600, Russ Anderson wrote: > On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: > > You appear to be saying "SGI ship hardware that doesn't work. We don't > > know why it doesn't work and we're not interested in fixing it, so we'd > > prefer the default

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: > > > There are any number of reasons why a BMC may not respond. > > BMCs are notorious for being flakey, with different types > > of BMCs that may or may not be reliable.

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: > For some customers _any_ amount is significant, especially > on large clustered systems where the amount is multiplied > by tens or hundreds of thousands of nodes. > > You many not think wasting their cpu cycles is important, but they do.

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 09:00:48PM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 14:59 -0600, Russ Anderson wrote: > > On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: > > > On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: > > > > > > > This is also a problem for

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 14:59 -0600, Russ Anderson wrote: > On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: > > On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: > > > > > This is also a problem for systems with functional BMCs. Our > > > large cluster systems do all IPMI

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: > > > This is also a problem for systems with functional BMCs. Our > > large cluster systems do all IPMI traffic (monitoring) through > > a system controller back door. We

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: > Why build a driver into the kernel? Because it provides functionality that other drivers may need without there being any mechanism to provide an explicit dependency. The same reason we build the ACPI embedded controller driver into the

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 08:16:22PM +, Matthew Garrett wrote: > On Thu, 2014-02-20 at 14:14 -0600, Russ Anderson wrote: > > > The distro that added this change created all sorts of support > > problems. Problems include kipmi0 spinning at 100% of cpu > > (creating a performance hit) and long

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 14:14 -0600, Russ Anderson wrote: > The distro that added this change created all sorts of support > problems. Problems include kipmi0 spinning at 100% of cpu > (creating a performance hit) and long boot delays (as the > kernel tries to talk to a BMC that will never

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-20 Thread Russ Anderson
On Tue, Feb 18, 2014 at 11:28:29AM -0500, Matthew Garrett wrote: > The ACPI IPMI driver implements IPMI operation region support for the ACPI > core. Systems that declare ACPI operation regions may reference them at any > time, including during kernel initialisation. These accesses will fail >

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Tue, Feb 18, 2014 at 11:28:29AM -0500, Matthew Garrett wrote: The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI operation regions may reference them at any time, including during kernel initialisation. These accesses will fail unless

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 14:14 -0600, Russ Anderson wrote: The distro that added this change created all sorts of support problems. Problems include kipmi0 spinning at 100% of cpu (creating a performance hit) and long boot delays (as the kernel tries to talk to a BMC that will never respond).

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 08:16:22PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:14 -0600, Russ Anderson wrote: The distro that added this change created all sorts of support problems. Problems include kipmi0 spinning at 100% of cpu (creating a performance hit) and long boot

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: Why build a driver into the kernel? Because it provides functionality that other drivers may need without there being any mechanism to provide an explicit dependency. The same reason we build the ACPI embedded controller driver into the

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: This is also a problem for systems with functional BMCs. Our large cluster systems do all IPMI traffic (monitoring) through a system controller back door. We do not

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 14:59 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: This is also a problem for systems with functional BMCs. Our large cluster systems do all IPMI traffic

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 09:00:48PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:59 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: This is also a problem for systems with

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: For some customers _any_ amount is significant, especially on large clustered systems where the amount is multiplied by tens or hundreds of thousands of nodes. You many not think wasting their cpu cycles is important, but they do.

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 14:40 -0600, Russ Anderson wrote: There are any number of reasons why a BMC may not respond. BMCs are notorious for being flakey, with different types of BMCs that may or may not be reliable. You do

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 15:49 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 08:46:04PM +, Matthew Garrett wrote: You appear to be saying SGI ship hardware that doesn't work. We don't know why it doesn't work and we're not interested in fixing it, so we'd prefer the default kernel

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 09:39:23PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: For some customers _any_ amount is significant, especially on large clustered systems where the amount is multiplied by tens or hundreds of thousands of nodes. You

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 16:06 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 09:39:23PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: For some customers _any_ amount is significant, especially on large clustered systems where the amount is

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 10:26:45PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 16:06 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 09:39:23PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 15:28 -0600, Russ Anderson wrote: For some customers _any_ amount is

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Tue, Feb 18, 2014 at 11:28:29AM -0500, Matthew Garrett wrote: The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI operation regions may reference them at any time, including during kernel initialisation. These accesses will fail unless

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 10:26:45PM +, Matthew Garrett wrote: Because I'm trying to ensure that the default behaviour of the kernel is to *work*. Defaulting to having IPMI be modular means that the default behaviour of the kernel,

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Rafael J. Wysocki
On Thursday, February 20, 2014 02:14:58 PM Russ Anderson wrote: On Tue, Feb 18, 2014 at 11:28:29AM -0500, Matthew Garrett wrote: The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI operation regions may reference them at any time,

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Russ Anderson
On Thu, Feb 20, 2014 at 11:09:42PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 10:26:45PM +, Matthew Garrett wrote: Because I'm trying to ensure that the default behaviour of the kernel is to *work*. Defaulting to

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Matthew Garrett
On Thu, 2014-02-20 at 17:59 -0600, Russ Anderson wrote: On Thu, Feb 20, 2014 at 11:09:42PM +, Matthew Garrett wrote: On Thu, 2014-02-20 at 16:45 -0600, Russ Anderson wrote: The ACPI spec requires IPMI functionality before a module loads at boot time? And the kernel is *broken* if

RE: [PATCH V2] Change ACPI IPMI support to default y

2014-02-20 Thread Zheng, Lv
Hi, Sorry for interrupting you. I have some information that may be helpful for your discussion. Please find them in the inlined replies. Well, I don't want to join the fight, just for your informations. :-) From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-18 Thread Corey Minyard
I just queued it up, too, but that's fine. This seems to be a good idea with the way ACPI is going. Acked-by: Corey Minyard On 02/18/2014 06:45 PM, Rafael J. Wysocki wrote: > On Tuesday, February 18, 2014 11:25:02 PM Matthew Garrett wrote: >> On Wed, 2014-02-19 at 00:35 +0100, Rafael J.

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-18 Thread Rafael J. Wysocki
On Tuesday, February 18, 2014 11:25:02 PM Matthew Garrett wrote: > On Wed, 2014-02-19 at 00:35 +0100, Rafael J. Wysocki wrote: > > On Tuesday, February 18, 2014 11:15:08 PM Matthew Garrett wrote: > > > For example, if you load the ACPI power meter driver before you've > > > installed the ACPI IPMI

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-18 Thread Matthew Garrett
On Wed, 2014-02-19 at 00:35 +0100, Rafael J. Wysocki wrote: > On Tuesday, February 18, 2014 11:15:08 PM Matthew Garrett wrote: > > For example, if you load the ACPI power meter driver before you've > > installed the ACPI IPMI driver you'll typically get failures (most > > vendors implement it via

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-18 Thread Rafael J. Wysocki
On Tuesday, February 18, 2014 11:15:08 PM Matthew Garrett wrote: > On Wed, 2014-02-19 at 00:26 +0100, Rafael J. Wysocki wrote: > > On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: > > > > > > The ACPI IPMI driver implements IPMI operation region support for the ACPI > > > core.

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-18 Thread Matthew Garrett
On Wed, 2014-02-19 at 00:26 +0100, Rafael J. Wysocki wrote: > On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: > > > > The ACPI IPMI driver implements IPMI operation region support for the ACPI > > core. Systems that declare ACPI operation regions may reference them at any > >

Re: [PATCH V2] Change ACPI IPMI support to "default y"

2014-02-18 Thread Rafael J. Wysocki
On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: > > The ACPI IPMI driver implements IPMI operation region support for the ACPI > core. Systems that declare ACPI operation regions may reference them at any > time, including during kernel initialisation. These accesses will fail >

[PATCH V2] Change ACPI IPMI support to "default y"

2014-02-18 Thread Matthew Garrett
The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI operation regions may reference them at any time, including during kernel initialisation. These accesses will fail unless the ACPI IPMI driver is present, and undesirable system behaviour may

[PATCH V2] Change ACPI IPMI support to default y

2014-02-18 Thread Matthew Garrett
The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI operation regions may reference them at any time, including during kernel initialisation. These accesses will fail unless the ACPI IPMI driver is present, and undesirable system behaviour may

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-18 Thread Rafael J. Wysocki
On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI operation regions may reference them at any time, including during kernel initialisation. These accesses will fail

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-18 Thread Matthew Garrett
On Wed, 2014-02-19 at 00:26 +0100, Rafael J. Wysocki wrote: On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that declare ACPI operation regions may reference them at any time,

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-18 Thread Rafael J. Wysocki
On Tuesday, February 18, 2014 11:15:08 PM Matthew Garrett wrote: On Wed, 2014-02-19 at 00:26 +0100, Rafael J. Wysocki wrote: On Tuesday, February 18, 2014 11:28:29 AM Matthew Garrett wrote: The ACPI IPMI driver implements IPMI operation region support for the ACPI core. Systems that

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-18 Thread Matthew Garrett
On Wed, 2014-02-19 at 00:35 +0100, Rafael J. Wysocki wrote: On Tuesday, February 18, 2014 11:15:08 PM Matthew Garrett wrote: For example, if you load the ACPI power meter driver before you've installed the ACPI IPMI driver you'll typically get failures (most vendors implement it via IPMI).

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-18 Thread Rafael J. Wysocki
On Tuesday, February 18, 2014 11:25:02 PM Matthew Garrett wrote: On Wed, 2014-02-19 at 00:35 +0100, Rafael J. Wysocki wrote: On Tuesday, February 18, 2014 11:15:08 PM Matthew Garrett wrote: For example, if you load the ACPI power meter driver before you've installed the ACPI IPMI driver

Re: [PATCH V2] Change ACPI IPMI support to default y

2014-02-18 Thread Corey Minyard
I just queued it up, too, but that's fine. This seems to be a good idea with the way ACPI is going. Acked-by: Corey Minyard cminy...@mvista.com On 02/18/2014 06:45 PM, Rafael J. Wysocki wrote: On Tuesday, February 18, 2014 11:25:02 PM Matthew Garrett wrote: On Wed, 2014-02-19 at 00:35 +0100,