On Wed, Jan 18, 2023 at 03:13:12PM -0800, Stephen Hemminger wrote:
> On Wed, 18 Jan 2023 11:54:02 -0800
> Tyler Retzlaff <roret...@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.

Reply via email to