On 15/08/17 15:39, Jan Beulich wrote: > @@ -422,8 +422,13 @@ get_maptrack_handle( > /* > * If we've run out of frames, try stealing an entry from another > * VCPU (in case the guest isn't mapping across its VCPUs evenly). > + * Also use this path in case we're out of memory, to avoid spurious > + * failures.
This comment isn't strictly correct any more. It is now "If we've run out of handles and still have frame headroom, try allocating a new maptrack frame. If there is no headroom, or Xen is out of memory, try stealing an entry from another vcpu". ~Andrew > */ > - if ( nr_maptrack_frames(lgt) >= max_maptrack_frames ) > + if ( nr_maptrack_frames(lgt) < max_maptrack_frames ) > + new_mt = alloc_xenheap_page(); > + > + if ( !new_mt ) > { > spin_unlock(&lgt->maptrack_lock); > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel