From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [RFC 7/11] virtio_pci: new, capability-aware driver. Date: Sun, 11 Dec 2011 11:42:56 +0200 Message-ID: <20111211094256.GB11504@redhat.com> References: <87pqfzgy6p.fsf@rustcorp.com.au> <87zkf3fiu2.fsf@rustcorp.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <87zkf3fiu2.fsf@rustcorp.com.au> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Rusty Russell Cc: Sasha Levin , virtualization List-Id: virtualization@lists.linuxfoundation.org On Thu, Dec 08, 2011 at 09:09:33PM +1030, Rusty Russell wrote: > +/* There is no iowrite64. We use two 32-bit ops. */ > +static void iowrite64(u64 val, const __le64 *addr) > +{ > + iowrite32((u32)val, (__le32 *)addr); > + iowrite32(val >> 32, (__le32 *)addr + 1); > +} > + Let's put addr_lo/addr_hi in the structure then, to make the fact this field is not atomic explicit?