From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============3429405948966523326==" MIME-Version: 1.0 From: Andrey Kuzmin Subject: Re: [SPDK] Bdev claim and single writer semantics Date: Wed, 09 May 2018 17:58:39 +0000 Message-ID: In-Reply-To: 1525887123.2784.56.camel@intel.com List-ID: To: spdk@lists.01.org --===============3429405948966523326== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Wed, May 9, 2018, 20:32 Walker, Benjamin wrote: > On Wed, 2018-05-09 at 08:12 +0000, Andrey Kuzmin wrote: > > Ben, > > > > the patch hs been submitted a week back, and then updated to account for > > review coments, but I still don't see it merged. Is there anything I can > do to > > make this happen? > > Hi Andrey, > > The patch is failing one of the tests. The test output is linked in the > comments > of the review. If you're referring to bdev unit tests, I have fixed that already in the updated patch. I re-ran it a few times and it consistently fails. I don't think > the problem is going to end up being your change, but rather your change > invalidates some assumptions or expected behavior in one of the tests. Th= at > needs to get debugged and resolved before we can merge. > If you detail on what test is failing (I'm not sure as all unitests pass locally), I could give it a shot as well. Thanks, A. > > Thanks, > Ben > _______________________________________________ > SPDK mailing list > SPDK(a)lists.01.org > https://lists.01.org/mailman/listinfo/spdk > -- = Regards, Andrey --===============3429405948966523326== Content-Type: text/html MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.html" PGJyPjxicj48ZGl2IGNsYXNzPSJnbWFpbF9xdW90ZSI+PGRpdiBkaXI9Imx0ciI+T24gV2VkLCBN YXkgOSwgMjAxOCwgMjA6MzIgV2Fsa2VyLCBCZW5qYW1pbiAmbHQ7PGEgaHJlZj0ibWFpbHRvOmJl bmphbWluLndhbGtlckBpbnRlbC5jb20iPmJlbmphbWluLndhbGtlckBpbnRlbC5jb208L2E+Jmd0 OyB3cm90ZTo8YnI+PC9kaXY+PGJsb2NrcXVvdGUgY2xhc3M9ImdtYWlsX3F1b3RlIiBzdHlsZT0i bWFyZ2luOjAgMCAwIC44ZXg7Ym9yZGVyLWxlZnQ6MXB4ICNjY2Mgc29saWQ7cGFkZGluZy1sZWZ0 OjFleCI+T24gV2VkLCAyMDE4LTA1LTA5IGF0IDA4OjEyICswMDAwLCBBbmRyZXkgS3V6bWluIHdy b3RlOjxicj4KJmd0OyBCZW4sPGJyPgomZ3Q7IDxicj4KJmd0OyB0aGUgcGF0Y2ggaHMgYmVlbiBz dWJtaXR0ZWQgYSB3ZWVrIGJhY2ssIGFuZCB0aGVuIHVwZGF0ZWQgdG8gYWNjb3VudCBmb3I8YnI+ CiZndDsgcmV2aWV3IGNvbWVudHMsIGJ1dCBJIHN0aWxsIGRvbiYjMzk7dCBzZWUgaXQgbWVyZ2Vk LiBJcyB0aGVyZSBhbnl0aGluZyBJIGNhbiBkbyB0bzxicj4KJmd0OyBtYWtlIHRoaXMgaGFwcGVu Pzxicj4KPGJyPgpIaSBBbmRyZXksPGJyPgo8YnI+ClRoZSBwYXRjaCBpcyBmYWlsaW5nIG9uZSBv ZiB0aGUgdGVzdHMuIFRoZSB0ZXN0IG91dHB1dCBpcyBsaW5rZWQgaW4gdGhlIGNvbW1lbnRzPGJy PgpvZiB0aGUgcmV2aWV3LjwvYmxvY2txdW90ZT48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2IGNs YXNzPSJnbWFpbF9xdW90ZSI+PGJsb2NrcXVvdGUgY2xhc3M9ImdtYWlsX3F1b3RlIiBzdHlsZT0i bWFyZ2luOjAgMCAwIC44ZXg7Ym9yZGVyLWxlZnQ6MXB4ICNjY2Mgc29saWQ7cGFkZGluZy1sZWZ0 OjFleCI+IDwvYmxvY2txdW90ZT48L2Rpdj48ZGl2PklmIHlvdSYjMzk7cmUgcmVmZXJyaW5nIHRv IGJkZXYgdW5pdCB0ZXN0cywgSSBoYXZlIGZpeGVkIHRoYXQgYWxyZWFkeSBpbiB0aGUgdXBkYXRl ZCBwYXRjaC48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2IGNsYXNzPSJnbWFpbF9xdW90ZSI+PGJs b2NrcXVvdGUgY2xhc3M9ImdtYWlsX3F1b3RlIiBzdHlsZT0ibWFyZ2luOjAgMCAwIC44ZXg7Ym9y ZGVyLWxlZnQ6MXB4ICNjY2Mgc29saWQ7cGFkZGluZy1sZWZ0OjFleCI+SSByZS1yYW4gaXQgYSBm ZXcgdGltZXMgYW5kIGl0IGNvbnNpc3RlbnRseSBmYWlscy4gSSBkb24mIzM5O3QgdGhpbms8YnI+ CnRoZSBwcm9ibGVtIGlzIGdvaW5nIHRvIGVuZCB1cCBiZWluZyB5b3VyIGNoYW5nZSwgYnV0IHJh dGhlciB5b3VyIGNoYW5nZTxicj4KaW52YWxpZGF0ZXMgc29tZSBhc3N1bXB0aW9ucyBvciBleHBl Y3RlZCBiZWhhdmlvciBpbiBvbmUgb2YgdGhlIHRlc3RzLiBUaGF0PGJyPgpuZWVkcyB0byBnZXQg ZGVidWdnZWQgYW5kIHJlc29sdmVkIGJlZm9yZSB3ZSBjYW4gbWVyZ2UuPGJyPjwvYmxvY2txdW90 ZT48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2PklmIHlvdSBkZXRhaWwgb24gd2hhdCB0ZXN0IGlz IGZhaWxpbmcgKEkmIzM5O20gbm90IHN1cmUgYXMgYWxsIHVuaXRlc3RzIHBhc3MgbG9jYWxseSks IEkgY291bGQgZ2l2ZSBpdCBhIHNob3QgYXMgd2VsbC48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2 PlRoYW5rcyw8L2Rpdj48ZGl2PkEuPC9kaXY+PGRpdiBjbGFzcz0iZ21haWxfcXVvdGUiPjxibG9j a3F1b3RlIGNsYXNzPSJnbWFpbF9xdW90ZSIgc3R5bGU9Im1hcmdpbjowIDAgMCAuOGV4O2JvcmRl ci1sZWZ0OjFweCAjY2NjIHNvbGlkO3BhZGRpbmctbGVmdDoxZXgiPgo8YnI+ClRoYW5rcyw8YnI+ CkJlbjxicj4KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX188 YnI+ClNQREsgbWFpbGluZyBsaXN0PGJyPgo8YSBocmVmPSJtYWlsdG86U1BES0BsaXN0cy4wMS5v cmciIHRhcmdldD0iX2JsYW5rIj5TUERLQGxpc3RzLjAxLm9yZzwvYT48YnI+CjxhIGhyZWY9Imh0 dHBzOi8vbGlzdHMuMDEub3JnL21haWxtYW4vbGlzdGluZm8vc3BkayIgcmVsPSJub3JlZmVycmVy IiB0YXJnZXQ9Il9ibGFuayI+aHR0cHM6Ly9saXN0cy4wMS5vcmcvbWFpbG1hbi9saXN0aW5mby9z cGRrPC9hPjxicj4KPC9ibG9ja3F1b3RlPjwvZGl2Pi0tIDxicj48ZGl2IGRpcj0ibHRyIiBjbGFz cz0iZ21haWxfc2lnbmF0dXJlIiBkYXRhLXNtYXJ0bWFpbD0iZ21haWxfc2lnbmF0dXJlIj48cCBk aXI9Imx0ciI+UmVnYXJkcyw8YnI+CkFuZHJleTwvcD4KPC9kaXY+Cg== --===============3429405948966523326==--