lkml.org 
[lkml]   [2010]   [Jun]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: Overview of concurrency managed workqueue
    Hello,

    On 06/16/2010 09:46 PM, Daniel Walker wrote:
    >> If you actually _solve_ the problem like this, other users wouldn't
    >> experience the problem at all once the update reaches them and you
    >> won't have to worry about your workaround breaking with the next
    >> kernel update or unexpected suspend/resume and we won't be having this
    >> discussion about adjusting workqueue priorities from userland.
    >
    > What you suggesting just means the user has to adjust an interrupt
    > thread instead of a workqueue thread. That really doesn't change
    > anything, since it's just another type of kernel thread.

    It allows writing an interrupt handler which requires context and
    PREEMPT_RT kernel will handle priority inheritance correctly with
    them, so there's no priority inversion problem.

    >> There are many wrong things about working around RT latency problems
    >> by setting workqueue priorities from userland. Please think about why
    >> the driver would have a separate workqueue for itself in the first
    >> place. It was to work around the limitation of workqueue facility and
    >> you're arguing that, because that work around allows yet another very
    >> fragile workaround, the property which made the original work around
    >> necessary in the first place needs to stay. That sounds really
    >> perverse to me.
    >
    > I have no idea what your trying to say here.. I'm sure there is no one
    > reason why people use workqueues in their drivers. In fact I'm sure
    > there are many reasons to use workqueues or not to use them.

    I mean that if workqueues don't have current limitations, there's no
    reason for most drivers which aren't in the allocation path to use
    separate workqueues. They can simply use the default system one.

    Thanks.

    --
    tejun


    \
     
     \ /
      Last update: 2010-06-16 22:01    [W:4.118 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site