On Tue, 18 Dec 2007 01:00:15 -0500 Kyle McMartin <kyle@xxxxxxxxxxx> wrote: > On Tue, Dec 18, 2007 at 10:05:45AM +0900, KAMEZAWA Hiroyuki wrote: > > On Thu, 13 Dec 2007 15:03:07 +0000 > > > > > + if (mm != active_mm) { > > > + /* Restore region IDs for mm */ > > > + if (mm && active_mm) { > > > + activate_context(mm); > > > + } else { > > > + flush_tlb_all(); > > > + return; > > > + } > > > } > > should be > > > > platform_global_tlb_purge is already (and afaict, only) called under > preempt_disable already. then again, the sn2 global_tlb_purge > does it, so possibly for completeness sake it should be added here as > well? > Thank you. I see. flush_tlb_range() is the only caller. ...maybe adding comment is helpful (for me :). Thanks, -Kame - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |