[Xen-devel] [PATCH 0/3] datacopier POLLHUP fixes handling when the fd is also readable

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Andrew Cooper writes ("Re: [Xen-devel] [PATCH v3 6/6] tools/libxl: Adjust 
datacopiers POLLHUP handling when the fd is also readable"):
> > 
> > libxl: error: libxl_bootloader.c:596:bootloader_copyfail:
> > unexpected eof copying bootloader output
> > 
> > libxl: debug: libxl_bootloader.c:638:bootloader_finished:
> > bootloader completed

This shows that the problem is with 7e9ec50b0535.  The bootloader code
relies on the difference between POLLHUP[|POLLIN] and just POLLIN.

I think this small series will fix the problem.  The commit messages
have more detailed explanations.

Note that I have NOT EXECUTED this code.  I have neither a FreeBSD
dom0 nor the migration v2 patches to hand.

I propose to push 1/3 (the revert of 7e9ec50b0535) immediately.
2/3 and 3/3 should await testing.

Andrew, do you have the capability to test 2/3 ?

3/3 is hard to test because the condition is probably very hard to
reproduce.  I will see if I can have a go.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel