public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrea Arcangeli <andrea@suse.de>
To: "David S. Miller" <davem@redhat.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: flush_tlb_all in vmalloc_area_pages
Date: Fri, 21 Sep 2001 00:25:47 +0200	[thread overview]
Message-ID: <20010921002547.G729@athlon.random> (raw)
In-Reply-To: <20010907165612.T11329@athlon.random> <20010920.142638.68040129.davem@redhat.com>
In-Reply-To: <20010920.142638.68040129.davem@redhat.com>; from davem@redhat.com on Thu, Sep 20, 2001 at 02:26:38PM -0700

On Thu, Sep 20, 2001 at 02:26:38PM -0700, David S. Miller wrote:
>    From: Andrea Arcangeli <andrea@suse.de>
>    Date: Fri, 7 Sep 2001 16:56:12 +0200
>    
>    For the flush_cache_all for the virtually indexed caches should be the
>    same issue in theory (at least the kmap logic only needs to flush the
>    caches before the unmapping [not before the mapping] too)
>    
>    Am I missing something, Dave?
> 
> Anything that creates or takes away vmalloc() mappings needs to flush
> the data cache if it is virtuall indexed.

The only question I'd like to get a answer is "what is actually the
data that can be virtually indexed) in the vmalloc range at the time we
run vmalloc?" Where does it cames from?

If there is no such data (as I think), we obviously don't need to flush
the virtually indexed caches at vmalloc time (but just at vfree).

Furthmore I recall on sparc you cannot flush the cache if you don't have
a mapping in place, and when you run vmalloc there should be no mapping
in place for the region of cache that you're trying to flush (or we
wouldn't trap the invalid faults there).

If anybody is using at boot time the vmalloc range for whatever purpuse
it should be its own business to flush the cache before dropping the
mappings from there.

Andrea

  reply	other threads:[~2001-09-20 22:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-07 14:56 flush_tlb_all in vmalloc_area_pages Andrea Arcangeli
2001-09-20 21:26 ` David S. Miller
2001-09-20 22:25   ` Andrea Arcangeli [this message]
2001-09-20 22:29     ` David S. Miller
2001-09-20 22:38       ` Andrea Arcangeli
2001-09-20 22:52         ` Andrea Arcangeli

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=20010921002547.G729@athlon.random \
    --to=andrea@suse.de \
    --cc=davem@redhat.com \
    --cc=linux-kernel@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