Re: [ovs-dev] [PATCH ovn v4 4/6] northd: Introduce struct northd_data

2021-11-08 Thread Mark Gray
On Wed, Nov 3, 2021 at 11:37 PM Han Zhou wrote: > On Wed, Oct 27, 2021 at 10:24 AM Mark Gray wrote: > > > > 'struct northd_data' is used to hold the global state data for the > > incremental processing node 'en_northd'. This structure will also hold > > 'struct northd_input' which will hold refe

Re: [ovs-dev] [PATCH ovn v4 4/6] northd: Introduce struct northd_data

2021-11-03 Thread Han Zhou
On Wed, Oct 27, 2021 at 10:24 AM Mark Gray wrote: > > 'struct northd_data' is used to hold the global state data for the > incremental processing node 'en_northd'. This structure will also hold > 'struct northd_input' which will hold references to output data from > its input nodes. In particular,

Re: [ovs-dev] [PATCH ovn v4 4/6] northd: Introduce struct northd_data

2021-10-27 Thread Mark Gray
On 27/10/2021 18:53, 0-day Robot wrote: > Bleep bloop. Greetings Mark Gray, 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: Comment with 'xxx' marker > #14

Re: [ovs-dev] [PATCH ovn v4 4/6] northd: Introduce struct northd_data

2021-10-27 Thread 0-day Robot
Bleep bloop. Greetings Mark Gray, 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: Comment with 'xxx' marker #1478 FILE: northd/northd.c:14488: /* XXX Having to explicit

[ovs-dev] [PATCH ovn v4 4/6] northd: Introduce struct northd_data

2021-10-27 Thread Mark Gray
'struct northd_data' is used to hold the global state data for the incremental processing node 'en_northd'. This structure will also hold 'struct northd_input' which will hold references to output data from its input nodes. In particular, this will hold references to database tables and indexes. In