All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steven Rostedt <rostedt@goodmis.org>
To: "Maciej W. Rozycki" <macro@linux-mips.org>
Cc: Andreas Schwab <schwab@suse.de>,
	linux-kernel@vger.kernel.org, "Gilbert, John" <JGG@dolby.com>,
	Kyle Moffett <mrmacman_g4@mac.com>, Adrian Bunk <bunk@stusta.de>,
	Arjan van de Ven <arjan@infradead.org>,
	linux-os@analogic.com
Subject: Re: Illegal use of reserved word in system.h
Date: Thu, 19 May 2005 10:59:40 -0400	[thread overview]
Message-ID: <1116514780.27471.8.camel@localhost.localdomain> (raw)
In-Reply-To: <Pine.LNX.4.61L.0505191532120.10681@blysk.ds.pg.gda.pl>

On Thu, 2005-05-19 at 15:43 +0100, Maciej W. Rozycki wrote:
> On Thu, 19 May 2005, Steven Rostedt wrote:
> 
> > > See create_elf_tables.  The aux table comes after the environment.
> > 
> > As I stated earlier, the page size passed in there is ELF_EXEC_PAGESIZE
> > which may not be the same as PAGE_SIZE.
> 
>  Well, AT_PAGESZ is specified as "system page size".  If we pass something 
> else, then it's asking for troubles.  What comes from AT_PAGESZ is used by 
> userland for stuff like masking arguments for mmap() and mprotect() so 
> it'd better be the right value.

Well, they probably are the same, but then what's the reason for the
lines in binfmt_elf.c:

#if ELF_EXEC_PAGESIZE > PAGE_SIZE
# define ELF_MIN_ALIGN	ELF_EXEC_PAGESIZE
#else
# define ELF_MIN_ALIGN	PAGE_SIZE
#endif


This looks to me that ELF_EXEC_PAGESIZE and PAGE_SIZE may not be the
same. And what's passed to AT_PAGESZ is ELF_EXEC_PAGESIZE.  In mips (as
your email address shows you are interested in) ELF_EXEC_PAGESIZE is
simply defined as PAGE_SIZE.  But in intel i386, it is defined as 4096,
which coincidentally is the same as PAGE_SIZE but there's no guarantee
that this will be the same, unless who ever changes PAGE_SIZE also
remembers to change ELF_EXEC_PAGESIZE.

In arm26 the PAGE_SIZE is configurable (16k or 32k) but the
ELF_EXEC_PAGESIZE stays as 32k.  So is this a bug?

-- Steve
 


  reply	other threads:[~2005-05-19 15:02 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-18 19:07 Illegal use of reserved word in system.h Gilbert, John
2005-05-18 19:53 ` Adrian Bunk
2005-05-19  2:22   ` Kyle Moffett
2005-05-19 11:28     ` Adrian Bunk
2005-05-19 12:19       ` Richard B. Johnson
2005-05-19 12:27         ` Arjan van de Ven
2005-05-19 12:47           ` Maciej W. Rozycki
2005-05-19 13:01             ` Richard B. Johnson
2005-05-19 13:47               ` Steven Rostedt
2005-05-19 14:06               ` Andreas Schwab
2005-05-19 14:15                 ` Steven Rostedt
2005-05-19 14:36                   ` Richard B. Johnson
2005-05-19 15:19                     ` Andreas Schwab
2005-05-19 15:56                       ` Steven Rostedt
2005-05-19 16:03                         ` Richard B. Johnson
2005-05-19 15:58                       ` Richard B. Johnson
2005-05-19 16:22                         ` Steven Rostedt
2005-05-19 17:06                           ` Richard B. Johnson
2005-05-19 17:36                             ` Steven Rostedt
2005-05-19 14:43                   ` Maciej W. Rozycki
2005-05-19 14:59                     ` Steven Rostedt [this message]
2005-05-20 12:23                       ` Maciej W. Rozycki
2005-05-19 14:53                   ` Arjan van de Ven
2005-05-19 14:58                     ` Maciej W. Rozycki
2005-05-19 15:06                       ` Arjan van de Ven
2005-05-19 13:56           ` Steven Rostedt
2005-05-19 17:30           ` Ben Greear
2005-05-19 17:36             ` Chris Friesen
2005-05-19 17:40               ` Ben Greear
2005-05-19 22:14                 ` Andreas Schwab
2005-05-19 22:30                   ` J.A. Magallon
2005-05-19 22:41                     ` Russell King
2005-05-19 23:10                       ` J.A. Magallon
2005-05-20  0:01                         ` Jeff Woods
2005-05-20  9:27             ` Helge Hafting
2005-05-20 16:47               ` Lee Revell
2005-05-20 16:56                 ` Ben Greear
2005-05-18 23:42 ` Andi Kleen
2005-05-18 23:59   ` Adrian Bunk
  -- strict thread matches above, loose matches on Subject: below --
2005-05-20 17:30 Gilbert, John
2005-05-19 20:18 Gilbert, John
2005-05-19  1:51 Gilbert, John
2005-05-18 21:31 Gilbert, John
2005-05-19 17:37 ` Alan Cox
2005-05-20 11:49   ` Dave Airlie
2005-05-20 16:42     ` Alan Cox
2005-05-18 18:23 Gilbert, John
2005-05-18 19:02 ` Adrian Bunk
2005-05-18 19:32 ` Richard B. Johnson
2005-05-19  1:06 ` rdunlap
2005-05-18  0:51 Gilbert, John
2005-05-18  0:57 ` Lee Revell
2005-05-18  1:19 ` Chris Wedgwood
2005-05-18  2:51 ` Steven Rostedt
2005-05-18  4:12   ` Willy Tarreau
2005-05-18 10:06   ` Patrick McFarland
2005-05-18 10:49     ` Coywolf Qi Hunt
2005-05-18 11:12     ` Richard B. Johnson
2005-05-18 21:52       ` Matan Peled
2005-05-19 11:03         ` Richard B. Johnson

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=1116514780.27471.8.camel@localhost.localdomain \
    --to=rostedt@goodmis.org \
    --cc=JGG@dolby.com \
    --cc=arjan@infradead.org \
    --cc=bunk@stusta.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-os@analogic.com \
    --cc=macro@linux-mips.org \
    --cc=mrmacman_g4@mac.com \
    --cc=schwab@suse.de \
    /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.