On 11/5/24 12:01, Ales Musil wrote:
> 
> 
> On Tue, Nov 5, 2024 at 11:47 AM Ilya Maximets <[email protected] 
> <mailto:[email protected]>> wrote:
> 
>     On 11/5/24 09:30, Ales Musil wrote:
>     > The ofproto/detrace command wouldn't output anything when the cache
>     > is empty. Add option to explicitly populate the cache when empty.
> 
>     Hi, Ales.  Thanks for the patch!  But why is this needed?
>     The cache is only not available until the first revalidation
>     that should happen shortly after the flow installation.
> 
>     I still think that practically revalidating the ukey from
>     a debug appctl is not a good thing to do.  So, I'd like to
>     avoid that, unless there is a good reason for doing so.
> 
>     We may print a message that translation cache is not available
>     yet, if that's the concern.
> 
>     Best regards, Ilya Maximets.
> 
> 
> Hi Ilya,
> 
> there are certain situations when there isn't any output
> but it would be nice to have one. The example that comes to
> my mind is arp sent from ovn-controller.

Why do we have a datapath flow for a packet generated by the
controller?

> I understand that it shouldn't
> be done all the time, but upon request it seems reasonable.

The problem with this is if there is an option, users will use
it every time.

> 
> Thanks,
> Ales
> 
> -- 
> 
> Ales Musil
> 
> Senior Software Engineer - OVN Core
> 
> Red Hat EMEA <https://www.redhat.com>
> 
> [email protected] <mailto:[email protected]> 
> 
> <https://red.ht/sig>
> 

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to