> Subject: Re: [dpdk-dev] [PATCH v5 09/10] eal: add EAL argument for setting
> thread priority
> 
> External email: Use caution opening links or attachments
> 
> 
> On Mon, 29 Mar 2021 15:40:39 -0700
> Narcisa Ana Maria Vasile <navas...@linux.microsoft.com> wrote:
> 
> > From: Narcisa Vasile <navas...@microsoft.com>
> >
> > Allow the user to choose the thread priority through an EAL command
> > line argument.
> >
> > The user can select the thread priority to be either 'normal'
> > or 'critical':
> > --thread-prio normal
> > --thread-prio realtime
> >
> > Signed-off-by: Narcisa Vasile <navas...@microsoft.com>
> 
> The discussion internally was that this was intended to resolve issues on
> Windows.
> So it makes sense for Windows, but it is not something that we want to have
> on Linux.
> Could you make this Windows only, and add update the documentation
> please.
> 
> I just don't want Linux users discovering it, trying it, then reporting more
> bugs.

Windows needs it from performance aspects.

However if we're keeping this option for Windows a warning like the one below 
from MSDN [1] should be added to docu:

" A thread with a base priority level above 11 interferes with the normal 
operation of the operating system. Using REALTIME_PRIORITY_CLASS may cause disk 
caches to not flush, cause the mouse to stop responding, and so on. "

[1] - 
https://docs.microsoft.com/en-us/windows/win32/api/processthreadsapi/nf-processthreadsapi-setthreadpriority


Reply via email to