From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH stable-only] virtio-net: fix build on m68k and sparc64 Date: Wed, 15 Jan 2014 10:46:05 +0200 Message-ID: <20140115084605.GF29177@redhat.com> References: <1389774152-1526-1-git-send-email-mst@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: 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: Geert Uytterhoeven Cc: "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , virtualization@lists.linux-foundation.org, Greg Kroah-Hartman , David Miller , Guenter Roeck List-Id: virtualization@lists.linuxfoundation.org On Wed, Jan 15, 2014 at 09:36:13AM +0100, Geert Uytterhoeven wrote: > On Wed, Jan 15, 2014 at 9:26 AM, Michael S. Tsirkin wrote: > > As a result of backporting a bugfix, virtio_net started passing void * > > to page_address, assuming that it will get silently converted to struct > > page *. But this does not happen on architectures where page_address is > > a macro, the result is build failure as the macro tries to dereference > > void*. > > > > Fix by reordering code slightly, so we always pass > > struct page * to page_address. > > Thanks for the clue! I was just investigating a similar failure in -next. > > Gr{oetje,eeting}s, > > Geert You don't mean linux-next? And not in virtio-net? I don't see page_address being used on void * anywhere there. > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds