* Re: [tip:x86/apic] x86/ioapic.c: unify ioapic_retrigger_irq() [not found] <tip-e25371d60cb06a44d7a32d7966ab9bfbeacb9390@git.kernel.org> @ 2009-07-22 19:54 ` Jeremy Fitzhardinge 2009-07-22 21:29 ` Eric W. Biederman 0 siblings, 1 reply; 2+ messages in thread From: Jeremy Fitzhardinge @ 2009-07-22 19:54 UTC (permalink / raw) To: linux-kernel@vger.kernel.org, mingo@redhat.com, hpa@zytor.com, tglx@linutronix.de, Jeremy Fitzhardinge Cc: Eric W. Biederman On 07/18/09 05:05, tip-bot for Jeremy Fitzhardinge wrote: > Commit-ID: e25371d60cb06a44d7a32d7966ab9bfbeacb9390 > Gitweb: http://git.kernel.org/tip/e25371d60cb06a44d7a32d7966ab9bfbeacb9390 > Author: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> > AuthorDate: Mon, 8 Jun 2009 03:49:01 -0700 > Committer: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> > CommitDate: Tue, 14 Jul 2009 13:32:51 -0700 > > x86/ioapic.c: unify ioapic_retrigger_irq() > > The 32 and 64-bit versions of ioapic_retrigger_irq() are identical > except the 64-bit one takes vector_lock. vector_lock is defined and > used on 32-bit too, so just use a common ioapic_retrigger_irq(). > Having another look at this patch, the other difference is that the 32-bit version just does: apic->send_IPI_self(irq_cfg(irq)->vector); whereas the 64-bit does: apic->send_IPI_mask(cpumask_of(cpumask_first(cfg->domain)), cfg->vector); Does 32-bit do the whole vector domain thing now? Are these actually equivalent? Sending to self seems like it should be more efficient. J ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [tip:x86/apic] x86/ioapic.c: unify ioapic_retrigger_irq() 2009-07-22 19:54 ` [tip:x86/apic] x86/ioapic.c: unify ioapic_retrigger_irq() Jeremy Fitzhardinge @ 2009-07-22 21:29 ` Eric W. Biederman 0 siblings, 0 replies; 2+ messages in thread From: Eric W. Biederman @ 2009-07-22 21:29 UTC (permalink / raw) To: Jeremy Fitzhardinge Cc: linux-kernel@vger.kernel.org, mingo@redhat.com, hpa@zytor.com, tglx@linutronix.de, Jeremy Fitzhardinge, Yinghai Lu Jeremy Fitzhardinge <jeremy@goop.org> writes: > On 07/18/09 05:05, tip-bot for Jeremy Fitzhardinge wrote: >> Commit-ID: e25371d60cb06a44d7a32d7966ab9bfbeacb9390 >> Gitweb: http://git.kernel.org/tip/e25371d60cb06a44d7a32d7966ab9bfbeacb9390 >> Author: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> >> AuthorDate: Mon, 8 Jun 2009 03:49:01 -0700 >> Committer: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> >> CommitDate: Tue, 14 Jul 2009 13:32:51 -0700 >> >> x86/ioapic.c: unify ioapic_retrigger_irq() >> >> The 32 and 64-bit versions of ioapic_retrigger_irq() are identical >> except the 64-bit one takes vector_lock. vector_lock is defined and >> used on 32-bit too, so just use a common ioapic_retrigger_irq(). >> > > Having another look at this patch, the other difference is that the > 32-bit version just does: > > apic->send_IPI_self(irq_cfg(irq)->vector); > > whereas the 64-bit does: > > apic->send_IPI_mask(cpumask_of(cpumask_first(cfg->domain)), > cfg->vector); > > > Does 32-bit do the whole vector domain thing now? Are these actually > equivalent? Sending to self seems like it should be more efficient. It should. I believe YH did that work when he merged the x86_64 and i386 versions io_apic.c. Sending to self is a problem if we retrigger this from the wrong irq. Eric ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-07-22 21:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <tip-e25371d60cb06a44d7a32d7966ab9bfbeacb9390@git.kernel.org>
2009-07-22 19:54 ` [tip:x86/apic] x86/ioapic.c: unify ioapic_retrigger_irq() Jeremy Fitzhardinge
2009-07-22 21:29 ` Eric W. Biederman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox