All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mel Gorman <mel@csn.ul.ie>
To: Alex Villac??s Lasso <avillaci@fiec.espol.edu.ec>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	avillaci@ceibo.fiec.espol.edu.ec,
	bugzilla-daemon@bugzilla.kernel.org,
	Andrea Arcangeli <aarcange@redhat.com>,
	bugme-daemon@bugzilla.kernel.org, linux-mm@kvack.org
Subject: Re: [Bugme-new] [Bug 31142] New: Large write to USB stick freezes unrelated tasks for a long time
Date: Sat, 19 Mar 2011 13:46:28 +0000	[thread overview]
Message-ID: <20110319134628.GG707@csn.ul.ie> (raw)
In-Reply-To: <4D839EDB.9080703@fiec.espol.edu.ec>

On Fri, Mar 18, 2011 at 01:05:15PM -0500, Alex Villac??s Lasso wrote:
> El 18/03/11 06:13, Mel Gorman escribio:
> >
> >\o/ ... no wait, it's the other one - :(
> >
> >If you look at the stack traces though, all of them had called
> >do_huge_pmd_anonymous_page() so while it looks similar to 12309, the trigger
> >is new because it's THP triggering compaction that is causing the stalls
> >rather than page reclaim doing direct writeback which was the culprit in
> >the past.
> >
> >To confirm if this is the case, I'd be very interested in hearing if this
> >problem persists in the following cases
> >
> >1. 2.6.38-rc8 with defrag disabled by
> >    echo never>/sys/kernel/mm/transparent_hugepage/defrag
> >    (this will stop THP allocations calling into compaction)
> >2. 2.6.38-rc8 with THP disabled by
> >    echo never>
> >/sys/kernel/mm/transparent_hugepage/enabled
> >    (if the problem still persists, then page reclaim is still a problem
> >     but we should still stop THP doing sync writes)
> >3. 2.6.37 vanilla
> >    (in case this is a new regression introduced since then)
> >
> >Migration can do sync writes on dirty pages which is why it looks so similar
> >to page reclaim but this can be controlled by the value of sync_migration
> >passed into try_to_compact_pages(). If we find that option 1 above makes
> >the regression go away or at least helps a lot, then a reasonable fix may
> >be to never set sync_migration if __GFP_NO_KSWAPD which is always set for
> >THP allocations. I've added Andrea to the cc to see what he thinks.
> >
> >Thanks for the report.
> >
>
> I have just done tests 1 and 2 on 2.6.38 (final, not -rc8), and I
> have verified that echoing "never" on either
> /sys/kernel/mm/transparent_hugepage/defrag or
> /sys/kernel/mm/transparent_hugepage/enabled does allow the file copy
> to USB to proceed smoothly (copying 4GB of data). Just to verify, I
> later wrote "always" to both files, and sure enough, some
> applications stalled when I repeated the same file copy. So I have
> at least a workaround for the issue. Given this evidence, will the
> patch at comment #14 fix the issue for good?
> 

Thanks for testing and reporting, it's very helpful. Based on that that
report the patch should help. Can you test it to be absolutly sure please?

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2011-03-19 13:46 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <bug-31142-10286@https.bugzilla.kernel.org/>
2011-03-15 20:53 ` [Bugme-new] [Bug 31142] New: Large write to USB stick freezes unrelated tasks for a long time Andrew Morton
2011-03-15 22:53   ` Alex Villací­s Lasso
2011-03-15 23:19     ` Andrew Morton
2011-03-16 15:25       ` Alex Villací­s Lasso
2011-03-16 22:02         ` Andrew Morton
2011-03-17 21:27           ` Alex Villací­s Lasso
2011-03-17 21:47             ` Andrew Morton
2011-03-17 22:11               ` Alex Villací­s Lasso
2011-03-17 22:25                 ` Andrew Morton
2011-03-18 11:13               ` Mel Gorman
2011-03-18 12:26                 ` Andrea Arcangeli
2011-03-18 18:05                 ` Alex Villací­s Lasso
2011-03-19 13:46                   ` Mel Gorman [this message]
2011-03-19 16:04                     ` Alex Villací­s Lasso
2011-03-19 23:51                       ` Andrea Arcangeli
2011-03-21  9:41                         ` Mel Gorman
2011-03-21 13:48                           ` Andrea Arcangeli
2011-03-21 15:22                             ` Alex Villací­s Lasso
2011-03-21 15:36                               ` Alex Villací­s Lasso
2011-03-21 15:40                               ` Andrea Arcangeli
2011-03-21 16:37                             ` Mel Gorman
2011-03-21 17:05                               ` Alex Villací­s Lasso
2011-03-21 20:16                                 ` Andrea Arcangeli
2011-03-21 23:35                                   ` Alex Villací­s Lasso
2011-03-22 11:20                                   ` Mel Gorman
2011-03-22 15:03                                     ` Andrea Arcangeli
2011-03-22 20:34                                       ` Alex Villací­s Lasso
2011-03-22 21:40                                         ` Andrea Arcangeli
2011-03-23  0:37                                           ` Andrea Arcangeli
2011-03-23 16:51                                             ` Alex Villací­s Lasso
2011-04-04 15:37                                               ` Alex Villací­s Lasso
2011-04-08 19:09                                                 ` Andrea Arcangeli
2011-04-08 20:06                                                   ` Alex Villací­s Lasso
2011-04-12 16:27                                                     ` Alex Villací­s Lasso
2011-04-14 17:25                                                       ` Alex Villací­s Lasso
2011-04-14 17:37                                                         ` Andrea Arcangeli

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=20110319134628.GG707@csn.ul.ie \
    --to=mel@csn.ul.ie \
    --cc=aarcange@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=avillaci@ceibo.fiec.espol.edu.ec \
    --cc=avillaci@fiec.espol.edu.ec \
    --cc=bugme-daemon@bugzilla.kernel.org \
    --cc=bugzilla-daemon@bugzilla.kernel.org \
    --cc=linux-mm@kvack.org \
    /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.