olivier (Olivier Cochard-Labbé)
User

Projects

User Details

User Since
Feb 5 2016, 2:19 PM (93 w, 2 d)

Recent Activity

Yesterday

olivier added a comment to D13017: Add IPSec tests in tunnel mode.

Good catch for the no_dad!

Sun, Nov 19, 8:56 AM
olivier updated the diff for D13017: Add IPSec tests in tunnel mode.

Add no_dad to inet6 setup for avoiding a sleep

Sun, Nov 19, 8:55 AM

Fri, Nov 17

olivier added a comment to D12446: Support attaching tx queues to cpus.

How to dump a panic when the kernel crash during boot before loading disk controller drivers ?
Can I compile a kernel with .debug embedded into the kernel ?

Fri, Nov 17, 5:58 PM
olivier added a comment to D13017: Add IPSec tests in tunnel mode.

New tests output following r325872 that fixed cbc key lenght:

Fri, Nov 17, 4:40 PM
olivier added inline comments to D13017: Add IPSec tests in tunnel mode.
Fri, Nov 17, 4:38 PM
olivier updated the diff for D13017: Add IPSec tests in tunnel mode.

Updating test scripts following kristof's advices

Fri, Nov 17, 4:35 PM

Wed, Nov 15

olivier accepted D12770: ipsec: Use the same keysize values for HMAC as prior to r324017.
Wed, Nov 15, 8:35 PM
olivier added a comment to D12446: Support attaching tx queues to cpus.

Once I've applied this patch on my system (that is already patched with D112727 and D13096) it panic.

Wed, Nov 15, 8:28 PM
olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

Created D13096 to fix the overallocation of queues. However, even with 24 queues, there should have been two per core (one per thread with HTT), not one per core across the two sockets.

Wed, Nov 15, 7:28 AM
olivier accepted D13097: net/xrdp-devel: Update to 0.9.4.
Wed, Nov 15, 7:15 AM
olivier accepted D13089: x11-drivers/xorgxrdp: Update to 0.2.4.
Wed, Nov 15, 6:19 AM

Tue, Nov 14

olivier added a comment to D12446: Support attaching tx queues to cpus.

I don't reach to apply the patch, and when I've tried to "manually" merge it, I've broke the compilation :-(

Tue, Nov 14, 9:57 PM

Mon, Nov 13

olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

Are you sure about the number of queues? For iflib version of driver the default number is based on number of cores. Are you setting dev.ix.X.override_n(r|t)xqs sysctls?

Mon, Nov 13, 10:50 PM
olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

Second, on a Dual CPU, Xeon_E5-2650 (12Cores), with Intel 82599ES 10Gigabit (using default 8 queues):

Hrm... is that two E5-2651s for 24 cores total, or two 6-core E5s for 12 cores total?

Mon, Nov 13, 7:56 PM
olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

@olivier Can you post the sysctl/loader.conf settings you are using from both iterations of the test?

Mon, Nov 13, 7:40 PM
olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

@olivier So a decrease of ~33% with IFLIB?

Mon, Nov 13, 7:34 PM
olivier accepted D13071: benchmarks/stress-ng: Update to 0.09.02.
Mon, Nov 13, 5:00 PM
olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

Here are my updated benches "forwarding smallest packet size" results on 2 different hardware.
I'm using a fresh head (r325763) and the latest diff (35190) of this review.

Mon, Nov 13, 3:57 PM
olivier committed rP454110: Update to 0.170.
Update to 0.170
Mon, Nov 13, 12:17 PM

Fri, Nov 10

olivier accepted D13037: www/yarn: Use www/node as default.
Fri, Nov 10, 7:51 PM
olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

I've tried to update my previous benches, but on a recent head (r325618), and this review need to be updated because it no more apply (a small part was committed into head).
And compilation failed on this recent head too:

Fri, Nov 10, 10:10 AM

Thu, Nov 9

olivier added reviewers for D13017: Add IPSec tests in tunnel mode: jmg, bz.
Thu, Nov 9, 2:27 PM
olivier created D13017: Add IPSec tests in tunnel mode.
Thu, Nov 9, 2:26 PM

Wed, Nov 8

olivier accepted D13006: www/yarn: Add support to www/node8.
Wed, Nov 8, 6:08 PM
olivier committed rP453751: Update to 3.0.2.
Update to 3.0.2
Wed, Nov 8, 3:17 PM

Sun, Nov 5

olivier accepted D12939: www/yarn: Update to 1.3.2.
Sun, Nov 5, 7:28 PM
olivier accepted D12938: benchmarks/stress-ng: Update to 0.09.01.
Sun, Nov 5, 4:20 PM

Fri, Nov 3

olivier accepted D12786: devel/liblas12: Fix build error on CURRENT.
Fri, Nov 3, 11:00 AM

Wed, Nov 1

olivier committed rP453268: Update to 1.11.2 and conditionally uses NLS.
Update to 1.11.2 and conditionally uses NLS
Wed, Nov 1, 8:40 AM

Tue, Oct 31

olivier committed rP453216: Update to 2.6.0.
Update to 2.6.0
Tue, Oct 31, 6:33 AM

Mon, Oct 30

olivier accepted D12835: benchmarks/stress-ng: Update to 0.09.00.
Mon, Oct 30, 3:28 PM
olivier added a comment to D12770: ipsec: Use the same keysize values for HMAC as prior to r324017.

I confirm: I'm using static key only on my tests.

Mon, Oct 30, 3:06 PM

Fri, Oct 27

olivier added a comment to D12770: ipsec: Use the same keysize values for HMAC as prior to r324017.

I've found the commit that broke AES-GCM: r324037 "aesni(4): Add support for x86 SHA intrinsics".
But how is possible that I'm impacted without loading aesni module ?!?

Fri, Oct 27, 11:10 PM
olivier added a comment to D12685: Make ipfw dynamic states lockless on fast path.

And now comparing pf vs ipfw:

Fri, Oct 27, 1:11 PM
olivier accepted D12806: devel/codeblocks-devel: new port.
Fri, Oct 27, 8:21 AM
olivier added a comment to D12685: Make ipfw dynamic states lockless on fast path.

Wow, results regarding the 5M UDP session bench:

Fri, Oct 27, 8:03 AM
olivier requested changes to D12806: devel/codeblocks-devel: new port.

After adding a new port, you need to add it to the category Makefile too (here in devel/Makefile): I didn't see modifications to devel/Makefile here.
And each time I'm touching such category Makefile file, I'm checking index is still building fine with an indexbuild (cd /usr/port; make index).

Fri, Oct 27, 5:45 AM

Thu, Oct 26

olivier added a comment to D12685: Make ipfw dynamic states lockless on fast path.

Here is on a Xeon 8cores with Chelsio results, 2000 UDP flows of small packet size:

Thu, Oct 26, 5:31 PM

Wed, Oct 25

olivier added a comment to D12770: ipsec: Use the same keysize values for HMAC as prior to r324017.
In D12770#265548, @ae wrote:
r324972 with D12770fbsd 11.1OKNOK: netstat display "packets dropped; no transform" on destination

Did you try without aesni module loaded?

Wed, Oct 25, 10:48 PM
olivier added a comment to D12770: ipsec: Use the same keysize values for HMAC as prior to r324017.

After more test: Your patch correctly fix aes-cbc key length.

Wed, Oct 25, 10:12 PM
olivier added a comment to D12770: ipsec: Use the same keysize values for HMAC as prior to r324017.

After more test there is still a problem somewhere: I'm digging it and will add more information.

Wed, Oct 25, 3:56 PM
olivier accepted D12787: benchmarks/stress-ng: Update to 0.08.18.
Wed, Oct 25, 1:34 PM
olivier accepted D12770: ipsec: Use the same keysize values for HMAC as prior to r324017.
Wed, Oct 25, 1:10 PM
olivier added a comment to D12786: devel/liblas12: Fix build error on CURRENT.

Does this fall into the "Build fixes which cause a package to become compilable where it was previously failing" of the "Examples of changes which do not require a PORTREVISION bump" ?

Wed, Oct 25, 10:18 AM
olivier accepted D12768: biology/htslib biology/samtools devel/bcftools: Update to 1.6.
Wed, Oct 25, 9:50 AM

Mon, Oct 23

olivier added inline comments to D12685: Make ipfw dynamic states lockless on fast path.
Mon, Oct 23, 7:30 PM

Oct 20 2017

olivier accepted D12739: www/py-libsass: Update to 0.13.3.
Oct 20 2017, 8:14 PM
olivier committed rP452541: Update FRR to 3.0.
Update FRR to 3.0
Oct 20 2017, 4:25 PM

Oct 19 2017

olivier accepted D12728: devel/tig: Update to 2.3.0.
Oct 19 2017, 1:37 PM
olivier accepted D12721: www/gitea: Update to 1.2.1.
Oct 19 2017, 5:26 AM

Oct 18 2017

olivier accepted D12720: devel/ioncube: Update to 10.0.2.
Oct 18 2017, 8:43 PM

Oct 16 2017

olivier accepted D12687: sysutils/logrotate: Update to 3.13.0.
Oct 16 2017, 8:49 PM
olivier accepted D12686: www/yarn: Update to 1.2.1.
Oct 16 2017, 2:44 PM

Oct 13 2017

olivier accepted D12651: emulators/simh: fix name clash with other ports and tidy compilations.
Oct 13 2017, 10:27 AM

Oct 12 2017

olivier added a comment to D12639: Enable VIMAGE by default.
In D12639#262566, @jtl wrote:

I remain concerned about the performance overhead of activating this where not needed. However, I have no performance information to either alleviate or confirm my fears. Has someone done the work to gather performance information to compare a non-VIMAGE kernel to a VIMAGE kernel with a single VNET?

Oct 12 2017, 4:47 AM

Oct 11 2017

olivier accepted D12637: benchmarks/stress-ng: Update 0.08.17.
Oct 11 2017, 2:31 PM
olivier accepted D12636: www/yarn: Update to 1.2.0.
Oct 11 2017, 12:52 PM

Oct 9 2017

olivier accepted D12626: science/py-scikit-fuzzy: Update to 0.3.1.
Oct 9 2017, 12:50 PM

Oct 8 2017

olivier accepted D12623: www/mod_security: update to 2.9.2.
Oct 8 2017, 7:01 PM

Oct 6 2017

olivier added a comment to D12602: databases/memcached: update to version 1.5.2.

Just wait for @swills approval before committing it because this port belongs to him.

Oct 6 2017, 3:24 PM

Sep 29 2017

olivier accepted D12542: www/yarn: Re-enabling -f option on readlink.

Only people who are working can break things.

Sep 29 2017, 11:16 PM

Sep 28 2017

olivier added a comment to D12525: devel/gdb: Updating to gdb 8.0.1.

Lot's of patch files were renamed with a special name: Is this correct ?

Sep 28 2017, 1:45 PM
olivier accepted D12517: benchmarks/stress-ng: Update to version 0.08.16.
Sep 28 2017, 5:29 AM
olivier accepted D12516: www/yarn: Updating to version 1.0.2.
Sep 28 2017, 4:48 AM

Sep 22 2017

olivier accepted D12439: Have ifmp_ring_enqueue() abdicate instead of switch to a consumer.
Sep 22 2017, 6:05 AM
olivier added a comment to D12439: Have ifmp_ring_enqueue() abdicate instead of switch to a consumer.

Yeah!!! You've fixed the problem.

Sep 22 2017, 6:04 AM

Sep 15 2017

olivier accepted D12378: net-mgmt/openbmp: new port.
Sep 15 2017, 1:44 PM
olivier requested changes to D12378: net-mgmt/openbmp: new port.

Small comestic proposal (check my 2 comments).

Sep 15 2017, 11:55 AM

Sep 13 2017

olivier accepted D12332: www/yarn: Updating to version 1.0.1.

It's okay, portlint is just an helper tool and didn't catch this port complexity.

Sep 13 2017, 9:33 AM
olivier accepted D12182: Fix build on MIPS and powerpc..
Sep 13 2017, 9:32 AM

Sep 12 2017

olivier accepted D12335: benchmarks/stress-ng: updating to 0.8.14.
Sep 12 2017, 1:42 PM

Sep 8 2017

olivier accepted D12273: benchmarks/phoronix-test-suite: update to 7.2.1.

Don't know what is a "mad64" arch, but it's okay :)-

Sep 8 2017, 9:33 AM

Sep 7 2017

olivier accepted D12262: databases/timescaledb: update to 0.4.1.

It's ok, but because the sources provides the license file (LICENSE), you should add a LICENSE_FILE=

Sep 7 2017, 5:14 AM

Sep 6 2017

olivier accepted D12244: audio/dsbmixer: update to version 1.2.
Sep 6 2017, 5:30 PM
olivier accepted D12234: devel/cccc: update to version 3.1.5.
Sep 6 2017, 9:45 AM
olivier added inline comments to D12234: devel/cccc: update to version 3.1.5.
Sep 6 2017, 8:06 AM

Sep 5 2017

olivier accepted D12234: devel/cccc: update to version 3.1.5.
Sep 5 2017, 10:10 PM

Sep 4 2017

olivier accepted D12224: benchmarks/unixbench: updating to version 5.1.3.
Sep 4 2017, 10:21 PM
olivier added inline comments to D12224: benchmarks/unixbench: updating to version 5.1.3.
Sep 4 2017, 3:47 PM

Sep 1 2017

olivier accepted D12204: benchmarks/stress-ng: Updating to version 0.08.13.
Sep 1 2017, 12:31 PM
olivier accepted D12199: Introduce information about new commiter - pizzamig.

I didn't rebuild the full doc for testing error syntax, but by reading all of your changes it seems okay.

Sep 1 2017, 11:53 AM

Aug 30 2017

olivier accepted D12166: benchmarks/nbench: fixing master site and the homepage entry.

Does a "LICENSE= NONE" valid in this case ? (avoid complains from portlint -A)

Aug 30 2017, 4:45 PM
olivier requested changes to D12166: benchmarks/nbench: fixing master site and the homepage entry.

Mmm… your email is not updated into the MAINTAINER field on this port: Your commit r448911 correctly updated all your email: Does this mean your are working on a non-updated local source tree ?

Aug 30 2017, 3:59 PM

Aug 29 2017

olivier accepted D12150: Change MAINTAINER email to my new FreeBSD email.

Good: You correctly used the good capitalization rule for "FreeBSD" :-)

Aug 29 2017, 12:51 PM

Aug 28 2017

olivier accepted D12148: astro/xearth: Add myself (pizzamig) to freebsd.committers.markers.
Aug 28 2017, 8:53 PM
olivier committed rP448874: Please welcome Luca Pizzamiglio (pizzamig@) as a new ports committer!.
Please welcome Luca Pizzamiglio (pizzamig@) as a new ports committer!
Aug 28 2017, 6:27 PM
olivier added a comment to D12132: Avoid spinning in random_harvest_queue.

How do you measure the 25% performance increase ?

Aug 28 2017, 4:34 PM
olivier accepted D12145: Adding personal information about pizzamig as port committer.
Aug 28 2017, 2:35 PM

Aug 14 2017

olivier added a comment to D11727: ixgbe(4): Update HEAD to use iflib.

About 30% performance drop regression (from 2.8Mpps to 1.8Mpps) during forwarding of smallest packet:

Aug 14 2017, 2:31 PM

Jul 21 2017

olivier committed rP446318: Fix collectd5's range version.
Fix collectd5's range version
Jul 21 2017, 2:39 PM

Jul 20 2017

olivier committed rP446296: MFH: r446072.
MFH: r446072
Jul 20 2017, 9:40 PM
olivier committed rP446295: MFH: r446193.
MFH: r446193
Jul 20 2017, 9:38 PM

Jul 19 2017

olivier committed rP446194: Fix vuxmlbuild by replacing lower case by upper case in cvename.
Fix vuxmlbuild by replacing lower case by upper case in cvename
Jul 19 2017, 10:38 AM
olivier committed rP446193: Update strongswan to 5.5.3.
Update strongswan to 5.5.3
Jul 19 2017, 10:29 AM
olivier committed rP446192: Document vulnerability in collectd5.
Document vulnerability in collectd5
Jul 19 2017, 10:14 AM
olivier committed rP446191: Document vulnerability in strongswan.
Document vulnerability in strongswan
Jul 19 2017, 9:43 AM

Jul 17 2017

olivier committed rP446072: Update to 5.7.2.
Update to 5.7.2
Jul 17 2017, 1:38 PM

Jul 6 2017

olivier added a comment to D11476: iflib - try m_defrag when m_collapse fails.
In D11476#238004, @ae wrote:

From a quick look, the iflib code does not bind irq to CPU cores. The old em/igb drivers did that and I guess, if you add bus_bind_intr() again, this will increase the performance.

Jul 6 2017, 10:44 PM
olivier added a comment to D11476: iflib - try m_defrag when m_collapse fails.

@olivier - Could you tell me what tools do you use? I'd like to check if forwarding performance on other HW is also affected.

Jul 6 2017, 3:34 PM

Jul 5 2017

olivier added a comment to D11476: iflib - try m_defrag when m_collapse fails.

On the first platform: PC Engines APU2C4 (quad core AMD GX-412T Processor 1 GHz), 3 Intel i210AT Gigabit Ethernet ports

  • LRO/TSO disabled
  • harvest.mask=351
  • 2000 flows of smallest UDP packets
  • Traffic load at 1.448Mpps (Gigabit line-rate)
Jul 5 2017, 12:11 PM