Ross Lagerwall writes ("Re: [PATCH] tools: xentoolcore_restrict_all: Do
deregistration before close"):
> On 11/14/2017 12:15 PM, Ian Jackson wrote:
> > + * Note for multi-threaded programs: If xentoolcore_restrict_all is
> > + * called concurrently with a function which /or closes Xen library
>
> "which /or closes..." - Is this a typo?
Yes, fixed, thanks.
> > - close(h->fd);
> > xentoolcore__deregister_active_handle(&h->tc_ah);
> > + close(h->fd);
> >
>
> Since the rest of this file uses tabs, you may as well use tabs for this
> line as well.
I didn't change the use of tabs vs. the use of spaces.
> Reviewed-by: Ross Lagerwall <ross.lagerw...@citrix.com>
Thanks,
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
