Re: Problem with freezable workqueues



On Wed 2007-02-28 23:39:30, Rafael J. Wysocki wrote:
On Wednesday, 28 February 2007 21:35, Oleg Nesterov wrote:
On 02/28, Rafael J. Wysocki wrote:

Okay, but I've just finished the patch that removes the freezability of
workqueues (appended), so can we please do this in a separate one?

Please, please, no. This patch is of course correct, but it breaks _a lot_
of patches in -mm tree.

May I ask you to send just

===================================================================
--- linux-2.6.21-rc2.orig/fs/xfs/linux-2.6/xfs_buf.c
+++ linux-2.6.21-rc2/fs/xfs/linux-2.6/xfs_buf.c
@@ -1829,11 +1829,11 @@ xfs_buf_init(void)
if (!xfs_buf_zone)
goto out_free_trace_buf;

- xfslogd_workqueue = create_freezeable_workqueue("xfslogd");
+ xfslogd_workqueue = create_workqueue("xfslogd");
if (!xfslogd_workqueue)
goto out_free_buf_zone;

- xfsdatad_workqueue = create_freezeable_workqueue("xfsdatad");
+ xfsdatad_workqueue = create_workqueue("xfsdatad");
if (!xfsdatad_workqueue)
goto out_destroy_xfslogd_workqueue;



this bit?

After that, we can do the "removes the freezability of workqueues" patch
against -mm tree.

Okay, if that's better.

Pavel, is that acceptable to you?

No problem, but get that acked-by: from XFS people ;-).
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • Re: Problem with freezable workqueues
    ... This patch is of course correct, ... we can do the "removes the freezability of workqueues" patch ... Okay, if that's better. ...
    (Linux-Kernel)
  • Re: Problem with freezable workqueues
    ... This patch is of course correct, ... of patches in -mm tree. ... we can do the "removes the freezability of workqueues" patch ...
    (Linux-Kernel)
  • Re: Fix up power managment in 2.6
    ... That is, until the latest round of patches, in which the ... Okay, my point was that the patch was not exactly obvious. ... Trying to fix error handling is good and welcome, ...
    (Linux-Kernel)
  • Re: Still here
    ... but really it was only a couple of days of feeling not-so-good, then it got okay again. ... After the first 2 nights of very little sleep punctuated by extremely detailed dreams, I went back to the pharmacist for more advice. ... then I can always put on a patch. ... than 6 weeks to get me through one certain upcoming vulnerable event on ...
    (alt.support.stop-smoking)
  • Re: Patch to add support for SGIs IOC4 chipset
    ... > Okay, ... I found that I was able to get rid of all of them. ... Attached is the latest patch, which also includes your patch sent previously. ... send the line "unsubscribe linux-kernel" in ...
    (Linux-Kernel)