In article <caemhiu6bww4dmrekus-psdjsgtaingpkkkthn88roz43i0e...@mail.gmail.com>, Cherry G. Mathew <[email protected]> wrote: >On 13 August 2011 11:45, Christos Zoulas <[email protected]> wrote: >> In article ><CAEmhiu7cetkMHgK=F1fzt2R9KTb8q0csyEA=1-m+-vwejdn...@mail.gmail.com>, >> Cherry G. Mathew <[email protected]> wrote: >>>On 13 August 2011 11:26, Christos Zoulas <[email protected]> wrote: >>>> In article <[email protected]>, Cherry G. Mathew <[email protected]> >>>> wrote: >>>>>ftp://ftp.netbsd.org/pub/NetBSD/misc/cherry/tmp/wrap_ipi.diff >>>>> >>>>>Hi, >>>>> >>>>>I'd like to ask peoples' opinions about the above patch. This would >>>>>remove a couple of #ifdef XEN/#endif pairs I had to introduce >>>>>lately. >>>> >>>> If those are performance critical, perhaps they should be inlined? >>> >>> >>>I don't think they are - the first is a HALT ipi and the second >>>"kicks" other sleeping CPUS via cpu_needs_resched() >> >> So the first one isn't, but how often is the second one called? >> > >Once per tick at worst.
Then, I guess it is ok. christos
