[00:08:00] *** Quits: lhodev_ (~lhodev@66-90-218-190.dyn.grandenetworks.net) (Read error: Connection reset by peer) [00:08:35] *** Joins: lhodev (~lhodev@66-90-218-190.dyn.grandenetworks.net) [00:08:49] *** Joins: Shuhei (caf6fc61@gateway/web/freenode/ip.202.246.252.97) [00:23:13] *** Quits: guerby (~guerby@april/board/guerby) (Read error: Connection reset by peer) [00:23:24] *** Joins: guerby (~guerby@ip165.tetaneutral.net) [00:23:24] *** Quits: guerby (~guerby@ip165.tetaneutral.net) (Changing host) [00:23:24] *** Joins: guerby (~guerby@april/board/guerby) [00:53:20] *** Joins: tkulasek (~tkulasek@134.134.139.72) [00:55:42] Hi All, memory leak issue Paul reported looks intermittent. I saw the same issue repeatedly a few hours ago but now looks goes well. I don't know what is the difference yet. [01:18:03] *** Quits: Shuhei (caf6fc61@gateway/web/freenode/ip.202.246.252.97) (Ping timeout: 252 seconds) [02:11:30] *** Joins: tomzawadzki (~tomzawadz@192.55.54.40) [02:33:43] *** Joins: johnmeneghini (~johnmeneg@pool-100-0-53-181.bstnma.fios.verizon.net) [02:58:00] *** Quits: johnmeneghini (~johnmeneg@pool-100-0-53-181.bstnma.fios.verizon.net) (Quit: Leaving.) [03:22:43] *** Quits: guerby (~guerby@april/board/guerby) (Read error: Connection reset by peer) [03:22:54] *** Joins: guerby (~guerby@ip165.tetaneutral.net) [03:22:55] *** Quits: guerby (~guerby@ip165.tetaneutral.net) (Changing host) [03:22:55] *** Joins: guerby (~guerby@april/board/guerby) [03:49:27] *** Quits: dlw (~Thunderbi@114.255.44.143) (Ping timeout: 240 seconds) [05:12:34] bwalker: should we consider adding full-name parameters parser to spdk apps? https://review.gerrithub.io/c/spdk/spdk/+/419156: env/app: add unlink hugepages option to app [05:12:47] we're running out of characters [05:14:37] *** Joins: lyan (~lyan@2605:a000:160e:2124:4a4d:7eff:fef2:eea3) [05:15:01] *** lyan is now known as Guest1853 [07:03:57] what about utf-8? :P [08:00:13] peluse: No, that one doesn't show up on their status page. [08:00:46] *** Joins: philipp-sk (~Philipp@ktnron0916w-lp130-02-76-66-162-159.dsl.bell.ca) [08:04:24] *** Quits: philipp-sk (~Philipp@ktnron0916w-lp130-02-76-66-162-159.dsl.bell.ca) (Read error: Connection reset by peer) [08:04:44] *** Quits: tomzawadzki (~tomzawadz@192.55.54.40) (Ping timeout: 268 seconds) [08:08:06] *** Joins: philipp-sk (~Philipp@ktnron0916w-lp130-02-76-66-162-159.dsl.bell.ca) [09:54:53] darsto: I was thinking the same thing [09:55:01] for 90% of the options we have, a full name would be much better [11:31:05] *** Quits: tkulasek (~tkulasek@134.134.139.72) (Ping timeout: 240 seconds) [12:54:25] *** Joins: alekseymmm (bcf3adf1@gateway/web/freenode/ip.188.243.173.241) [13:20:29] bwalker, FYI the crypto dependency patch w/Jim's comments addressed (to enable CI for crypto) is ready when you have time... https://review.gerrithub.io/c/spdk/spdk/+/404983 [13:49:41] *** Quits: alekseymmm (bcf3adf1@gateway/web/freenode/ip.188.243.173.241) (Quit: Page closed) [13:49:55] lgtm [15:07:24] *** Joins: lhodev_ (~lhodev@66-90-218-190.dyn.grandenetworks.net) [15:09:00] *** Quits: lhodev (~lhodev@66-90-218-190.dyn.grandenetworks.net) (Ping timeout: 268 seconds) [15:15:14] *** Quits: lhodev_ (~lhodev@66-90-218-190.dyn.grandenetworks.net) (Ping timeout: 244 seconds) [15:27:00] *** Joins: lhodev (~lhodev@66-90-218-190.dyn.grandenetworks.net) [15:47:21] *** Quits: philipp-sk (~Philipp@ktnron0916w-lp130-02-76-66-162-159.dsl.bell.ca) (Ping timeout: 240 seconds) [17:27:25] *** Joins: johnmeneghini (~johnmeneg@pool-100-0-53-181.bstnma.fios.verizon.net) [17:28:14] Hello! [17:28:47] Can someone please submit patch 414861? [17:28:49] https://review.gerrithub.io/#/c/spdk/spdk/+/414861/ [18:09:58] johnmeneghini, just reviewed/tested your vagrant patch [18:10:10] freaking AWESOME!!! [18:10:29] one of the biggest improvements in test stuff in a while... [18:11:01] I gave it a -1 because of a typo in a .sh filename in the FreeBSD stuff that should be "copy-n-paste" and just work [18:11:51] a maintainer can of course override me but as I put in the patch comments, there's no reason to rush this through that I can think of. You have like a half dozen little cleanup things to do so IMHO its worth spinning it one more time. Thanks again man, that looked like a TON of work!!! [18:38:42] I'll fix the typo [18:45:46] *** Quits: Guest1853 (~lyan@2605:a000:160e:2124:4a4d:7eff:fef2:eea3) (Remote host closed the connection) [18:46:29] gracias John!! [19:30:04] *** Joins: dlw (~Thunderbi@114.255.44.143) [19:30:22] peluse: do you still have your windows machine there? [19:31:08] I think the "/usr/bin/env bash" errors are coming from your Windows platform. [20:15:17] johnmeneghini, yes, for sure. I think I put that in my comment. [20:15:27] I do have it here [20:17:41] peluse: What happens if you type /usr/bin/env bash at the git-bash command line? [20:19:00] I never use windows and have to switch over my KVM to that machine. I just opened a cmd prompt and git worked, can't remember if I installed it or if was there win Win10. Give me a min... [20:20:23] OK. We'll if you can give another look at https://review.gerrithub.io/#/c/spdk/spdk/+/414861/ [20:20:34] I've cleaned it up a bit more. [20:21:22] Hopefully we can merge this patch tomorrow morning. I think there are more bugs lurking, but I want to merge this in and move forward. [20:30:07] *** Quits: johnmeneghini (~johnmeneg@pool-100-0-53-181.bstnma.fios.verizon.net) (Quit: Leaving.) [20:31:49] *** Joins: peluse2 (48d0c853@gateway/web/freenode/ip.72.208.200.83) [20:32:15] I'm retesting with git bash, will be a few min.... [20:38:59] wish my freaking Mac vbox install would work, previous version worked fine. If I have to use Windows again in the next month I'm going to eat a bullet... almost to the step that I was getting the bash error before.... [20:42:30] ugh, I'm done for tonight. this is with git bash and when I installed I used 'check out windows style, check in unix style' I think which I assume would be OK if using git bash prompt. [20:42:36] vagrant@vagrant:~$ script -c 'sudo spdk_repo/spdk/scripts/vagrant/update.sh' update.log Script started, file is update.log /usr/bin/env: 'bash\r': No such file or directory Script done, file is update.log [20:42:47] that was ubuntu18 VM [20:43:05] I'll look at it again in the morn, it won't merge this late anyways. thanks again for addressing the comments John [22:16:28] *** Quits: sage_ (~quassel@2607:f298:5:101d:f816:3eff:fe21:1966) (*.net *.split) [22:35:18] *** Joins: sage_ (~quassel@2607:f298:5:101d:f816:3eff:fe21:1966) [23:06:43] *** Joins: tomzawadzki (~tomzawadz@192.55.54.40) [23:21:38] *** Quits: dlw (~Thunderbi@114.255.44.143) (Read error: Connection reset by peer) [23:21:51] *** Joins: dlw (~Thunderbi@114.255.44.143) [23:26:46] klateck: what happened to jenkins logs here? https://review.gerrithub.io/c/spdk/spdk/+/419609 [23:31:20] bwalker: what's the purpose of spdk_bdev_io_set_buf being public? [23:33:48] and why does it set iov_len to an arbitrary len? it should be at least blocksize aligned [23:48:53] *** Joins: klateck (c0c69724@gateway/web/freenode/ip.192.198.151.36)