On 11/5/24 12:08, Ilya Maximets wrote: > 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?
More importantly, why can't we wait half a second until the next revalidation? > >> 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
