On Wed, Apr 13, 2016 at 10:07:18PM -0600, Jan Beulich wrote: > >>> Konrad Rzeszutek Wilk <konrad.w...@oracle.com> 04/14/16 12:03 AM >>> > >- /* If we are done with the L2E, check if it is now empty. */ > >- if ( (v != e) && (l1_table_offset(v) != 0) ) > >+ /* > >+ * If we not destroying mappings, or are not done with the L2E, > >+ * skip the empty&free check. > > It would have been really nice if you had added the missing "are" ... > > >- /* If we are done with the L3E, check if it is now empty. */ > >- if ( (v != e) && (l2_table_offset(v) + l1_table_offset(v) != 0) ) > >+ /* > >+ * If we are not destroying mappings, or not done with the L3E, > >+ * skip the empty&free check. > > ... just like you did here.
<smack his head> Yes. Fixed! > > Jan > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
