From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@linaro.org (Kevin Hilman) Date: Tue, 29 Oct 2013 11:55:53 -0700 Subject: [PATCH] ARM: Support arch_irq_work_raise() via self IPIs In-Reply-To: <20131029060058.GD21983@codeaurora.org> (Stephen Boyd's message of "Mon, 28 Oct 2013 23:00:58 -0700") References: <526DBEAA.4030707@linaro.org> <1382984701-2371-1-git-send-email-sboyd@codeaurora.org> <878uxcq1m7.fsf@linaro.org> <20131029060058.GD21983@codeaurora.org> Message-ID: <874n7zq53a.fsf@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Stephen Boyd writes: > On 10/28, Kevin Hilman wrote: >> Stephen Boyd writes: >> >> > This will allow the scheduler tick to be restarted if we're in >> > full NOHZ mode. >> > >> > Cc: Kevin Hilman >> > Cc: Frederic Weisbecker >> > Signed-off-by: Stephen Boyd >> >> Minor nit, but I'd prefer a more verbose changelog (I forget things >> quickly and like to rely on changelogs for my memory.) Probably worth >> adding something like: "By default, irq_work is tied to the tick >> processing (update_process_times()) but in full NOHZ mode, no tick means >> no IRQ work. In order for IRQ work to be done in full NOHZ mode, a >> self-IPI is used to process IRQ work." >> >> Other than the changelog nit, patch looks good, feel free to add >> >> Reviewed-by: Kevin Hilman >> >> If Russell is OK with this, it can go to his patch system. >> > > Fair enough. This is what I came up with. I'll send it off to the > patch tracker in about 12 hours if nobody else has anymore > comments. Nice description, thanks for the respin. Kevin