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

[Matthew Garrett]

On Thu, 2014-02-20 at 16:06 -0600, Russ Anderson wrote:
> On Thu, Feb 20, 2014 at 09:39:23PM +0000, 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 many not think wasting their cpu cycles is important, but they do.
> > 
> > Then they should be running locally built kernels in order to ensure
> 
> Why don't YOU run a locally built kernel?

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, as far as the ACPI spec goes, is to be broken.

>> If you have specific bug reports, that would be helpful. But you're not
> > describing actual failure conditions or showing any willingness to
> > figure out what the underlying problem is.
> 
> You can't fix your problem without creating problems for
> others to fix?

ACPI 4.0 includes support for IPMI operation regions. Modular IPMI means
that the kernel will spend a significant amount of time (potentially
until a user manually loads a driver) failing to implement part of the
IPMI specification. That's a problem, and the correct fix is to ensure
that the kernel always implements IPMI support.

Now, you've described some other problems. I don't disagree that those
are problems. The correct thing for us to do with those problems is to
fix them, not to simply change the kernel defaults such that it's
possible for users to choose between two differently broken states. I'm
absolutely willing to help, as long as you're willing to put some
reasonable amount of effort into describing them.

-- 
Matthew Garrett <matthew.garrett@xxxxxxxxxx>