public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Jan Stancek <jstancek@redhat.com>
To: Maxim Uvarov <maxim.uvarov@linaro.org>
Cc: ltp-list@lists.sourceforge.net,
	Maxim Uvarov <maxim.uvarov@linaro.com>,
	linaro-networking@linaro.org
Subject: Re: [LTP] [PATCH] LTP max_map_count: filer out not accounted vector	stub for arm
Date: Wed, 9 Oct 2013 03:59:55 -0400 (EDT)	[thread overview]
Message-ID: <31658697.3684822.1381305595230.JavaMail.root@redhat.com> (raw)
In-Reply-To: <1380881832-7418-1-git-send-email-maxim.uvarov@linaro.com>



----- Original Message -----
> From: "Maxim Uvarov" <maxim.uvarov@linaro.org>
> To: ltp-list@lists.sourceforge.net
> Cc: linaro-networking@linaro.org, "Maxim Uvarov" <maxim.uvarov@linaro.com>
> Sent: Friday, 4 October, 2013 12:17:11 PM
> Subject: [LTP] [PATCH] LTP max_map_count: filer out not accounted vector	stub for arm
> 
> Arm has following vector stab:
> ffff0000-ffff1000 r-xp 00000000 00:00 0 [vectors]
> which is not accounted in map_count.
> 
> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.com>
> ---
>  testcases/kernel/mem/tunable/max_map_count.c |   19 +++++++++++++++----
>  1 file changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/testcases/kernel/mem/tunable/max_map_count.c
> b/testcases/kernel/mem/tunable/max_map_count.c
> index 01fbe5e..ca28fba 100644
> --- a/testcases/kernel/mem/tunable/max_map_count.c
> +++ b/testcases/kernel/mem/tunable/max_map_count.c
> @@ -121,12 +121,24 @@ void cleanup(void)
>   * for in the vm_area_struct's map_count.
>   */
>  #if defined(__x86_64__) || defined(__x86__)
> -static int filter_map(char *buf)
> +static int filter_map(char *line)
>  {
> +	char buf[BUFSIZ];
> +	int ret;
> +
> +	ret = sscanf(line, "%*p-%*p %*4s %*p %*2d:%*2d %*d %s", buf);
> +	if (ret != 1)
> +		return 0;
> +
>  	return strcmp(buf, "[vsyscall]") == 0;
>  }
> +#elif defined(__arm__)
> +static int filter_map(char *line)
> +{
> +	return strncmp(line, "ffff0000-ffff1000", 17) == 0;

Hi,

just curious, why comparing the range instead of name ([vectors])?

Regards,
Jan

> +}
>  #else
> -static int filter_map(char *buf)
> +static int filter_map(char *line)
>  {
>  	return 0;
>  }
> @@ -146,8 +158,7 @@ static long count_maps(pid_t pid)
>  		tst_brkm(TBROK | TERRNO, cleanup, "fopen %s", buf);
>  	while (getline(&line, &len, fp) != -1) {
>  		/* exclude vdso and vsyscall */
> -		if (sscanf(line, "%*p-%*p %*4s %*p %*2d:%*2d %*d %s", buf) ==
> -				1 && filter_map(buf))
> +		if (filter_map(line))
>  			continue;
>  		map_count++;
>  	}
> --
> 1.7.9.5
> 
> 
> ------------------------------------------------------------------------------
> October Webinars: Code for Performance
> Free Intel webinars can help you accelerate application performance.
> Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from
> the latest Intel processors and coprocessors. See abstracts and register >
> http://pubads.g.doubleclick.net/gampad/clk?id=60134791&iu=/4140/ostg.clktrk
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
> 

------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from 
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60134071&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  parent reply	other threads:[~2013-10-09  8:00 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-04 10:17 [LTP] [PATCH] LTP max_map_count: filer out not accounted vector stub for arm Maxim Uvarov
2013-10-04 10:17 ` [LTP] [PATCH] LTP cacheflush01 add arm variant Maxim Uvarov
2013-10-09  7:55   ` Jan Stancek
2013-10-09  7:59 ` Jan Stancek [this message]
2013-10-09 10:33   ` [LTP] [PATCH] LTP max_map_count: filer out not accounted vector stub for arm Maxim Uvarov
2013-10-18 12:22     ` Maxim Uvarov
2013-10-21  7:50       ` Jan Stancek

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=31658697.3684822.1381305595230.JavaMail.root@redhat.com \
    --to=jstancek@redhat.com \
    --cc=linaro-networking@linaro.org \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=maxim.uvarov@linaro.com \
    --cc=maxim.uvarov@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox