Yes, I can commit both patches when it's ready.
On Mon, Oct 21, 2013 at 2:36 PM, Andy Zhou <az...@nicira.com> wrote:
> I will send out a V4 to address those remaining issues. Ben has reviewed and
> acked the user space part of
> patch, would you please consider commit both if v4 looks O.K.?
>
>
> On Fri, Oct 18, 2013 at 5:35 PM, Jesse Gross <je...@nicira.com> wrote:
>>
>> On Fri, Oct 18, 2013 at 4:54 PM, Andy Zhou <az...@nicira.com> wrote:
>> > Collect mega flow mask stats. ovs-dpctl show command can be used to
>> > display them.
>> >
>> > Signed-off-by: Andy Zhou <az...@nicira.com>
>>
>> I noticed a few things that I was originally planning on just
>> correcting but it was a little more than I usually would like to do.
>>
>> > diff --git a/datapath/datapath.c b/datapath/datapath.c
>> > index 9e6df12..4ac5fb8 100644
>> > --- a/datapath/datapath.c
>> > +++ b/datapath/datapath.c
>> > -static void get_dp_stats(struct datapath *dp, struct ovs_dp_stats
>> > *stats)
>> > +static void get_dp_stats(struct datapath *dp, struct ovs_dp_stats
>> > *stats,
>> > + struct ovs_dp_megaflow_stats *mega_stats)
>> > {
>> > int i;
>> >
>> > + memset(mega_stats, 0, sizeof(*mega_stats);
>>
>> There's a compiler error here due to a missing parenthesis.
>>
>> > stats->n_flows = ovs_flow_tbl_count(&dp->table);
>> > + mega_stats->n_masks = ovs_flow_tbl_num_masks(&dp->table);
>> >
>> > stats->n_hit = stats->n_missed = stats->n_lost = 0;
>> > + mega_stats->n_mask_hit = 0;
>>
>> This probably isn't necessary any more since we zero out the whole struct.
>>
>> > +struct ovs_dp_megaflow_stats {
>> > + __u64 n_masks; /* Number of masks for the datapath */
>> > + __u32 n_mask_hit; /* Number of masks used for flow
>> > lookups. */
>>
>> Aren't the respective sizes of these two fields reversed?
>
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
