[00:42:29] lhodev: about DPDK version check you pointed out - I can add this check. I think simple >= 17.04 won't work. I think there will be needed also <= MAX_VERSION [00:48:12] other issue with linking... I will check them but don't think "maximum flexibility" might be hard to do when doing packaging as we need everything installed *before* building SPDK. [00:49:20] But we should at least try to give "maximum flexibility" for user. [01:01:35] jimharris: there is an update in https://review.gerrithub.io/#/c/spdk/spdk/+/417962/ [03:23:25] *** Quits: alekseymmm (050811aa@gateway/web/freenode/ip.5.8.17.170) (Ping timeout: 256 seconds) [07:33:21] pwodkowx: Yes, I think you are correct. We may need to add a MAX_VERSION for DPDK. While I would like to believe the DPDK would strive to retain backwards compatibility as long as possible, software that is in use for a long time does tend to deprecate functionality so putting that into place is probably a good idea. [07:50:42] pwodkowx: I noticed that a couple of the symbols we use from dpdk are listed in the experimental portion of the dpdk map file (e.g. rte_memseg_contig_walk). Do you know how the packaged versions of the dpdk shared objects handle experimental features like that? I couldn't find rte_memseg_contig_walk in the 18.07 version of the shared objects, but I figure that may just be because it is too old. [07:58:06] sethhowe: Curious: what's the latest version of DPDK of which you are aware that contains rte_memseg_contig_walk ? [08:09:55] lhodev pwodkowx: Sorry,there was a typo. The version of the shared libraries I was using was 17.08. [08:56:14] pwodkowx: thanks - reading it now [08:57:05] this doesn't seem like silent overflow though - if you specify a smaller payload than the command requires, the SSD is completing it with INVALID FIELD error [09:07:51] *** Joins: travis-ci (~travis-ci@ec2-54-234-151-219.compute-1.amazonaws.com) [09:07:52] (spdk/master) spdk: fix check_format.sh (Piotr Pelplinski) [09:07:52] Diff URL: https://github.com/spdk/spdk/compare/b3c330339d17...c3d9530cd07b [09:07:52] *** Parts: travis-ci (~travis-ci@ec2-54-234-151-219.compute-1.amazonaws.com) () [10:51:46] *** Joins: travis-ci (~travis-ci@ec2-54-234-151-219.compute-1.amazonaws.com) [10:51:47] (spdk/master) test/spdkcli: Add a match test to iscsi spdkcli test (Pawel Niedzwiecki) [10:51:47] Diff URL: https://github.com/spdk/spdk/compare/c3d9530cd07b...598d3cfbd3d4 [10:51:47] *** Parts: travis-ci (~travis-ci@ec2-54-234-151-219.compute-1.amazonaws.com) () [15:00:46] *** Joins: travis-ci (~travis-ci@ec2-54-205-133-187.compute-1.amazonaws.com) [15:00:47] (spdk/master) ut/bdev: Exercise the ENOMEM path in the IO splitting (Shuhei Matsumoto) [15:00:47] Diff URL: https://github.com/spdk/spdk/compare/803449ca7b18...a8780177c66b [15:00:47] *** Parts: travis-ci (~travis-ci@ec2-54-205-133-187.compute-1.amazonaws.com) () [15:32:07] *** Joins: pawelkax (pawelkax@nat/intel/x-vqgsztwewaflxspg)