On Wed, Dec 15, 2021 at 5:37 PM Van Haaren, Harry <[email protected]> wrote: > > I'll keep the current implementation, but respin with an ovs_assert() > > where needed as agreed previously. > > More high level logs might be added in dpif in the future. > > Hi Ilya & David, > > Sorry, I'm not following what we are trying to achieve here. My understanding > as follows; > - David sends patch to change the CPU detection to be in OVS, using CPUID > impl (~ like DPDK) > - I raised concerns, suggesting an alternative implementation still using > CPUID but better logging > - Ilya raises concerns around using CPUID "as a whole", and notes that > constructors on Windows are OK > - [About a week goes by, I was waiting for technical input/response from > David] > - David suggests "OK, will keep current implementation" (with minor > modifications for assert)
Ilya comments made sense, I was expeecting replies from your side but seen none. So I understood this as you were ok with Ilya comments, so I only updated the parts we agreed on previously. -- David Marchand _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
