From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Tokarev Subject: Re: [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment Date: Tue, 28 Aug 2012 11:35:57 +0400 Message-ID: <503C74DD.3020307@msgid.tls.msk.ru> References: <825e653c9cfe9d8e26185917cbe1f1dd7ae299e2.1346048917.git.jan.kiszka@web.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: Jan Kiszka , Avi Kivity , Marcelo Tosatti , Alex Williamson , qemu-devel@nongnu.org, kvm@vger.kernel.org, "Michael S. Tsirkin" To: Blue Swirl Return-path: Received: from isrv.corpit.ru ([86.62.121.231]:35316 "EHLO isrv.corpit.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751790Ab2H1HgD (ORCPT ); Tue, 28 Aug 2012 03:36:03 -0400 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: On 27.08.2012 22:56, Blue Swirl wrote: [] >> +static uint32_t slow_bar_readb(void *opaque, target_phys_addr_t addr) >> +{ >> + AssignedDevRegion *d = opaque; >> + uint8_t *in = d->u.r_virtbase + addr; > > Don't perform arithmetic with void pointers. There are a few places in common qemu code which does this for a very long time. So I guess it is safe now. /mjt