[dpdk-dev] [PATCH v8 1/9] eal: move OS common functions to single file

Thomas Monjalon thomas at monjalon.net
Tue Jun 23 09:16:02 CEST 2020


23/06/2020 08:45, Tal Shnaiderman:
> > 22/06/2020 09:55, talshn at mellanox.com:
> > > +struct internal_config *
> > > +rte_eal_get_internal_configuration(void)
> > > +{
> > > +	return &internal_config;
> > > +}
> > 
> > This function should be private to EAL, no rte_ prefix.
> 
> When creating the new functions I based the pattern on the already existing function
> rte_eal_get_configuration() which encapsulates the formally global rte_config.
> 
> Do we want to remove the rte_ prefix from this function as well?

No, don't change the existing function.
It could be a separate cleanup.





More information about the dev mailing list