Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure

On Mon, 2007-08-06 at 20:50 +0400, Oleg Nesterov wrote:
> Yes.
> 
> > Do you agree that if the context was the same there is a bug?  Or did I
> > miss something else?
> 
> Yes sure. We can't expect we can "flush" work_struct with flush_workqueue()
> unless we know it doesn't re-schedule itself.

Agreed.

> 
> OTOH, it is not the bug to call flush_workqueue() even if that work was
> queued by us, it should complete.

Well, I guess it depends on the application but that would be highly
unusual unless the flush was already superfluous to begin with.
Typically you only call flush to ensure a strongly ordered operation.
The reschedule defeats the strong ordering and thus would break anything
that was dependent on it.

But we are splitting hairs at this point since we both agree that the
API as put forth in the PI patch was deficient ;)

-Greg

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Oleg Nesterov <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Gregory Haskins <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Oleg Nesterov <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Gregory Haskins <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Oleg Nesterov <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Gregory Haskins <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Oleg Nesterov <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Gregory Haskins <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Oleg Nesterov <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Gregory Haskins <[email protected]>
  • Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
    • From: Oleg Nesterov <[email protected]>

• Prev by Date: Re: [alsa-devel] PROBLEM: snd-au8830: dead after software suspend
• Next by Date: Re: Disk spin down issue on shut down/suspend to disk
• Previous by thread: Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
• Next by thread: Re: [PATCH] RT: Add priority-queuing and priority-inheritance to workqueue infrastructure
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]