Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1268355
| From | Kevin Hilman <khilman@kernel.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips |
| Date | 2015-11-13 00:30 +0100 |
| Message-ID | <qu9tx-71V-59@gated-at.bofh.it> (permalink) |
| References | <qtifw-6eV-3@gated-at.bofh.it> <qtipc-6iz-15@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
Jon Hunter <jonathanh@nvidia.com> writes: > Some IRQ chips may be located in a power domain outside of the CPU subsystem > and hence will require device specific runtime power management. Ideally, > rather than adding more functions to the irq_chip_ops function table, using > existing chip functions such as irq_startup/shutdown or > irq_request/release_resources() would be best. However, these existing chip > functions are called in the context of a spinlock which is not ideal for > power management operations that may take some time to power up a domain. > > Two possible solutions are: > 1. Move existing chip operators such as irq_request/release_resources() > outside of the spinlock and use these helpers. > 2. Add new chip operators that are called outside of any spinlocks while > setting up and freeing an IRQ. > Not knowing whether we can safely move irq_request/release_resources() to > outside the spinlock (but hopefully this will solicit some feedback), add > new chip operators for runtime resuming and suspending of an IRQ chip. I'm not quite seeing how this would connect to the actual hardware power domain (presumabaly managed by genpd) and any other devices in that domain (presumably managed by runtime PM.) If all the RPM devices in the domain go idle, it will be powered off independently of the status of the irqchip because the irqchip isn't using RPM. Is there a longer-term plan to handle the irqchips as a "normal" device and use RPM? IMO, that approach would be helpful even for irqchips that share power domains with CPUs, since there are efforts working towards using genpd/RPM to manage CPUs/clusters. Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[RFC PATCH 0/2] Add support for Tegra210 AGIC Jon Hunter <jonathanh@nvidia.com> - 2015-11-10 15:40 +0100
[RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-10 15:50 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Thomas Gleixner <tglx@linutronix.de> - 2015-11-10 16:30 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-10 17:00 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Grygorii Strashko <grygorii.strashko@ti.com> - 2015-11-10 17:50 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Lars-Peter Clausen <lars@metafoo.de> - 2015-11-10 19:10 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-11 11:20 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Grygorii Strashko <grygorii.strashko@ti.com> - 2015-11-11 16:50 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-12 12:00 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Lars-Peter Clausen <lars@metafoo.de> - 2015-11-12 14:30 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-12 14:40 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Lars-Peter Clausen <lars@metafoo.de> - 2015-11-12 14:50 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-12 15:10 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Lars-Peter Clausen <lars@metafoo.de> - 2015-11-12 15:40 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-12 16:40 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Grygorii Strashko <grygorii.strashko@ti.com> - 2015-11-12 14:40 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Lars-Peter Clausen <lars@metafoo.de> - 2015-11-12 14:40 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Grygorii Strashko <grygorii.strashko@ti.com> - 2015-11-13 19:10 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Sören Brinkmann <soren.brinkmann@xilinx.com> - 2015-11-12 18:40 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Kevin Hilman <khilman@kernel.org> - 2015-11-13 00:30 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-13 10:10 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Thomas Gleixner <tglx@linutronix.de> - 2015-11-13 21:10 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-16 10:50 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Geert Uytterhoeven <geert@linux-m68k.org> - 2015-11-16 10:50 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Jon Hunter <jonathanh@nvidia.com> - 2015-11-16 11:40 +0100
Re: [RFC PATCH 1/2] genirq: Add runtime resume/suspend support for IRQ chips Geert Uytterhoeven <geert@linux-m68k.org> - 2015-11-16 11:50 +0100
csiph-web