On Tue, Oct 8, 2019 at 10:39 AM Bruce Richardson <[email protected]> wrote: > > On Mon, Oct 07, 2019 at 08:28:21AM -0700, Jim Harris wrote: > > This ensures secondary processes never have to > > calculate the TSC rate themselves, which can be > > noticeable in VMs that don't have access to > > arch-specific detection mechanism (such as > > CPUID leaf 0x15 or MSR 0xCE on x86). > > > > Since rte_mem_config is now internal to the rte_eal > > library, we can add tsc_hz without ABI breakage > > concerns. > > > > Reduces rte_eal_init() execution time in a secondary > > process from 165ms to 66ms on my test system. > > > > Signed-off-by: Jim Harris <[email protected]> > > --- > This seems a good idea to me. > > Acked-by: Bruce Richardson <[email protected]>
I wonder if we can get rid of eal_tsc_resolution_hz and just rely on the shared memory to get/store this info. Feel free to look at this later and send a followup patch :-). Applied, thanks. -- David Marchand
