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

Groups > linux.kernel > #1371156

Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly non-modular

Show all headers | View raw

On Mon, Apr 04, 2016 at 03:55:35PM -0400, Paul Gortmaker wrote:
> > This was done in commit 6ce9cd8669fa1195fdc21643370e34523c7ac988
> > ("intel_idle: disable module support") since "...the module capability
> > is cauing more trouble than it is worth."

The reason given in that commit was that "it lost the init race with
ACPI", whatever that means.

> > Since this was done over 5y ago, it is safe to say there is no big desire
> > to overcome the issues with modular versions.  So lets remove the modular
> > code that is essentially orphaned, so that when reading the driver there
> > is no doubt it is builtin-only.

So you want to make the driver non-modular due to lack of desire to
fix it?

> This patch will no longer apply since there were several updates to this
> driver by Richard Cochran dated March 29th.   Before I go and refresh
> the patch for a v2, is there any objections to the general goal of what
> the patch was aiming to achieve -- avoiding use of modular infrastructure
> in non-modular code, and not having module_exit code that can't be run?

On the one hand, the better way is to fix the issues, keeping the
driver's modular form.  That way, by loading and unloading, you can
observe how well it works.  I already started by fixing several bugs
WRT module init/exit.

On the other hand, if there is some technical reason why the modular
form is impossible, then the patch should state it.

Thanks,
Richard

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-04-04 22:00 +0200
  Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular rcochran@linutronix.de - 2016-04-05 05:20 +0200
    RE: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular "Brown, Len" <len.brown@intel.com> - 2016-04-05 06:30 +0200
      Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular rcochran@linutronix.de - 2016-04-05 06:40 +0200
        RE: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular "Brown, Len" <len.brown@intel.com> - 2016-04-05 08:00 +0200
          Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular rcochran@linutronix.de - 2016-04-05 09:40 +0200
            Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular Daniel Lezcano <daniel.lezcano@linaro.org> - 2016-04-07 19:00 +0200
    Re: [PATCH] drivers/idle: make intel_idle.c driver more explicitly  non-modular Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-04-05 20:30 +0200

csiph-web