From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?Um9nZXIgUGF1IE1vbm7DqQ==?= Subject: Re: [PATCH v3 6/6] tools/libxl: Adjust datacopiers POLLHUP handling when the fd is also readable Date: Wed, 1 Apr 2015 12:34:24 +0200 Message-ID: <551BC9B0.10000@citrix.com> References: <1426512594-11585-1-git-send-email-ross.lagerwall@citrix.com> <1426512594-11585-6-git-send-email-ross.lagerwall@citrix.com> <551423A8.3000308@citrix.com> <1427712044.13935.284.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1427712044.13935.284.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Ross Lagerwall , Andrew Cooper , Ian Jackson , Wei Liu , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org SGVsbG8sCgpFbCAzMC8wMy8xNSBhIGxlcyAxMi40MCwgSWFuIENhbXBiZWxsIGhhIGVzY3JpdDoK PiBPbiBUaHUsIDIwMTUtMDMtMjYgYXQgMTY6MjAgKzAxMDAsIFJvZ2VyIFBhdSBNb25uw6kgd3Jv dGU6Cj4+IEVsIDE2LzAzLzE1IGEgbGVzIDE0LjI5LCBSb3NzIExhZ2Vyd2FsbCBoYSBlc2NyaXQ6 Cj4+PiBGcm9tOiBBbmRyZXcgQ29vcGVyIDxhbmRyZXcuY29vcGVyM0BjaXRyaXguY29tPgo+Pj4K Pj4+IFBPTExIVVB8UE9MTElOIGlzIGEgdmFsaWQgcmV2ZW50IHRvIHJlY2VpdmUgd2hlbiB0aGVy ZSBpcyByZWFkYWJsZSBkYXRhIGluIGEKPj4+IHBpcGUsIGJ1dCB0aGUgd3JpdGFibGUgZmQgaGFz IGJlZW4gY2xvc2VkLiAgVGhpcyBvY2N1cnMgaW4gbWlncmF0aW9uIHYyIHdoZW4KPj4+IHRoZSBs ZWdhY3kgY29udmVyc2lvbiBwcm9jZXNzICh3aGljaCB0cmFuc2Zvcm1zIHRoZSBkYXRhIGlubGlu ZSkgY29tcGxldGVzIGFuZAo+Pj4gZXhpdHMgc3VjY2Vzc2Z1bGx5Lgo+Pj4KPj4+IEluIHRoZSBj YXNlIHRoYXQgdGhlcmUgaXMgZGF0YSB0byByZWFkLCBzdXBwcmVzcyB0aGUgUE9MTEhVUC4gIFBP U0lYIHN0YXRlcwo+Pj4gdGhhdCB0aGUgaGFuZ3VwIHN0YXRlIGlzIGxhdGNoZWRbMV0sIHdoaWNo IG1lYW5zIGl0IHdpbGwgcmVvY2N1ciBvbiBzdWJzZXF1ZW50Cj4+PiBwb2xsKCkgY2FsbHMuICBU aGUgZGF0YWNvcGllciBpcyB0aHVzIHByb3ZpZGVkIHRoZSBvcHBvcnR1bml0eSB0byByZWFkIHVu dGlsCj4+PiBFT0YsIGlmIHBvc3NpYmxlLgo+Pj4KPj4+IEEgUE9MTEhVUCBvbiBpdHMgb3duIGlz IHRyZWF0ZWQgZXhhY3RseSBhcyBiZWZvcmUsIGluZGljYXRpbmcgYSBkaWZmZXJlbnQKPj4+IGVy cm9yIHdpdGggdGhlIGZkLgo+Pj4KPj4+IFsxXSBodHRwOi8vcHVicy5vcGVuZ3JvdXAub3JnL29u bGluZXB1YnMvOTY5OTkxOTc5OS9mdW5jdGlvbnMvcG9sbC5odG1sCj4+Cj4+IFRoaXMgcGF0Y2gg YnJlYWtzIGJvb3RpbmcgcHlncnViIGd1ZXN0cyBvbiBGcmVlQlNELCBpdCBzZWVtcyB0byB0cmln Z2VyIAo+PiBzb21lIGtpbmQgb2YgZXJyb3Igd2hlbiBweWdydWIgZXhpdHM6Cj4+Cj4+IGxpYnhs OiBlcnJvcjogbGlieGxfYm9vdGxvYWRlci5jOjU5Njpib290bG9hZGVyX2NvcHlmYWlsOiB1bmV4 cGVjdGVkIGVvZiBjb3B5aW5nIGJvb3Rsb2FkZXIgb3V0cHV0Cj4+IGxpYnhsOiBlcnJvcjogbGli eGxfY3JlYXRlLmM6MTEyOTpkb21jcmVhdGVfcmVidWlsZF9kb25lOiBjYW5ub3QgKHJlLSlidWls ZCBkb21haW46IC0zCj4+Cj4+IEkgd2lsbCB0cnkgdG8gbG9vayBpbnRvIGl0LCBoYXZlIHlvdSB0 ZXN0ZWQgaWYgdGhlIHNhbWUgaGFwcGVucyBvbiAKPj4gTGludXg/Cj4gCj4gUm9zcy9BbmRyZXcs IGhhdmUgeW91IHRyaWVkIHRoaXMgb24gTGludXg/Cj4gCj4gUm9nZXIsIERvZXMgeGwgLXZ2diBv ciBldmVuIHN0cmFjZSBnaXZlIGFueSBjbHVlcyBhcyB0byB0aGUgc2VxdWVuY2VzIG9mCj4gZXZl bnRzIHdoaWNoIGxlYWQgdG8gdGhlIGVycm9yPwoKSGVyZSBpcyB0aGUgb3V0cHV0IG9mIHhsIC12 dnYsIGl0IGRvZXNuJ3Qgc2VlbSB0byBjb250YWluIGFueSB1c2VmdWwgCmluZm8gYXBhcnQgZnJv bSB3aGF0IEkndmUgYWxyZWFkeSBwb3N0ZWQ6CgojIHhsIC12dnYgY3JlYXRlIGRlYmlhbi5jZmcK UGFyc2luZyBjb25maWcgZnJvbSBkZWJpYW4uY2ZnCmxpYnhsOiBkZWJ1ZzogbGlieGxfY3JlYXRl LmM6MTUyMTpkb19kb21haW5fY3JlYXRlOiBhbyAweDgwMjgzNDA4MDogY3JlYXRlOiBob3c9MHgw IGNhbGxiYWNrPTB4MCBwb2xsZXI9MHg4MDI4MWYwYTAKbGlieGw6IGRlYnVnOiBsaWJ4bF9kZXZp Y2UuYzoyNjk6bGlieGxfX2RldmljZV9kaXNrX3NldF9iYWNrZW5kOiBEaXNrIHZkZXY9eHZkYSBz cGVjLmJhY2tlbmQ9dW5rbm93bgpsaWJ4bDogZGVidWc6IGxpYnhsX2RldmljZS5jOjI5ODpsaWJ4 bF9fZGV2aWNlX2Rpc2tfc2V0X2JhY2tlbmQ6IERpc2sgdmRldj14dmRhLCB1c2luZyBiYWNrZW5k IHBoeQpsaWJ4bDogZGVidWc6IGxpYnhsX2NyZWF0ZS5jOjkyNDppbml0aWF0ZV9kb21haW5fY3Jl YXRlOiBydW5uaW5nIGJvb3Rsb2FkZXIKbGlieGw6IGRlYnVnOiBsaWJ4bF9kZXZpY2UuYzoyNjk6 bGlieGxfX2RldmljZV9kaXNrX3NldF9iYWNrZW5kOiBEaXNrIHZkZXY9KG51bGwpIHNwZWMuYmFj a2VuZD1waHkKbGlieGw6IGRlYnVnOiBsaWJ4bC5jOjMwNjU6bGlieGxfX2RldmljZV9kaXNrX2xv Y2FsX2luaXRpYXRlX2F0dGFjaDogbG9jYWxseSBhdHRhY2hpbmcgUEhZIGRpc2sgL2Rldi96dm9s L3RhbmsvZGViaWFuCmxpYnhsOiBkZWJ1ZzogbGlieGxfYm9vdGxvYWRlci5jOjQxMTpib290bG9h ZGVyX2Rpc2tfYXR0YWNoZWRfY2I6IENvbmZpZyBib290bG9hZGVyIHZhbHVlOiBweWdydWIKbGli eGw6IGRlYnVnOiBsaWJ4bF9ib290bG9hZGVyLmM6NDI3OmJvb3Rsb2FkZXJfZGlza19hdHRhY2hl ZF9jYjogQ2hlY2tpbmcgZm9yIGJvb3Rsb2FkZXIgaW4gbGliZXhlYyBwYXRoOiAvdXNyL2xvY2Fs L2xpYi94ZW4vYmluL3B5Z3J1YgpsaWJ4bDogZGVidWc6IGxpYnhsX2NyZWF0ZS5jOjE1Mzc6ZG9f ZG9tYWluX2NyZWF0ZTogYW8gMHg4MDI4MzQwODA6IGlucHJvZ3Jlc3M6IHBvbGxlcj0weDgwMjgx ZjBhMCwgZmxhZ3M9aQpsaWJ4bDogZGVidWc6IGxpYnhsX2V2ZW50LmM6NTc3OmxpYnhsX19ldl94 c3dhdGNoX3JlZ2lzdGVyOiB3YXRjaCB3PTB4ODAyODM2N2EwIHdwYXRoPS9sb2NhbC9kb21haW4v MiB0b2tlbj0zLzA6IHJlZ2lzdGVyIHNsb3RudW09MwpsaWJ4bDogZGVidWc6IGxpYnhsX2V2ZW50 LmM6MTk0MjpsaWJ4bF9fYW9fcHJvZ3Jlc3NfcmVwb3J0OiBhbyAweDgwMjgzNDA4MDogcHJvZ3Jl c3MgcmVwb3J0OiBpZ25vcmVkCmxpYnhsOiBkZWJ1ZzogbGlieGxfYm9vdGxvYWRlci5jOjUzOTpi b290bG9hZGVyX2dvdHB0eXM6IGV4ZWN1dGluZyBib290bG9hZGVyOiAvdXNyL2xvY2FsL2xpYi94 ZW4vYmluL3B5Z3J1YgpsaWJ4bDogZGVidWc6IGxpYnhsX2Jvb3Rsb2FkZXIuYzo1NDM6Ym9vdGxv YWRlcl9nb3RwdHlzOiAgIGJvb3Rsb2FkZXIgYXJnOiAvdXNyL2xvY2FsL2xpYi94ZW4vYmluL3B5 Z3J1YgpsaWJ4bDogZGVidWc6IGxpYnhsX2Jvb3Rsb2FkZXIuYzo1NDM6Ym9vdGxvYWRlcl9nb3Rw dHlzOiAgIGJvb3Rsb2FkZXIgYXJnOiAtLW91dHB1dD0vdmFyL3J1bi94ZW4vYm9vdGxvYWRlci4y Lm91dApsaWJ4bDogZGVidWc6IGxpYnhsX2Jvb3Rsb2FkZXIuYzo1NDM6Ym9vdGxvYWRlcl9nb3Rw dHlzOiAgIGJvb3Rsb2FkZXIgYXJnOiAtLW91dHB1dC1mb3JtYXQ9c2ltcGxlMApsaWJ4bDogZGVi dWc6IGxpYnhsX2Jvb3Rsb2FkZXIuYzo1NDM6Ym9vdGxvYWRlcl9nb3RwdHlzOiAgIGJvb3Rsb2Fk ZXIgYXJnOiAtLW91dHB1dC1kaXJlY3Rvcnk9L3Zhci9ydW4veGVuL2Jvb3Rsb2FkZXIuMi5kCmxp YnhsOiBkZWJ1ZzogbGlieGxfYm9vdGxvYWRlci5jOjU0Mzpib290bG9hZGVyX2dvdHB0eXM6ICAg Ym9vdGxvYWRlciBhcmc6IC9kZXYvenZvbC90YW5rL2RlYmlhbgpsaWJ4bDogZGVidWc6IGxpYnhs X2V2ZW50LmM6NTE0OndhdGNoZmRfY2FsbGJhY2s6IHdhdGNoIHc9MHg4MDI4MzY3YTAgd3BhdGg9 L2xvY2FsL2RvbWFpbi8yIHRva2VuPTMvMDogZXZlbnQgZXBhdGg9L2xvY2FsL2RvbWFpbi8yCmxp YnhsOiBlcnJvcjogbGlieGxfYm9vdGxvYWRlci5jOjU5Njpib290bG9hZGVyX2NvcHlmYWlsOiB1 bmV4cGVjdGVkIGVvZiBjb3B5aW5nIGJvb3Rsb2FkZXIgb3V0cHV0CmxpYnhsOiBkZWJ1ZzogbGli eGxfYm9vdGxvYWRlci5jOjYzODpib290bG9hZGVyX2ZpbmlzaGVkOiBib290bG9hZGVyIGNvbXBs ZXRlZApsaWJ4bDogZGVidWc6IGxpYnhsX2V2ZW50LmM6NjE1OmxpYnhsX19ldl94c3dhdGNoX2Rl cmVnaXN0ZXI6IHdhdGNoIHc9MHg4MDI4MzY3YTAgd3BhdGg9L2xvY2FsL2RvbWFpbi8yIHRva2Vu PTMvMDogZGVyZWdpc3RlciBzbG90bnVtPTMKbGlieGw6IGVycm9yOiBsaWJ4bF9jcmVhdGUuYzox MTM4OmRvbWNyZWF0ZV9yZWJ1aWxkX2RvbmU6IGNhbm5vdCAocmUtKWJ1aWxkIGRvbWFpbjogLTMK bGlieGw6IGluZm86IGxpYnhsLmM6MTY5OTpkZXZpY2VzX2Rlc3Ryb3lfY2I6IGZvcmtlZCBwaWQg NzYzIGZvciBkZXN0cm95IG9mIGRvbWFpbiAyCmxpYnhsOiBkZWJ1ZzogbGlieGxfZXZlbnQuYzox NzY2OmxpYnhsX19hb19jb21wbGV0ZTogYW8gMHg4MDI4MzQwODA6IGNvbXBsZXRlLCByYz0tMwps aWJ4bDogZGVidWc6IGxpYnhsX2V2ZW50LmM6MTczODpsaWJ4bF9fYW9fX2Rlc3Ryb3k6IGFvIDB4 ODAyODM0MDgwOiBkZXN0cm95CnhjOiBkZWJ1ZzogaHlwZXJjYWxsIGJ1ZmZlcjogdG90YWwgYWxs b2NhdGlvbnM6NDQgdG90YWwgcmVsZWFzZXM6NDQKeGM6IGRlYnVnOiBoeXBlcmNhbGwgYnVmZmVy OiBjdXJyZW50IGFsbG9jYXRpb25zOjAgbWF4aW11bSBhbGxvY2F0aW9uczoyCnhjOiBkZWJ1Zzog aHlwZXJjYWxsIGJ1ZmZlcjogY2FjaGUgY3VycmVudCBzaXplOjIKeGM6IGRlYnVnOiBoeXBlcmNh bGwgYnVmZmVyOiBjYWNoZSBoaXRzOjM1IG1pc3NlczoyIHRvb2JpZzo3CgpBbmQgdGhpcyBpcyAo SSB0aGluaykgdGhlIHJlbGV2YW50IG91dHB1dCBmcm9tIGt0cmFjZS9rZHVtcDoKCiAgIDc2NiB4 bCAgICAgICBSRVQgICByZWFkIDIwLzB4MTQKICAgNzY2IHhsICAgICAgIENBTEwgIHJlYWQoMHhj LDB4ODAyODA3MDJmLDB4M2NkKQogICA3NjYgeGwgICAgICAgUkVUICAgcmVhZCAtMSBlcnJubyAz NSBSZXNvdXJjZSB0ZW1wb3JhcmlseSB1bmF2YWlsYWJsZQogICA3NjYgeGwgICAgICAgQ0FMTCAg Z2V0dGltZW9mZGF5KDB4N2ZmZmZmZmZkZTkwLDApCiAgIDc2NiB4bCAgICAgICBSRVQgICBnZXR0 aW1lb2ZkYXkgMAogICA3NjYgeGwgICAgICAgQ0FMTCAgcG9sbCgweDgwMjgxNjE4MCwweDYsMHhm ZmZmZmZmZikKICAgNzY2IHhsICAgICAgIFJFVCAgIHBvbGwgMQogICA3NjYgeGwgICAgICAgQ0FM TCAgZ2V0dGltZW9mZGF5KDB4N2ZmZmZmZmZkZTkwLDApCiAgIDc2NiB4bCAgICAgICBSRVQgICBn ZXR0aW1lb2ZkYXkgMAogICA3NjYgeGwgICAgICAgQ0FMTCAgd3JpdGUoMHgxMCwweDgwMjgwNzAx NCwweDFiKQogICA3NjYgeGwgICAgICAgR0lPICAgZmQgMTYgd3JvdGUgMjcgYnl0ZXMKICAgICAg IDB4MDAwMCAxYjViIDNmMzEgNmMxYiAzZTFiIDViMzIgMzQzYiAzMTQ4IDAwMDAgMDAwMCAwMDBk IDFiNWIgIHwuWz8xbC4+LlsyNDsxSC4uLi4uLi5bfAogICAgICAgMHgwMDE2IDNmMzEgNmMxYiAz ZSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgfD8xbC4+fAoKICAg NzY2IHhsICAgICAgIFJFVCAgIHdyaXRlIDI3LzB4MWIKICAgNzY2IHhsICAgICAgIENBTEwgIGdl dHRpbWVvZmRheSgweDdmZmZmZmZmZGU5MCwwKQogICA3NjYgeGwgICAgICAgUkVUICAgZ2V0dGlt ZW9mZGF5IDAKICAgNzY2IHhsICAgICAgIENBTEwgIHBvbGwoMHg4MDI4MTYxODAsMHg1LDB4ZmZm ZmZmZmYpCiAgIDc2NiB4bCAgICAgICBSRVQgICBwb2xsIDEKICAgNzY2IHhsICAgICAgIFBTSUcg IFNJR0NITEQgY2F1Z2h0IGhhbmRsZXI9MHg4MDExNGJkZjAgbWFzaz0weDAgY29kZT1DTERfRVhJ VEVECiAgIDc2NiB4bCAgICAgICBDQUxMICBzaWdwcm9jbWFzayhTSUdfU0VUTUFTSywweDdmZmZm ZmZmZDgxYywwKQogICA3NjYgeGwgICAgICAgUkVUICAgc2lncHJvY21hc2sgMAogICA3NjYgeGwg ICAgICAgQ0FMTCAgd3JpdGUoMHhlLDB4ODAwYWYxZmQyLDB4MSkKICAgNzY2IHhsICAgICAgIEdJ TyAgIGZkIDE0IHdyb3RlIDEgYnl0ZQogICAgICAgIlwwIgogICA3NjYgeGwgICAgICAgUkVUICAg d3JpdGUgMQogICA3NjYgeGwgICAgICAgQ0FMTCAgc2lncmV0dXJuKDB4N2ZmZmZmZmZkNDQwKQog ICA3NjYgeGwgICAgICAgUkVUICAgc2lncmV0dXJuIEpVU1RSRVRVUk4KICAgNzY2IHhsICAgICAg IENBTEwgIGdldHRpbWVvZmRheSgweDdmZmZmZmZmZGU5MCwwKQogICA3NjYgeGwgICAgICAgUkVU ICAgZ2V0dGltZW9mZGF5IDAKICAgNzY2IHhsICAgICAgIENBTEwgIHJlYWQoMHhjLDB4ODAyODA3 MDE0LDB4M2U4KQogICA3NjYgeGwgICAgICAgR0lPICAgZmQgMTIgcmVhZCAwIGJ5dGVzCiAgICAg ICAiIgogICA3NjYgeGwgICAgICAgUkVUICAgcmVhZCAwCiAgIDc2NiB4bCAgICAgICBDQUxMICB3 cml0ZSgweDIsMHg3ZmZmZmZmZmQwYjAsMHg3KQogICA3NjYgeGwgICAgICAgR0lPICAgZmQgMiB3 cm90ZSA3IGJ5dGVzCiAgICAgICAibGlieGw6ICIKICAgNzY2IHhsICAgICAgIFJFVCAgIHdyaXRl IDcKICAgNzY2IHhsICAgICAgIENBTEwgIHdyaXRlKDB4MiwweDdmZmZmZmZmZDBiMCwweDcpCiAg IDc2NiB4bCAgICAgICBHSU8gICBmZCAyIHdyb3RlIDcgYnl0ZXMKICAgICAgICJlcnJvcjogIgoK PiBEaWQvZG9lcyBjb21taXQgOTE2NzM1ZTE4MTRkICJ0b29scy9saWJ4bDogUmVzdG9yZSBlcnJu b3ZhbCBiZWhhdmlvciBmb3IKPiBkYXRhY29waWVyIGNhbGxiYWNrIiBtYWtlIGFueSBkaWZmZXJl bmNlPwoKTm8sIEkgc3RpbGwgZ2V0IHRoZSBzYW1lIGVycm9yLiBJIHdpbGwgdHJ5IHRvIHRha2Ug YSBjbG9zZXIgbG9vayBpbnRvIHRoaXMgCndoZW4gSSBmaW5pc2ggc29tZSBwZW5kaW5nIHN0dWZm Li4uCgpSb2dlci4KCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwpYZW4tZGV2ZWwgbWFpbGluZyBsaXN0Clhlbi1kZXZlbEBsaXN0cy54ZW4ub3JnCmh0dHA6 Ly9saXN0cy54ZW4ub3JnL3hlbi1kZXZlbAo=