WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [Qemu-devel] [PATCH] xen: net backend doesn't need linux

To: Blue Swirl <blauwirbel@xxxxxxxxx>
Subject: [Xen-devel] Re: [Qemu-devel] [PATCH] xen: net backend doesn't need linux headers.
From: Gerd Hoffmann <kraxel@xxxxxxxxxx>
Date: Mon, 08 Jun 2009 09:14:56 +0200
Cc: Andreas Färber <andreas.faerber@xxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, xen-discuss@xxxxxxxxxxxxxxx, qemu-devel@xxxxxxxxxx
Delivery-date: Mon, 08 Jun 2009 00:15:34 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <f43fc5580906062356q11313ff5rbd2edb3b8966d245@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1243926324-13133-1-git-send-email-kraxel@xxxxxxxxxx> <f43fc5580906060852i671ee4a0ie575ce061271d9a2@xxxxxxxxxxxxxx> <28EBEC28-6D03-4321-8629-1F54EA30C634@xxxxxx> <1DB8C28C-ECE6-459C-B9BE-D4747CA4C9CF@xxxxxx> <f43fc5580906062356q11313ff5rbd2edb3b8966d245@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1b3pre) Gecko/20090513 Fedora/3.0-2.3.beta2.fc11 Lightning/1.0pre Thunderbird/3.0b2
On 06/07/09 08:56, Blue Swirl wrote:
  -                    tmpbuf = malloc(PAGE_SIZE);
  +                    tmpbuf = malloc(XC_PAGE_SIZE);

It also looks like there is no real need for using host or target page
size, so we could use for example MIN(4096,  txreq.size).

No. XC_PAGE_SIZE is the max buffer size. As the buffer allocated here could be resused for multiple packets you'll have to use that, otherwise you'll overflow the buffer in case the second packet is larger than the first one.

  If then goes on to compile Xen stuff into sparc-softmmu and fails:

Hmm, it shouldn't try that in the first place. Guess this is the result of this commit:

http://git.savannah.gnu.org/cgit/qemu.git/commit/?id=2567f5796c38238d6f2055f074e347a0338140ce

  xc_gnttab_open
  collect2: ld returned 1 exit status
  Same errors when linking i386-softmmu.

For i386-softmmu it should work.

configure should probe for xc_gnttab symbols and disable Xen if these
are not found.

Strange. configure checks for xc_interface_open. Both xc_interface_open and xc_gnttab_open are in the same shared library. So it should work in theory, dunno what the problem here is.

Hmm, the linker complains about xc_gnttab_* only, not about the other xc_* functions missing. Maybe solaris simply has no support for them?

cheers,
  Gerd

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