>>> On 04.12.17 at 16:28, <andrew.coop...@citrix.com> wrote: > On 04/12/17 10:45, Jan Beulich wrote: >> @@ -2477,10 +2478,9 @@ static int _put_page_type(struct page_in >> continue; >> /* We cleared the 'valid bit' so we do the clean up. */ >> rc = _put_final_page_type(page, x, preemptible, ptpg); >> - ptpg = NULL; >> if ( x & PGT_partial ) >> put_page(page); >> - break; > > Newline here.
Hmm, I've added one because I don't really mind, but this isn't the main return point of the function. > Otherwise, Reviewed-by: Andrew Cooper > <andrew.coop...@citix.com> Thanks, Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
