From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Christoph Egger" <Christoph.Egger@amd.com>
Subject: Re: [PATCH] two missing bits (was Re: Add
	-Wcast-qual flag)
Date: Fri, 22 Dec 2006 10:44:41 +0100
Message-ID: <200612221044.42035.Christoph.Egger@amd.com>
References: <200612221005.06702.Christoph.Egger@amd.com>
	<458BB3E5.76E4.0078.0@novell.com>
Mime-Version: 1.0
Content-Type: multipart/mixed;
 boundary="Boundary-00=_Kk6iFiK7MxzsJv+"
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <458BB3E5.76E4.0078.0@novell.com>
List-Unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: xen-devel@lists.xensource.com
Cc: Keir Fraser <keir@xensource.com>, Jan Beulich <jbeulich@novell.com>
List-Id: xen-devel@lists.xenproject.org


--Boundary-00=_Kk6iFiK7MxzsJv+
Content-Type: text/plain;
 charset=iso-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On Friday 22 December 2006 10:31, Jan Beulich wrote:
> I object to the change to variable_test_bit() - why is that needed? I had
> specifically submitted a patch to get the const qualifier added there. Jan

The ADDR macro discards the const again.
I attached an alternative patch.


--Boundary-00=_Kk6iFiK7MxzsJv+
Content-Type: text/x-diff;
 charset=iso-8859-1;
 name=xen-cast-xen.diff
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename=xen-cast-xen.diff

diff -r f1ddc050d848 xen/common/gdbstub.c
--- a/xen/common/gdbstub.c	Thu Dec 21 20:44:35 2006 +0000
+++ b/xen/common/gdbstub.c	Fri Dec 22 09:51:42 2006 +0100
@@ -382,7 +382,7 @@ static int
 static int 
 process_command(struct cpu_user_regs *regs, struct gdb_context *ctx)
 {
-    char *ptr;
+    const char *ptr;
     unsigned long addr, length;
     int resume = 0;
 
diff -r f1ddc050d848 xen/include/asm-x86/bitops.h
--- a/xen/include/asm-x86/bitops.h	Thu Dec 21 20:44:35 2006 +0000
+++ b/xen/include/asm-x86/bitops.h	Fri Dec 22 10:39:57 2006 +0100
@@ -253,7 +253,7 @@ static __inline__ int variable_test_bit(
 	__asm__ __volatile__(
 		"btl %2,%1\n\tsbbl %0,%0"
 		:"=r" (oldbit)
-		:"m" (ADDR),"dIr" (nr));
+		:"m" (*(const volatile long *)addr),"dIr" (nr));
 	return oldbit;
 }
 

--Boundary-00=_Kk6iFiK7MxzsJv+
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

--Boundary-00=_Kk6iFiK7MxzsJv+--