Re: [PATCH RFC v2 2/3] ec: add ec space notifier

From: Rafael J. Wysocki
Date: Mon Jun 10 2013 - 07:34:16 EST


On Monday, June 10, 2013 08:46:54 AM li guang wrote:
> å 2013-06-06åç 12:59 +0200ïRafael J. Wysockiåéï
> > On Thursday, June 06, 2013 09:40:34 AM liguang wrote:
> > > add a notifier for anyone who are instresting in
> > > ec space changing.
> > >
> > > Signed-off-by: liguang <lig.fnst@xxxxxxxxxxxxxx>
> >
> > I'm not going to apply this anyway, but can you please explain what's the
> > problem you're trying to solve here?
>
> OK, currently it is for QEMU to do cpu online automatically after
> hot-plug a cpu,
> and maybe potentially for real platform's cpu
> plug/unplug implementation.
> with this notifier, we don't need any GPIOes,IO ports, and other
> hardware cost if we do have an EC chip in board to trigger kernel's
> cpu process for this.

You need to make ACPI eject/bus/device notifications go to the processor
objects rather than to listen to the EC events.

> Yep, you said you'll reject this anyway,
> but I have to ask do this notifier offend
> any rules of your development?

Well, yes, it does. CPU offline/online is not only for hotplugging and people
do not expect CPUs to actually physically go away after using offline.

Thanks,
Rafael


> > > ---
> > > drivers/acpi/ec.c | 32 ++++++++++++++++++++++++++++++++
> > > include/linux/acpi.h | 2 ++
> > > 2 files changed, 34 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
> > > index edc0081..dee3417 100644
> > > --- a/drivers/acpi/ec.c
> > > +++ b/drivers/acpi/ec.c
> > > @@ -124,6 +124,35 @@ static int EC_FLAGS_MSI; /* Out-of-spec MSI controller */
> > > static int EC_FLAGS_VALIDATE_ECDT; /* ASUStec ECDTs need to be validated */
> > > static int EC_FLAGS_SKIP_DSDT_SCAN; /* Not all BIOS survive early DSDT scan */
> > >
> > > +/* notifier chain for who are intresting in ec space changing */
> > > +static RAW_NOTIFIER_HEAD(ec_space_chain);
> > > +
> > > +int __ref register_ec_space_notifier(struct notifier_block *nb)
> > > +{
> > > + int ret;
> > > +
> > > + ret = raw_notifier_chain_register(&ec_space_chain, nb);
> > > +
> > > + return ret;
> > > +}
> > > +EXPORT_SYMBOL(register_ec_space_notifier);
> > > +
> > > +void __ref unregister_ec_space_notifier(struct notifier_block *nb)
> > > +{
> > > +
> > > + raw_notifier_chain_unregister(&ec_space_chain, nb);
> > > +}
> > > +EXPORT_SYMBOL(unregister_ec_space_notifier);
> > > +
> > > +static int ec_space_notify(void *data)
> > > +{
> > > + int ret;
> > > +
> > > + ret = __raw_notifier_call_chain(&ec_space_chain, 0, data, -1, NULL);
> > > +
> > > + return notifier_to_errno(ret);
> > > +}
> > > +
> > > /* --------------------------------------------------------------------------
> > > Transaction Management
> > > -------------------------------------------------------------------------- */
> > > @@ -638,6 +667,9 @@ static u32 acpi_ec_gpe_handler(acpi_handle gpe_device,
> > > wake_up(&ec->wait);
> > > ec_check_sci(ec, acpi_ec_read_status(ec));
> > > }
> > > +
> > > + ec_space_notify(data);
> > > +
> > > return ACPI_INTERRUPT_HANDLED | ACPI_REENABLE_GPE;
> > > }
> > >
> > > diff --git a/include/linux/acpi.h b/include/linux/acpi.h
> > > index 17b5b59..4fe2247 100644
> > > --- a/include/linux/acpi.h
> > > +++ b/include/linux/acpi.h
> > > @@ -158,6 +158,8 @@ extern int ec_transaction(u8 command,
> > > const u8 *wdata, unsigned wdata_len,
> > > u8 *rdata, unsigned rdata_len);
> > > extern acpi_handle ec_get_handle(void);
> > > +extern int register_ec_space_notifier(struct notifier_block *nb);
> > > +extern void unregister_ec_space_notifier(struct notifier_block *nb);
> > >
> > > #if defined(CONFIG_ACPI_WMI) || defined(CONFIG_ACPI_WMI_MODULE)
> > >
> > >
>
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/