public inbox for linux-arch@vger.kernel.org
 help / color / mirror / Atom feed
From: Arun Sharma <arun.sharma@intel.com>
To: "David S. Miller" <davem@redhat.com>
Cc: davidm@hpl.hp.com, linux-arch@vger.kernel.org
Subject: Re: SHMLBA and compat tasks
Date: Mon, 1 Mar 2004 11:33:08 -0800	[thread overview]
Message-ID: <20040301193308.GA13305@intel.com> (raw)
In-Reply-To: <20040229215752.3a6f0ce7.davem@redhat.com>

On Sun, Feb 29, 2004 at 09:57:52PM -0800, David S. Miller wrote:
> 
> Why don't we declare that SHM_LBA must be abided by on all platforms?

I assume you're proposing changing the man page to say that SHMLBA will
be forced, in all cases (even though SHM_RND is not specified). Yes,
that would make the implementation consistent with the man page, so it'd
be an improvement over the current situation.

But ia32 on ia64 doesn't match this declaration today.  

shmat(id, 0, ...) 

can return an address that's PAGE_SIZE aligned, but not SHMLBA
aligned. 

David, are you ok with changing this code to force SHMLBA alignment for 
both 32 and 64 bit tasks ?

	-Arun

arch/ia64/kernel/sys_ia64.c:

arch_get_unmapped_area(...)
{
...
       if (map_shared && (TASK_SIZE > 0xfffffffful))
                /*
                 * For 64-bit tasks, align shared segments to 1MB to avoid potential
                 * performance penalty due to virtual aliasing (see ASDM).  For 32-bit
                 * tasks, we prefer to avoid exhausting the address space too quickly by
                 * limiting alignment to a single page.
                 */
                align_mask = SHMLBA - 1;

  reply	other threads:[~2004-03-01 19:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-02-28  1:41 SHMLBA and compat tasks Arun Sharma
2004-02-28 23:55 ` David S. Miller
2004-02-29  2:11   ` Arun Sharma
2004-03-01  5:57     ` David S. Miller
2004-03-01 19:33       ` Arun Sharma [this message]
2004-03-01 19:41         ` David Mosberger
2004-03-01 20:11           ` David S. Miller
2004-03-01 20:17             ` David Mosberger
2004-03-01 20:16           ` Arun Sharma
2004-03-01 20:55             ` Matthew Wilcox
2004-03-02 19:43               ` Arun Sharma
2004-03-06  2:37                 ` Arun Sharma
2004-03-06  7:39                   ` David S. Miller

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=20040301193308.GA13305@intel.com \
    --to=arun.sharma@intel.com \
    --cc=davem@redhat.com \
    --cc=davidm@hpl.hp.com \
    --cc=linux-arch@vger.kernel.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