All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thorsten Leemhuis <fedora@leemhuis.info>
To: Johannes Weiner <hannes@cmpxchg.org>
Cc: Mel Gorman <mgorman@suse.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Rik van Riel <riel@redhat.com>,
	George Spelvin <linux@horizon.com>,
	Johannes Hirte <johannes.hirte@fem.tu-ilmenau.de>,
	Tomas Racek <tracek@redhat.com>, Jan Kara <jack@suse.cz>,
	Dave Hansen <dave@linux.vnet.ibm.com>,
	Josh Boyer <jwboyer@gmail.com>,
	Valdis Kletnieks <Valdis.Kletnieks@vt.edu>,
	Jiri Slaby <jslaby@suse.cz>, Zdenek Kabelac <zkabelac@redhat.com>,
	Bruno Wolff III <bruno@wolff.to>, linux-mm <linux-mm@kvack.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	John Ellson <john.ellson@comcast.net>
Subject: Re: kswapd craziness in 3.7
Date: Thu, 06 Dec 2012 09:09:06 +0100	[thread overview]
Message-ID: <50C052A2.90303@leemhuis.info> (raw)
In-Reply-To: <20121203194208.GZ24381@cmpxchg.org>

Hi!

Just a quick update

Johannes Weiner wrote on 03.12.2012 20:42:
> On Mon, Dec 03, 2012 at 09:30:12AM +0100, Thorsten Leemhuis wrote:
>
>> BTW, I built that kernel without the patch you mentioned in
>> http://thread.gmane.org/gmane.linux.kernel.mm/90911/focus=91153
>> ("buffer_heads_over_limit can put kswapd into reclaim, but it's ignored
>> [...]) It looked to me like that patch was only meant for debugging. Let
>> me know if that was wrong. Ohh, and I didn't update to a fresher
>> mainline checkout yet to make sure the base for John's testing didn't
>> change.
> 
> Ah, yes, the ApplyPatch is commented out.
> 
> I think we want that upstream as well, but it's not critical.
> [...]

Sorry, it had no "Singed-off-by", so I assumed it was just for debugging.

> Not rebasing sounds reasonable to me to verify the patch.  It might be
> worth testing that the final version that will be 3.8 still works for
> John, however, once that is done.  Just to be sure.

Just to be sure, I yesterday built a rc8 kernel with the patch
referenced above and the one that is not yet merged (these two, to be
precise: http://thread.gmane.org/gmane.linux.kernel.mm/90911/focus=91153
http://thread.gmane.org/gmane.linux.kernel.mm/90911/focus=91300
; all the others patches my kswap test kernels contained earlier were
afaics merged a few days ago) and mentioned it in the Fedora bug report.
John gave them a try and  in
https://bugzilla.redhat.com/show_bug.cgi?id=866988#c65 reported "No
problems so far.  I'll check back again in ~24hours."

CU, Thorsten

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Thorsten Leemhuis <fedora@leemhuis.info>
To: Johannes Weiner <hannes@cmpxchg.org>
Cc: Mel Gorman <mgorman@suse.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Rik van Riel <riel@redhat.com>,
	George Spelvin <linux@horizon.com>,
	Johannes Hirte <johannes.hirte@fem.tu-ilmenau.de>,
	Tomas Racek <tracek@redhat.com>, Jan Kara <jack@suse.cz>,
	Dave Hansen <dave@linux.vnet.ibm.com>,
	Josh Boyer <jwboyer@gmail.com>,
	Valdis Kletnieks <Valdis.Kletnieks@vt.edu>,
	Jiri Slaby <jslaby@suse.cz>, Zdenek Kabelac <zkabelac@redhat.com>,
	Bruno Wolff III <bruno@wolff.to>, linux-mm <linux-mm@kvack.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	John Ellson <john.ellson@comcast.net>
Subject: Re: kswapd craziness in 3.7
Date: Thu, 06 Dec 2012 09:09:06 +0100	[thread overview]
Message-ID: <50C052A2.90303@leemhuis.info> (raw)
In-Reply-To: <20121203194208.GZ24381@cmpxchg.org>

Hi!

Just a quick update

Johannes Weiner wrote on 03.12.2012 20:42:
> On Mon, Dec 03, 2012 at 09:30:12AM +0100, Thorsten Leemhuis wrote:
>
>> BTW, I built that kernel without the patch you mentioned in
>> http://thread.gmane.org/gmane.linux.kernel.mm/90911/focus=91153
>> ("buffer_heads_over_limit can put kswapd into reclaim, but it's ignored
>> [...]) It looked to me like that patch was only meant for debugging. Let
>> me know if that was wrong. Ohh, and I didn't update to a fresher
>> mainline checkout yet to make sure the base for John's testing didn't
>> change.
> 
> Ah, yes, the ApplyPatch is commented out.
> 
> I think we want that upstream as well, but it's not critical.
> [...]

Sorry, it had no "Singed-off-by", so I assumed it was just for debugging.

> Not rebasing sounds reasonable to me to verify the patch.  It might be
> worth testing that the final version that will be 3.8 still works for
> John, however, once that is done.  Just to be sure.

Just to be sure, I yesterday built a rc8 kernel with the patch
referenced above and the one that is not yet merged (these two, to be
precise: http://thread.gmane.org/gmane.linux.kernel.mm/90911/focus=91153
http://thread.gmane.org/gmane.linux.kernel.mm/90911/focus=91300
; all the others patches my kswap test kernels contained earlier were
afaics merged a few days ago) and mentioned it in the Fedora bug report.
John gave them a try and  in
https://bugzilla.redhat.com/show_bug.cgi?id=866988#c65 reported "No
problems so far.  I'll check back again in ~24hours."

CU, Thorsten

  parent reply	other threads:[~2012-12-06  8:09 UTC|newest]

Thread overview: 129+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-27 20:48 kswapd craziness in 3.7 Johannes Weiner
2012-11-27 20:48 ` Johannes Weiner
2012-11-27 20:48 ` [patch] mm: vmscan: fix kswapd endless loop on higher order allocation Johannes Weiner
2012-11-27 20:48   ` Johannes Weiner
2012-11-27 20:58 ` kswapd craziness in 3.7 Linus Torvalds
2012-11-27 20:58   ` Linus Torvalds
2012-11-27 21:16   ` Rik van Riel
2012-11-27 21:16     ` Rik van Riel
2012-11-27 21:49     ` Johannes Weiner
2012-11-27 21:49       ` Johannes Weiner
2012-11-27 22:02       ` Rik van Riel
2012-11-27 22:02         ` Rik van Riel
2012-11-27 22:26         ` Johannes Weiner
2012-11-27 22:26           ` Johannes Weiner
2012-11-27 23:19           ` Linus Torvalds
2012-11-27 23:19             ` Linus Torvalds
2012-11-28 10:13             ` Mel Gorman
2012-11-28 10:13               ` Mel Gorman
2012-11-28 10:51               ` Thorsten Leemhuis
2012-11-28 10:51                 ` Thorsten Leemhuis
2012-11-28 16:42               ` Mel Gorman
2012-11-28 16:42                 ` Mel Gorman
2012-11-28 22:52               ` Andrew Morton
2012-11-28 22:52                 ` Andrew Morton
2012-11-28 23:54                 ` Mel Gorman
2012-11-28 23:54                   ` Mel Gorman
2012-11-29  0:14                   ` Andrew Morton
2012-11-29  0:14                     ` Andrew Morton
2012-11-29 15:30                   ` Thorsten Leemhuis
2012-11-29 15:30                     ` Thorsten Leemhuis
2012-11-29 17:05                     ` Johannes Weiner
2012-11-29 17:05                       ` Johannes Weiner
2012-11-30 12:39                       ` Thorsten Leemhuis
2012-11-30 12:39                         ` Thorsten Leemhuis
2012-12-01  0:45                         ` Johannes Weiner
2012-12-01  0:45                           ` Johannes Weiner
2012-12-03  8:30                           ` Thorsten Leemhuis
2012-12-03  8:30                             ` Thorsten Leemhuis
2012-12-03 13:08                             ` Fedora repo (was: Re: kswapd craziness in 3.7) Borislav Petkov
2012-12-03 13:08                               ` Borislav Petkov
2012-12-03 19:42                             ` kswapd craziness in 3.7 Johannes Weiner
2012-12-03 19:42                               ` Johannes Weiner
2012-12-04 21:42                               ` Johannes Weiner
2012-12-04 21:42                                 ` Johannes Weiner
2012-12-05  3:01                                 ` Bruno Wolff III
2012-12-05  3:01                                   ` Bruno Wolff III
2012-12-06 17:37                                   ` Bruno Wolff III
2012-12-06 17:37                                     ` Bruno Wolff III
2012-12-06 19:31                                     ` Linus Torvalds
2012-12-06 19:31                                       ` Linus Torvalds
2012-12-06 19:43                                       ` Rik van Riel
2012-12-06 19:43                                         ` Rik van Riel
2012-12-06 20:23                                       ` Johannes Weiner
2012-12-06 20:23                                         ` Johannes Weiner
2012-12-06 20:32                                         ` Rik van Riel
2012-12-06 20:32                                           ` Rik van Riel
2012-12-08 12:06                                       ` Zlatko Calusic
2012-12-08 12:06                                         ` Zlatko Calusic
2012-12-08 21:22                                         ` Zlatko Calusic
2012-12-08 21:22                                           ` Zlatko Calusic
2012-12-09  1:01                                           ` Linus Torvalds
2012-12-09  1:01                                             ` Linus Torvalds
2012-12-09 21:59                                             ` Zdenek Kabelac
2012-12-09 21:59                                               ` Zdenek Kabelac
2012-12-10 11:03                                             ` Mel Gorman
2012-12-10 11:03                                               ` Mel Gorman
2012-12-10 16:39                                               ` Johannes Weiner
2012-12-10 16:39                                                 ` Johannes Weiner
2012-12-10 18:01                                                 ` Mel Gorman
2012-12-10 18:01                                                   ` Mel Gorman
2012-12-10 18:33                                                   ` Zlatko Calusic
2012-12-10 18:33                                                     ` Zlatko Calusic
2012-12-10 19:13                                                     ` Linus Torvalds
2012-12-10 19:13                                                       ` Linus Torvalds
2012-12-10 20:35                                                       ` Zlatko Calusic
2012-12-10 20:35                                                         ` Zlatko Calusic
2012-12-10 21:28                                                         ` Linus Torvalds
2012-12-10 21:28                                                           ` Linus Torvalds
2012-12-10 21:42                                                           ` Borislav Petkov
2012-12-10 21:42                                                             ` Borislav Petkov
2012-12-10 21:47                                                             ` Linus Torvalds
2012-12-10 21:47                                                               ` Linus Torvalds
2012-12-10 21:54                                                               ` Borislav Petkov
2012-12-10 21:54                                                                 ` Borislav Petkov
2012-12-10 22:15                                                                 ` Zlatko Calusic
2012-12-10 22:15                                                                   ` Zlatko Calusic
2012-12-10 23:27                                                           ` Hugh Dickins
2012-12-10 23:27                                                             ` Hugh Dickins
2012-12-11  0:19                                                         ` Zlatko Calusic
2012-12-11 21:56                                                           ` Zlatko Calusic
2012-12-11 21:56                                                             ` Zlatko Calusic
2012-12-19 22:24                                                           ` Zlatko Calusic
2012-12-19 22:24                                                             ` Zlatko Calusic
2012-12-10 18:29                                               ` Zlatko Calusic
2012-12-10 18:29                                                 ` Zlatko Calusic
2012-12-06  8:09                               ` Thorsten Leemhuis [this message]
2012-12-06  8:09                                 ` Thorsten Leemhuis
2012-11-27 21:29   ` Johannes Weiner
2012-11-27 21:29     ` Johannes Weiner
2012-11-28 13:35   ` Zdenek Kabelac
2012-11-28 13:35     ` Zdenek Kabelac
2012-11-28 14:04     ` Jiri Slaby
2012-11-28 14:04       ` Jiri Slaby
2012-11-28  9:45 ` Mel Gorman
2012-11-28  9:45   ` Mel Gorman
2012-12-03 15:23   ` Zdenek Kabelac
2012-12-03 15:23     ` Zdenek Kabelac
2012-12-03 19:18     ` Johannes Weiner
2012-12-03 19:18       ` Johannes Weiner
2012-12-04  9:05       ` Zdenek Kabelac
2012-12-04  9:05         ` Zdenek Kabelac
2012-12-04  9:15         ` Jiri Slaby
2012-12-04  9:15           ` Jiri Slaby
2012-12-04 16:11           ` Johannes Weiner
2012-12-04 16:11             ` Johannes Weiner
2012-12-04 16:22             ` Jiri Slaby
2012-12-04 16:22               ` Jiri Slaby
2012-12-04 19:50               ` Johannes Weiner
2012-12-04 19:50                 ` Johannes Weiner
2012-12-08 10:35             ` Jiri Slaby
2012-12-08 10:35               ` Jiri Slaby
2012-12-04 16:15         ` Johannes Weiner
2012-12-04 16:15           ` Johannes Weiner
2012-12-06 13:51         ` Zdenek Kabelac
2012-12-06 13:51           ` Zdenek Kabelac
2012-12-03 13:14 ` Jiri Slaby
2012-12-03 13:14   ` Jiri Slaby
2012-12-04  8:55   ` Jiri Slaby
2012-12-04  8:55     ` Jiri Slaby

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50C052A2.90303@leemhuis.info \
    --to=fedora@leemhuis.info \
    --cc=Valdis.Kletnieks@vt.edu \
    --cc=akpm@linux-foundation.org \
    --cc=bruno@wolff.to \
    --cc=dave@linux.vnet.ibm.com \
    --cc=hannes@cmpxchg.org \
    --cc=jack@suse.cz \
    --cc=johannes.hirte@fem.tu-ilmenau.de \
    --cc=john.ellson@comcast.net \
    --cc=jslaby@suse.cz \
    --cc=jwboyer@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux@horizon.com \
    --cc=mgorman@suse.de \
    --cc=riel@redhat.com \
    --cc=torvalds@linux-foundation.org \
    --cc=tracek@redhat.com \
    --cc=zkabelac@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.