[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2] x86/mm: also flush TLB when putting writable foreign page reference
At 03:23 -0600 on 27 Apr (1493263380), Jan Beulich wrote: > ... it wouldn't better be the other way around: We use the patch > in its current (or even v1) form, and try to do something about > performance only if we really find a case where it matters. To be > honest, I'm not even sure how I could meaningfully measure the > impact here: Simply counting how many extra flushes there would > end up being wouldn't seem all that useful, and whether there > would be any measurable difference in the overall execution time > of e.g. domain creation I would highly doubt (but if it's that what > you're after, I could certainly collect a few numbers). I think that would be a good idea, just as a sanity-check. But apart from that the patch looks correct to me, so: Reviewed-by: Tim Deegan <tim@xxxxxxx> for v2 (not v1). Cheers, Tim. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |