[ovs-dev] [PATCH v4 ovn] ovn-nbctl: add the capability to specify CoPP name

2022-01-29 Thread Lorenzo Bianconi
Introduce the capability to specify CoPP name in order to reuse the same CoPP reference on multiple datapaths. Introduce the following CoPP commands: - ovn-nbctl copp-add - ovn-nbctl copp-del - ovn-nbctl copp-list - ovn-nbctl ls-copp-add - ovn-nbctl lr-copp-add Introduce name and externa

Re: [ovs-dev] [PATCH v4 ovn] ovn-nbctl: add the capability to specify CoPP name

2022-01-29 Thread 0-day Robot
Bleep bloop. Greetings Lorenzo Bianconi, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line lacks whitespace around operator #573 FILE: utilities/ovn-nbctl.c:440: copp

[ovs-dev] [PATCH v2] daemon-unix: Close log file in monitor process while waiting on child.

2022-01-29 Thread Ben Pfaff
Until now, the monitor process held its log file open while it waited for the child to exit, and then it reopened it after the child exited. Holding the log file open this way prevented log rotation from freeing disk space. This commit avoids the problem by closing the log file before waiting, the

Re: [ovs-dev] [PATCH] daemon-unix: Close log file in monitor process while waiting on child.

2022-01-29 Thread Ben Pfaff
On Wed, Jan 26, 2022 at 06:39:52PM -0800, William Tu wrote: > On Wed, Jan 26, 2022 at 8:56 AM Ben Pfaff wrote: > > > > Until now, the monitor process held its log file open while it waited for > > the child to exit, and then it reopened it after the child exited. Holding > > the log file open thi