public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrea Arcangeli <andrea@suse.de>
To: Andrew Morton <akpm@osdl.org>
Cc: pavel@suse.cz, linux-kernel@vger.kernel.org, hugh@veritas.com
Subject: Re: Properly stop kernel threads on aic7xxx
Date: Fri, 2 Apr 2004 03:09:00 +0200	[thread overview]
Message-ID: <20040402010900.GJ18585@dualathlon.random> (raw)
In-Reply-To: <20040401170403.76d86432.akpm@osdl.org>

On Thu, Apr 01, 2004 at 05:04:03PM -0800, Andrew Morton wrote:
> Andrea Arcangeli <andrea@suse.de> wrote:
> >
> > I'm also unsure why _all_ multipage allocations really need this
> > compound thing setup and why can't the owner of the page take care of
> > the refcounting itself by always using the head page. I may actually
> > add a GFP bitflag asking for a multipage but w/o a compound setup. There
> > are million ways to fix this, none of which is obvious.
> 
> For direct-io into higher-order pages.  When doing direct-io into a hugetlb
> page we need to make sure that get_user_pages() pins the correct pageframe.

we do the same in bigpages 2.4 just fine with a basic math on the page_t
knowing it's 2M naturally aligned, very easy to find the head page on
vmas with VM_HUGETLBFS.

> Possibly we need to turn it on regardless of CONFIG_HUGETLBPAGE for people
> who want to do direct-io or PEEKTEXT/POKETEXT into mmapped soundcard
> buffers, for example.  Perhaps there's a race which could permit direct-io
> to write to a freed page via this route..
> 
> Davem had some reason why he might want to turn on the compound logic
> permanently - related to TCP access to/from higher-order pages, I think.

currently it's turned on only with HUGTLBFS=y though.

      reply	other threads:[~2004-04-02  1:09 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-01 17:08 Properly stop kernel threads on aic7xxx Pavel Machek
2004-04-02  0:35 ` Andrea Arcangeli
2004-04-02  1:04   ` Andrew Morton
2004-04-02  1:09     ` Andrea Arcangeli [this message]

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=20040402010900.GJ18585@dualathlon.random \
    --to=andrea@suse.de \
    --cc=akpm@osdl.org \
    --cc=hugh@veritas.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@suse.cz \
    /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