public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Jan Stancek <jstancek@redhat.com>
To: Li Wang <liwang@redhat.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH] mem/hugeshmat: new case for hugepage leak inspection
Date: Wed, 7 Jan 2015 07:32:34 -0500 (EST)	[thread overview]
Message-ID: <1910508304.3829751.1420633954389.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <169945440.3971026.1420553236395.JavaMail.zimbra@redhat.com>





----- Original Message -----
> From: "Li Wang" <liwang@redhat.com>
> To: "Jan Stancek" <jstancek@redhat.com>
> Cc: ltp-list@lists.sourceforge.net
> Sent: Tuesday, 6 January, 2015 3:07:16 PM
> Subject: Re: [LTP] [PATCH] mem/hugeshmat: new case for hugepage leak inspection
> 
> > I'm assuming that both absolute address and sleep is not necessary.
> > I think I'll take some recent kernel, revert that patch and try to
> > reproduce
> > the failure.
> 
> Hi Jan,
> 
> I just tried kernel-3.10.0+ without the fixed patch, and I hit a CPU stuck
> issue
> with absolute address and sleep(3).

I tried the same with 2.6.18-92.el5. When I revert the patch I can reproduce the leak.
sleep(3) doesn't seem to have any effect, I think we can safely remove it.
Attaching at BOUNDARY however seems necessary. As soon as I change shmat to use "NULL",
the leak no longer happens, I'm not sure why that is.

+        if (huge_total != hugepages || huge_free != hugepages)
This condition in setup() seems to strict. I think we don't need all hugepages
to be free when test starts. 

Regards,
Jan


------------------------------------------------------------------------------
Dive into the World of Parallel Programming! The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2015-01-07 12:32 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-27  7:13 [LTP] [PATCH] mem/hugeshmat: new case for hugepage leak inspection Li Wang
2014-12-30  9:23 ` Jan Stancek
2015-01-04  3:10   ` Li Wang
2015-01-05 10:16     ` Jan Stancek
2015-01-06  9:51       ` Li Wang
2015-01-06 10:35         ` Jan Stancek
2015-01-06 14:07           ` Li Wang
2015-01-07 12:32             ` Jan Stancek [this message]
2015-01-07 14:59               ` Li Wang
2015-01-26 16:48                 ` Cyril Hrubis
  -- strict thread matches above, loose matches on Subject: below --
2014-12-28  3:20 Li Wang
     [not found] <1419737900-6117-1-git-send-email-liwang@redhat.com>
2014-12-28  4:06 ` Li Wang
     [not found] <1422336738-617-1-git-send-email-liwang@redhat.com>
2015-01-27 15:31 ` Cyril Hrubis

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=1910508304.3829751.1420633954389.JavaMail.zimbra@redhat.com \
    --to=jstancek@redhat.com \
    --cc=liwang@redhat.com \
    --cc=ltp-list@lists.sourceforge.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox