public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Caspar Zhang <czhang@redhat.com>
To: Garrett Cooper <yanegomi@gmail.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH v4] make pid_list dynamically sized with memory
Date: Sun, 16 Jan 2011 19:05:38 +0800	[thread overview]
Message-ID: <4D32D102.4040506@redhat.com> (raw)
In-Reply-To: <1294339211-19449-1-git-send-email-czhang@redhat.com>

On 01/07/2011 02:40 AM, czhang@redhat.com wrote:
> We get segfaults during testing mtest01 on a 5TB memory machine, the
> problem was traced to the array pid_list[] went to overflow and
> corrupted memory. This fix makes pid_list dynamically sized with
> correct memory size to avoid overflow.
> 
> v2: not split into different bits when allocating pid_list.
> v3: 1) fix optargs: -b and -p should not be used at the same time.
>     2) pre_mem maybe not initialized before using if -p option not used.
>        fix it by moving it outside ``if(maxpercent)'' block.
> v4: fix code format.
> 
> Signed-off-by: Caspar Zhang <czhang@redhat.com>

Hi Garrett, would you please review this patch? Thanks!

Caspar

-- 
Quality Assurance Associate (Kernel) in
Red Hat Software (Beijing) Co., R&D Branch

TEL: +86-10-62608150
WEB: http://www.redhat.com/
--
C-A-S-P-A-R, Caspar /kæspɑ:/ ;-)

------------------------------------------------------------------------------
Protect Your Site and Customers from Malware Attacks
Learn about various malware tactics and how to avoid them. Understand 
malware threats, the impact they can have on your business, and how you 
can protect your company and customers by using code signing.
http://p.sf.net/sfu/oracle-sfdevnl
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  parent reply	other threads:[~2011-01-16 11:06 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-06 18:40 [LTP] [PATCH v4] make pid_list dynamically sized with memory czhang
2011-01-06 18:42 ` [LTP] Final version [Re: [PATCH v4] make pid_list dynamically sized with memory] Caspar Zhang
2011-01-11  9:26   ` Caspar Zhang
2011-01-16 11:05 ` Caspar Zhang [this message]
2011-01-18  8:29 ` [LTP] [PATCH v4] make pid_list dynamically sized with memory Caspar Zhang
2011-01-18  8:33   ` Garrett Cooper
2011-01-18  8:45     ` Caspar Zhang
2011-01-18  9:14     ` [LTP] [PATCH v5] " Caspar Zhang

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=4D32D102.4040506@redhat.com \
    --to=czhang@redhat.com \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=yanegomi@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox