From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: Re: [PATCH 3/7] oxenstored: refactor request processing Date: Thu, 24 Mar 2016 19:57:30 -0400 Message-ID: <56F47EEA.3070103@oracle.com> References: <1458237075-13777-1-git-send-email-jonathan.davies@citrix.com> <1458237075-13777-4-git-send-email-jonathan.davies@citrix.com> <56F468A5.9080609@oracle.com> <56F46EFA.9060204@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ajF86-0000dm-SF for xen-devel@lists.xenproject.org; Thu, 24 Mar 2016 23:57:30 +0000 In-Reply-To: <56F46EFA.9060204@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Andrew Cooper , Jonathan Davies , xen-devel@lists.xenproject.org Cc: Jon Ludlam , Euan Harris , Dave Scott List-Id: xen-devel@lists.xenproject.org T24gMDMvMjQvMjAxNiAwNjo0OSBQTSwgQW5kcmV3IENvb3BlciB3cm90ZToKPiBPbiAyNC8wMy8y MDE2IDIyOjIyLCBCb3JpcyBPc3Ryb3Zza3kgd3JvdGU6Cj4+IE9uIDAzLzE3LzIwMTYgMDE6NTEg UE0sIEpvbmF0aGFuIERhdmllcyB3cm90ZToKPj4+IEVuY2Fwc3VsYXRlIHRoZSByZXF1ZXN0IGlu IGEgcmVjb3JkIHRoYXQgaXMgcGFzc2VkIGZyb20gZG9faW5wdXQgdG8KPj4+IHByb2Nlc3NfcGFj a2V0IGFuZCBpbnB1dF9oYW5kbGVfZXJyb3IuCj4+Pgo+Pj4gVGhpcyB3aWxsIGJlIGhlbHBmdWwg d2hlbiBrZWVwaW5nIHRyYWNrIG9mIHRoZSByZXF1ZXN0cyBtYWRlIGFzIHBhcnQKPj4+IG9mIGEK Pj4+IHRyYW5zYWN0aW9uLgo+Pj4KPj4+IFNpZ25lZC1vZmYtYnk6IEpvbmF0aGFuIERhdmllcyA8 am9uYXRoYW4uZGF2aWVzQGNpdHJpeC5jb20+Cj4+PiBSZXZpZXdlZC1ieTogQW5kcmV3IENvb3Bl ciA8YW5kcmV3LmNvb3BlcjNAY2l0cml4LmNvbT4KPj4+IFJldmlld2VkLWJ5OiBKb24gTHVkbGFt IDxqb25hdGhhbi5sdWRsYW1AY2l0cml4LmNvbT4KPj4+IFJldmlld2VkLWJ5OiBFdWFuIEhhcnJp cyA8ZXVhbi5oYXJyaXNAY2l0cml4LmNvbT4KPj4+IC0tLQo+Pj4gICAgdG9vbHMvb2NhbWwveGVu c3RvcmVkL3Byb2Nlc3MubWwgfCAgIDE1ICsrKysrKysrKystLS0tLQo+Pj4gICAgMSBmaWxlIGNo YW5nZWQsIDEwIGluc2VydGlvbnMoKyksIDUgZGVsZXRpb25zKC0pCj4+Pgo+Pj4gZGlmZiAtLWdp dCBhL3Rvb2xzL29jYW1sL3hlbnN0b3JlZC9wcm9jZXNzLm1sCj4+PiBiL3Rvb2xzL29jYW1sL3hl bnN0b3JlZC9wcm9jZXNzLm1sCj4+PiBpbmRleCA3YTczNjY5Li5jOTJiZWM3IDEwMDY0NAo+Pj4g LS0tIGEvdG9vbHMvb2NhbWwveGVuc3RvcmVkL3Byb2Nlc3MubWwKPj4+ICsrKyBiL3Rvb2xzL29j YW1sL3hlbnN0b3JlZC9wcm9jZXNzLm1sCj4+PiBAQCAtMzQ0LDExICszNDQsMTEgQEAgbGV0IGZ1 bmN0aW9uX29mX3R5cGUgdHkgPQo+Pj4gICAgICAgIHwgWGVuYnVzLlhiLk9wLkludmFsaWQgICAg ICAgICAgIC0+IHJlcGx5X2FjayBkb19lcnJvcgo+Pj4gICAgICAgIHwgXyAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIC0+IHJlcGx5X2FjayBkb19lcnJvcgo+Pj4gICAgLWxldCBpbnB1dF9o YW5kbGVfZXJyb3IgfmNvbnMgfmRvbXMgfmZjdCB+dHkgfmNvbiB+dCB+cmlkIH5kYXRhID0KPj4+ ICtsZXQgaW5wdXRfaGFuZGxlX2Vycm9yIH5jb25zIH5kb21zIH5mY3QgfmNvbiB+dCB+cmVxID0K Pj4+ICAgICAgICBsZXQgcmVwbHlfZXJyb3IgZSA9Cj4+PiAgICAgICAgICAgIFBhY2tldC5FcnJv ciBlIGluCj4+PiAgICAgICAgdHJ5Cj4+PiAtICAgICAgICBmY3QgY29uIHQgZG9tcyBjb25zIGRh dGEKPj4+ICsgICAgICAgIGZjdCBjb24gdCBkb21zIGNvbnMgcmVxLlBhY2tldC5kYXRhCj4+PiAg ICAgICAgd2l0aAo+Pj4gICAgICAgIHwgRGVmaW5lLkludmFsaWRfcGF0aCAgICAgICAgICAtPiBy ZXBseV9lcnJvciAiRUlOVkFMIgo+Pj4gICAgICAgIHwgRGVmaW5lLkFscmVhZHlfZXhpc3QgICAg ICAgICAtPiByZXBseV9lcnJvciAiRUVYSVNUIgo+Pj4gQEAgLTM3MCw3ICszNzAsMTAgQEAgbGV0 IGlucHV0X2hhbmRsZV9lcnJvciB+Y29ucyB+ZG9tcyB+ZmN0IH50eSB+Y29uCj4+PiB+dCB+cmlk IH5kYXRhID0KPj4+ICAgICgqKgo+Pj4gICAgICogTm90aHJvdyBndWFyYW50ZWUuCj4+PiAgICAg KikKPj4+IC1sZXQgcHJvY2Vzc19wYWNrZXQgfnN0b3JlIH5jb25zIH5kb21zIH5jb24gfnRpZCB+ cmlkIH50eSB+ZGF0YSA9Cj4+PiArbGV0IHByb2Nlc3NfcGFja2V0IH5zdG9yZSB+Y29ucyB+ZG9t cyB+Y29uIH5yZXEgPQo+Pj4gKyAgICBsZXQgdHkgPSByZXEuUGFja2V0LnR5IGluCj4+PiArICAg IGxldCB0aWQgPSByZXEuUGFja2V0LnRpZCBpbgo+Pj4gKyAgICBsZXQgcmlkID0gcmVxLlBhY2tl dC5yaWQgaW4KPj4+ICAgICAgICB0cnkKPj4+ICAgICAgICAgICAgbGV0IGZjdCA9IGZ1bmN0aW9u X29mX3R5cGUgdHkgaW4KPj4+ICAgICAgICAgICAgbGV0IHQgPQo+Pj4gQEAgLTM3OSw3ICszODIs NyBAQCBsZXQgcHJvY2Vzc19wYWNrZXQgfnN0b3JlIH5jb25zIH5kb21zIH5jb24gfnRpZAo+Pj4g fnJpZCB+dHkgfmRhdGEgPQo+Pj4gICAgICAgICAgICAgICAgZWxzZQo+Pj4gICAgICAgICAgICAg ICAgICAgIENvbm5lY3Rpb24uZ2V0X3RyYW5zYWN0aW9uIGNvbiB0aWQKPj4+ICAgICAgICAgICAg ICAgIGluCj4+PiAtICAgICAgICBsZXQgcmVzcG9uc2UgPSBpbnB1dF9oYW5kbGVfZXJyb3IgfmNv bnMgfmRvbXMgfmZjdCB+dHkgfmNvbgo+Pj4gfnQgfnJpZCB+ZGF0YSBpbgo+Pj4gKyAgICAgICAg bGV0IHJlc3BvbnNlID0gaW5wdXRfaGFuZGxlX2Vycm9yIH5jb25zIH5kb21zIH5mY3QgfmNvbiB+ dAo+Pj4gfnJlcSBpbgo+Pj4gICAgICAgICAgICAgICgqIFB1dCB0aGUgcmVzcG9uc2Ugb24gdGhl IHdpcmUgKikKPj4+ICAgICAgICAgICAgc2VuZF9yZXNwb25zZSB0eSBjb24gdCByaWQgcmVzcG9u c2UKPj4+IEBAIC00MTIsMTEgKzQxNSwxMyBAQCBsZXQgZG9faW5wdXQgc3RvcmUgY29ucyBkb21z IGNvbiA9Cj4+PiAgICAgICAgaWYgbmV3cGFja2V0IHRoZW4gKAo+Pj4gICAgICAgICAgICBsZXQg cGFja2V0ID0gQ29ubmVjdGlvbi5wb3BfaW4gY29uIGluCj4+PiAgICAgICAgICAgIGxldCB0aWQs IHJpZCwgdHksIGRhdGEgPSBYZW5idXMuWGIuUGFja2V0LnVucGFjayBwYWNrZXQgaW4KPj4+ICsg ICAgICAgIGxldCByZXEgPSB7UGFja2V0LnRpZDsgUGFja2V0LnJpZDsgUGFja2V0LnR5OyBQYWNr ZXQuZGF0YX0gaW4KPj4+ICsKPj4gSSB0aGluayB0aGlzIGNoYW5nZSBicmVha3MgdGhlIGJ1aWxk IHdpdGggb2xkZXIgb2NhbWwgdmVyc2lvbnM6Cj4+Cj4+IHJvb3RAaGFzd2VsbD4gb2NhbWxvcHQg LXYKPj4gVGhlIE9DYW1sIG5hdGl2ZS1jb2RlIGNvbXBpbGVyLCB2ZXJzaW9uIDQuMDAuMQo+PiBT dGFuZGFyZCBsaWJyYXJ5IGRpcmVjdG9yeTogL3Vzci9saWI2NC9vY2FtbAo+PiByb290QGhhc3dl bGw+IG9jYW1sb3B0IC1nIC1jY29wdCAiICAgICIgLWR0eXBlcyAtSQo+PiAvcm9vdC90bXAveGVu L3Rvb2xzL29jYW1sL3hlbnN0b3JlZC8uLi9saWJzL3hiIC1JCj4+IC9yb290L3RtcC94ZW4vdG9v bHMvb2NhbWwveGVuc3RvcmVkLy4uL2xpYnMvbW1hcCAtSQo+PiAvcm9vdC90bXAveGVuL3Rvb2xz L29jYW1sL3hlbnN0b3JlZC8uLi9saWJzL3hjIC1JCj4+IC9yb290L3RtcC94ZW4vdG9vbHMvb2Nh bWwveGVuc3RvcmVkLy4uL2xpYnMvZXZlbnRjaG4gLWNjIGdjYyAtdyBGCj4+IC13YXJuLWVycm9y IEYgLWMgLW8gcHJvY2Vzcy5jbXggcHJvY2Vzcy5tbAo+PiByb290QGhhc3dlbGw+Cj4+Cj4+Cj4+ IHJvb3RAb3ZzMTA0PiBvY2FtbG9wdCAtdgo+PiBUaGUgT2JqZWN0aXZlIENhbWwgbmF0aXZlLWNv ZGUgY29tcGlsZXIsIHZlcnNpb24gMy4xMS4yCj4+IFN0YW5kYXJkIGxpYnJhcnkgZGlyZWN0b3J5 OiAvdXNyL2xpYjY0L29jYW1sCj4+IHJvb3RAb3ZzMTA0PiBvY2FtbG9wdCAtZyAtY2NvcHQgIiAg ICAiIC1kdHlwZXMgLUkKPj4gL3Jvb3QvdG1wL3hlbi90b29scy9vY2FtbC94ZW5zdG9yZWQvLi4v bGlicy94YiAtSQo+PiAvcm9vdC90bXAveGVuL3Rvb2xzL29jYW1sL3hlbnN0b3JlZC8uLi9saWJz L21tYXAgLUkKPj4gL3Jvb3QvdG1wL3hlbi90b29scy9vY2FtbC94ZW5zdG9yZWQvLi4vbGlicy94 YyAtSQo+PiAvcm9vdC90bXAveGVuL3Rvb2xzL29jYW1sL3hlbnN0b3JlZC8uLi9saWJzL2V2ZW50 Y2huIC1jYyBnY2MgLXcgRgo+PiAtd2Fybi1lcnJvciBGIC1jIC1vIHByb2Nlc3MuY214IHByb2Nl c3MubWwKPj4gRmlsZSAicHJvY2Vzcy5tbCIsIGxpbmUgNDg3LCBjaGFyYWN0ZXJzIDIzLTI0Ogo+ PiBFcnJvcjogU3ludGF4IGVycm9yCj4+IHJvb3RAb3ZzMTA0Pgo+Pgo+PiBJIGRvbid0IGtub3cg bXVjaCBhYm91dCBvY2FtbCAoT0ssIEkga25vdyAqbm90aGluZyogYWJvdXQgb2NhbWwpIHNvIEkK Pj4gY2FuJ3Qgc2F5IHdoYXQgZXhhY3RseSBtaWdodCBiZSB3cm9uZy4KPiBDb3VsZCB5b3UgcGVy aGFwcyB0cnkgdGhpcyBpbnN0ZWFkPwo+Cj4gbGV0IHJlcSA9IHt0aWQgPSBQYWNrZXQudGlkOyBy aWQgPSBQYWNrZXQucmlkOyB0eSA9IFBhY2tldC50eTsgZGF0YSA9Cj4gUGFja2V0LmRhdGF9IGlu Cj4KPiBJdCBpcyBtb3N0IGxpa2VseSB0aGF0IHRoZSBvbGRlciB2ZXJzaW9uIG9mIE9jYW1sIGNh bid0IGluZmVyIHRoZSBvcmRlcgo+IG9mIGZpZWxkcy4KPgo+IH5BbmRyZXcKCgpObywgaXQgbm93 IGdpdmVzIG1lICJFcnJvcjogVW5ib3VuZCByZWNvcmQgZmllbGQgbGFiZWwgdGlkIgoKLWJvcmlz CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpYZW4tZGV2 ZWwgbWFpbGluZyBsaXN0Clhlbi1kZXZlbEBsaXN0cy54ZW4ub3JnCmh0dHA6Ly9saXN0cy54ZW4u b3JnL3hlbi1kZXZlbAo=