[ovs-dev] [PATCH] dpif/dpcls: limit count subtable search info logs

Van Haaren, Harry harry.van.haaren at intel.com
Thu Jul 1 14:04:14 UTC 2021


> -----Original Message-----
> From: Flavio Leitner <fbl at sysclose.org>
> Sent: Thursday, July 1, 2021 2:55 PM
> To: Van Haaren, Harry <harry.van.haaren at intel.com>
> Cc: Amber, Kumar <kumar.amber at intel.com>; dev at openvswitch.org;
> i.maximets at ovn.org
> Subject: Re: [ovs-dev] [PATCH] dpif/dpcls: limit count subtable search info logs
> 
> On Thu, Jul 01, 2021 at 12:12:18PM +0000, Van Haaren, Harry wrote:

<snip convo>

> > Would we prefer to just print a message to the user?
> > "please run ovs-appctl dpctl/dump-flows -m, and mail results to ovs ML"?
> 
> Thanks for all the answers. I get that it is important to know the
> situation and likely all subtables, but users can run a command to
> dump all that information.
> 
> What about VLOG_INFO_ONCE("Using non-specialized AVX512 lookup for
> subtable (%d,%d) and possibly others.")
> and add a reference to that message in the documentation explaining
> what that means and how to check other subtables? I think that would
> resolve the issue from my point of view.
> 
> How does that sound to you?

Good suggestion, and a workable solution. Will update this patch to 
handle it in that way, and include doc updates too.

Thanks for input! -Harry


More information about the dev mailing list