Date: Wed, 22 Oct 2014 14:59:50 -0400 (EDT)
Message-Id: <20141022.145950.1585224138048242923.davem@davemloft.net>
To: oleg@redhat.com
Cc: peterz@infradead.org, cl@linux.com, tkhai@yandex.ru, ktkhai@parallels.com,
        linux-kernel@vger.kernel.org, mingo@redhat.com, vdavydov@parallels.com
Subject: Re: introduce probe_slab_address?
From: David Miller <davem@davemloft.net>
In-Reply-To: <20141022181412.GA8159@redhat.com>
References: <20141022161450.GA27607@redhat.com>
	<20141022163743.GI21513@worktop.programming.kicks-ass.net>
	<20141022181412.GA8159@redhat.com>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org

From: Oleg Nesterov <oleg@redhat.com>
Date: Wed, 22 Oct 2014 20:14:12 +0200

> So perhaps something like this makes sense?
> 
> If some arch has problems with D-cache aliasing (because the freed page
> can be already mapped by user-space or vmalloc'ed), it can redefine this
> helper.
> 
> Do you think we can use it to access rq->curr? (although let me repeat
> that I won't really argue with SLAB_DESTROY_BY_RCU).

Do we really need this?

We fully initialize and read from the area using the same virtual
address, there is no possiblity for corruption from SLAB's
perspective.

It's when you store at vaddr X then read at vaddr Y and expect to see
what you wrote at X that you have problems.

That is very much not what is happening here.

The lifetime is contained to SLAB's usage at one single virtual
address.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/