lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 24 Oct 2021 17:31:00 +0200 From: Thomas Bogendoerfer <tsbogend@...ha.franken.de> To: Mark Rutland <mark.rutland@....com> Cc: linux-kernel@...r.kernel.org, aou@...s.berkeley.edu, catalin.marinas@....com, deanbo422@...il.com, green.hu@...il.com, guoren@...nel.org, jonas@...thpole.se, kernelfans@...il.com, linux-arm-kernel@...ts.infradead.org, linux@...linux.org.uk, maz@...nel.org, nickhu@...estech.com, palmer@...belt.com, paulmck@...nel.org, paul.walmsley@...ive.com, peterz@...radead.org, shorne@...il.com, stefan.kristiansson@...nalahti.fi, tglx@...utronix.de, torvalds@...ux-foundation.org, vgupta@...nel.org, will@...nel.org Subject: Re: [PATCH 03/15] irq: mips: simplify do_domain_IRQ() On Thu, Oct 21, 2021 at 07:02:24PM +0100, Mark Rutland wrote: > There's no need fpr arch/mips's do_domain_IRQ() to open-code the NULL > check performed by handle_irq_desc(), nor the resolution of the desc > performed by generic_handle_domain_irq(). > > Use generic_handle_domain_irq() directly, as this is functioanlly > equivalent and clearer. > > There should be no functional change as a result of this patch. > > Signed-off-by: Mark Rutland <mark.rutland@....com> > Cc: Marc Zyngier <maz@...nel.org> > Cc: Thomas Bogendoerfer <tsbogend@...ha.franken.de> > Cc: Thomas Gleixner <tglx@...utronix.de> > --- > arch/mips/kernel/irq.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/arch/mips/kernel/irq.c b/arch/mips/kernel/irq.c > index d20e002b3246..1fee96ef8059 100644 > --- a/arch/mips/kernel/irq.c > +++ b/arch/mips/kernel/irq.c > @@ -115,11 +115,7 @@ void __irq_entry do_domain_IRQ(struct irq_domain *domain, unsigned int hwirq) > > irq_enter(); > check_stack_overflow(); > - > - desc = irq_resolve_mapping(domain, hwirq); > - if (likely(desc)) > - handle_irq_desc(desc); > - > + generic_handle_domain_irq(domain, hwirq); > irq_exit(); > } > #endif > -- > 2.11.0 Acked-by: Thomas Bogendoerfer <tsbogend@...ha.franken.de> -- Crap can work. Given enough thrust pigs will fly, but it's not necessarily a good idea. [ RFC1925, 2.3 ]
Powered by blists - more mailing lists