> -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Friday, October 30, 2020 5:02 AM > To: McDaniel, Timothy <[email protected]> > Cc: [email protected]; Carrillo, Erik G <[email protected]>; Eads, Gage > <[email protected]>; Van Haaren, Harry <[email protected]>; > [email protected]; [email protected] > Subject: Re: [PATCH v5 00/23] Add DLB2 PMD > > 30/10/2020 10:43, Timothy McDaniel: > > - note that the code still uses its private byte-encoded versions of > > umonitor/umwait, rather than the new functions in the power > > patch that are built on top of those intrinsics. This is intentional. > > Why? Now these intrinsics are available in the main branch. > We should avoid duplicating such code. > >
I had asked that the low level intrinsics (UMWAIT/UMONITOR) be split out so that DLB/DLB2 could use them instead of its own private byte-encoded versions, but instead we have these wrappers that call the low level intrinsics. Those wrappers introduce additional overhead that is not required for DLB/DLB2. I have a meeting with Ma Liang on Monday to discuss.
