From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([140.186.70.92]:46998)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <B07421@freescale.com>) id 1Qbb1L-0002g4-S4
	for qemu-devel@nongnu.org; Tue, 28 Jun 2011 12:20:17 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <B07421@freescale.com>) id 1Qbb1K-0006WR-KB
	for qemu-devel@nongnu.org; Tue, 28 Jun 2011 12:20:15 -0400
Received: from tx2ehsobe004.messaging.microsoft.com ([65.55.88.14]:33067
	helo=TX2EHSOBE007.bigfish.com) by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <B07421@freescale.com>) id 1Qbb1K-0006WM-7S
	for qemu-devel@nongnu.org; Tue, 28 Jun 2011 12:20:14 -0400
Date: Tue, 28 Jun 2011 11:20:08 -0500
From: Scott Wood <scottwood@freescale.com>
Message-ID: <20110628112008.31cf6237@schlenkerla.am.freescale.net>
In-Reply-To: <4E098E23.3040605@adacore.com>
References: <1309180555-3942-1-git-send-email-chouteau@adacore.com>
	<20110627112807.22346e82@schlenkerla.am.freescale.net>
	<4E098E23.3040605@adacore.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH] Add e500 instructions dcblc,
 dcbtls and dcbtstl as no-op
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Fabien Chouteau <chouteau@adacore.com>
Cc: qemu-devel@nongnu.org

On Tue, 28 Jun 2011 10:17:39 +0200
Fabien Chouteau <chouteau@adacore.com> wrote:

> On 27/06/2011 18:28, Scott Wood wrote:
> > On Mon, 27 Jun 2011 15:15:55 +0200
> > Fabien Chouteau <chouteau@adacore.com> wrote:
> >
> >> +/* dcbtls */
> >> +static void gen_dcbtls(DisasContext *ctx)
> >> +{
> >> +    /* interpreted as no-op */
> >> +}
> >> +
> >> +/* dcbtstls */
> >> +static void gen_dcbtstls(DisasContext *ctx)
> >> +{
> >> +    /* interpreted as no-op */
> >> +}
> >
> > Set L1CSR0[CUL] (unable to lock)?
> 
> Why do you want to set this bit? Can't we consider that the instruction is
> always effective?

But it's not.  Why claim it is, in the absence of some specific workload
that needs to be fooled (which could take many different forms, not all of
which are appropriate defaults)?

-Scott