[08:28:33] *** Joins: gila (~gila@5ED4FE92.cm-7-5d.dynamic.ziggo.nl) [09:07:40] *** Quits: gila (~gila@5ED4FE92.cm-7-5d.dynamic.ziggo.nl) (Quit: My Mac Pro has gone to sleep. ZZZzzz…) [09:40:32] *** Joins: dwaddington (Daniel@nat/ibm/x-okaxwmndmchiybza) [09:49:22] Do you think DPDK plans to keep support for physical address (even if root priv is needed). I'm using physical addresses in other parts of the solution and so I need them currently. [09:57:18] hi dwaddington - yes, I think DPDK still plans to keep support for physical addresses [09:59:06] drv: i rebased bwalker's whole env patch series - I think we should try to get a lot of that committed today [09:59:18] ok, I'll take a look [10:14:20] two of cunyin's blobfs patches are ready to go in I think - the subdirectory one was already +2'd by bwalker before the small change I asked him to make [10:26:14] jimharris: I have reviewed bwalker's env series - feel free to merge if you are good with it [10:33:25] cunyinch's patches look good too [11:07:10] for bwalker's memory unit tests - are we intentionally keeping these under test/lib/env for some reason instead of test/unit/lib/env_dpdk? [11:07:39] i'm ok keeping them there for now [11:09:31] hmm, they should probably go in test/unit eventually, but we need to add some way to detect whether we're building with DPDK or not [11:09:46] since the env_dpdk stuff all includes dpdk headers and can't be built without them [11:10:06] drv, which of cuntinch's patches are you referring to? I was just looking at https://review.gerrithub.io/#/c/376276/ which is also related to a blobfs one that I haven't looked at yet... [11:10:56] peluse: the two that just got checked in were RocksDB related [11:11:14] https://github.com/spdk/spdk/commit/970891ccb43914cab8dbea785fecc451937e4df6 and https://github.com/spdk/spdk/commit/14f45fc832159a42591b49674fc9c678095ea8cf [11:11:14] K, thanks [11:11:43] well i'm ok putting the memory tests there for now - although they are getting executed anywhere in the autotest hierarchy [11:11:54] ah - strike that [11:14:16] drv: can you add a +2 back on this patch? https://review.gerrithub.io/#/c/375829/ [11:14:36] done [12:21:29] drv, jimharris : updated the last few little things in blobcli. certainly not a priority item but would be nice to reference it on master during SDC talk.. https://review.gerrithub.io/#/c/373383/ [14:15:16] peluse: https://review.gerrithub.io/#/c/376703/ needs a filename change so it is published with the right date [14:16:04] argh, copy n pasted and forgot to update... thx [14:16:19] yep, easy to miss in the filename [14:19:09] drv, yeah sorry. done [14:20:39] cool, no problem - just wanted to get it done today or else the filename will go stale again :) [14:40:55] fixed the submodule ver in blocli patch as well. It must be close to beer-thirty [14:44:01] the git command-line tooling just generally interacts poorly/unintuitively with submodules [14:44:24] yeah and my habit of doing commit -a doesn't help much :)