From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: David Miller <davem@davemloft.net>,
clameter@sgi.com, hugh@veritas.com, James.Bottomley@steeleye.com,
rmk+lkml@arm.linux.org.uk, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Containment measures for slab objects on scatter gather lists
Date: Fri, 29 Jun 2007 23:39:22 +0100 [thread overview]
Message-ID: <20070629233922.689865ca@the-village.bc.nu> (raw)
In-Reply-To: <20070629134529.3e12a131.akpm@linux-foundation.org>
On Fri, 29 Jun 2007 13:45:29 -0700
Andrew Morton <akpm@linux-foundation.org> wrote:
> On Fri, 29 Jun 2007 13:16:57 +0100
> Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
>
> > > If those operations involve modifying that slab page's pageframe then what
> > > stops concurrent dma'ers from stomping on each other's changes? As in:
> > > why aren't we already buggy?
> >
> > Or DMA operations falling out with CPU operations in the same memory
> > area. Not all platforms have hardware consistency and some will blat the
> > entire page out of cache.
>
> Is that just a performance problem, or can data be lost here? It depends
> on the meaning of "blat": writeback? invalidate? More details, please.
Invalidate. Sorry didn't realise it they hadn't discovered that word down
under.
If you've got something packing objects in tight we are going
to have fun with cache handling simply because the CPU cache granularity
may mean that the invalidate also invalidates a few bytes on (ie a 12
byte object will invalidate 16 bytes of memory) and you've just removed
any CPU held changes in the start of the next object.
Alan
next prev parent reply other threads:[~2007-06-29 22:33 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-29 4:01 Christoph Lameter
2007-06-29 4:10 ` David Miller
2007-06-29 4:22 ` Christoph Lameter
2007-06-29 4:28 ` David Miller
2007-06-29 4:39 ` Christoph Lameter
2007-06-29 5:06 ` David Miller
2007-06-29 5:24 ` Andrew Morton
2007-06-29 5:37 ` David Miller
2007-06-29 5:45 ` Andrew Morton
2007-06-29 6:51 ` Christoph Lameter
2007-06-29 12:16 ` Alan Cox
2007-06-29 20:45 ` Andrew Morton
2007-06-29 21:14 ` Russell King
2007-06-29 23:11 ` Alan Cox
2007-06-30 7:54 ` Russell King
2007-06-29 22:39 ` Alan Cox [this message]
2007-06-29 6:50 ` Christoph Lameter
2007-06-30 8:49 ` Christoph Hellwig
2007-06-29 7:00 ` Christoph Lameter
2007-06-29 9:06 ` David Miller
2007-06-29 13:04 ` Hugh Dickins
2007-06-29 14:15 ` Christoph Lameter
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=20070629233922.689865ca@the-village.bc.nu \
--to=alan@lxorguk.ukuu.org.uk \
--cc=James.Bottomley@steeleye.com \
--cc=akpm@linux-foundation.org \
--cc=clameter@sgi.com \
--cc=davem@davemloft.net \
--cc=hugh@veritas.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rmk+lkml@arm.linux.org.uk \
/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
all inboxes | Powered by JetHome®