[PATCH v6 1/5] eal: add thread set name API operating on rte thread

Tyler Retzlaff roretzla at linux.microsoft.com
Thu Jan 19 00:44:34 CET 2023


On Wed, Jan 18, 2023 at 03:13:12PM -0800, Stephen Hemminger wrote:
> On Wed, 18 Jan 2023 11:54:02 -0800
> Tyler Retzlaff <roretzla at linux.microsoft.com> wrote:
> 
> > +	if (name != NULL)
> > +		rte_thread_set_name((rte_thread_t){(uintptr_t)*thread}, name)
> 
> Do we really need so many casts here? Looks like the wrong type was passed
> in the first place?

we do until this code is converted to use rte_thread_create. the
pthread_t needs to be casted to the uintptr_t (implementation detail of
rte_thread_t) and rte_thread_t is required by rte_thread_set_name.


More information about the dev mailing list