xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Guillaume Rousse <Guillaume.Rousse@inria.fr>
To: Xen-devel <xen-devel@lists.xensource.com>
Subject: Re: status of PV drivers in Xen 4.0
Date: Tue, 13 Apr 2010 23:35:40 +0200	[thread overview]
Message-ID: <4BC4E3AC.9050801@inria.fr> (raw)
In-Reply-To: <n2if4527be1004120840g75b6746dn760449e34c949c80@mail.gmail.com>

Le 12/04/2010 17:40, Andrew Lyon a écrit :
> On Mon, Apr 12, 2010 at 12:27 PM, Guillaume Rousse
> <Guillaume.Rousse@inria.fr> wrote:
>> Le 10/04/2010 14:58, Pasi Kärkkäinen a écrit :
>>>> Have you been successful building the drivers for .31 or .32?
>> I just tried 2.6.33 so far, without success.
>>
>>>> I'd like to make a set of patches available to apply to Xen 3.4.2 or
>>>> 4.0 to enable building the drivers with recent kernels and the forward
>>>> ported kernels I usually use, but while I've been able to rebuild them
>>>> on a opensuse 11.2 system running 2.6.31.12 I am finding it hard to
>>>> work out which patches need to be applied and what else the rpm build
>>>> process does to enable compilation.
>>>>
>>>> Any pointers would be much appreciated.
>>>>
>>>
>>> Did you take a look at novell's sles11 sp1 kernel? it's 2.6.32 and
>>> should have pv-on-hvm drivers included?
>> Yes, that's Jan Beulich said on this list. However, I've buid a
>> 2.6.32.11 kernel, with all xen patches available from SLE11-SP1 (as well
>> as a few others needed for applying xen patches properly), and used the
>> corresponding defconfig file. But I can't find any reference to those
>> drivers, at least with the names given in README file of xen tarball
>> (xen-platform-pci.ko, xen-vbd.ko, xen-vnif.ko and xen-balloon.ko).
>> That's probably just because of a name change.
>>
>> Anyway (unless I'm wrong), the actual usefulness of pv-on-hvm is on
>> standard linux kernels, not on dom0 itself, and the issue seems to be on
>> xen Makefile side.
>> --
> 
> I've been looking at this over the weekend and I've made some
> progress, you are correct about the usefulness of the pv-on-hvm
> drivers, however you need to bear in mind that the openSUSE/SLES11
> kernels *always* include the xen patches even if they are not
> activated for a particular kernel flavour (e.g. desktop, debug etc),
> so there are some differences in how the drivers are built,
> specifically the build process uses header files from the xenified
> kernel instead of from Xen itself.
OK, I understand the issue: you can't build those drivers against a
standard kernel, you need to have some xen patches applied, even if the
resulting options are  not activated.

Indeed, I have different build results when trying to build those
drivers against a xenified 2.6.32.11 kernel:
/home/guillomovitch/cooker/xen/xen-4.0.0/unmodified_drivers/linux-2.6/balloon/balloon.c:
In function ‘balloon_init’:
/home/guillomovitch/cooker/xen/xen-4.0.0/unmodified_drivers/linux-2.6/balloon/balloon.c:583:
erreur: implicit declaration of function ‘pagevec_init’
/home/guillomovitch/cooker/xen/xen-4.0.0/unmodified_drivers/linux-2.6/balloon/balloon.c:583:
erreur: ‘free_pagevec’ undeclared (first use in this function)
/home/guillomovitch/cooker/xen/xen-4.0.0/unmodified_drivers/linux-2.6/balloon/balloon.c:583:
erreur: (Each undeclared identifier is reported only once
/home/guillomovitch/cooker/xen/xen-4.0.0/unmodified_drivers/linux-2.6/balloon/balloon.c:583:
erreur: for each function it appears in.)
make[2]: ***
[/home/guillomovitch/cooker/xen/xen-4.0.0/unmodified_drivers/linux-2.6/balloon/balloon.o]
Erreur 1

Including "<linux/pagevec.h>" in ballon.c fixe the first error, but not
the second one.

Anyway, I can't ask the mandriva kernel team to include the whole xen
patches in all kernel just for this. I probably have to isolate the
strict subset of patches needed to make a kernel a suitable target for
those drivers.

-- 
BOFH excuse #223:

The lines are all busy (busied out, that is -- why let them in to begin
with?).

      parent reply	other threads:[~2010-04-13 21:35 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-29 21:02 status of PV drivers in Xen 4.0 Guillaume Rousse
2010-03-30  6:04 ` Pasi Kärkkäinen
2010-03-30  7:49   ` Jan Beulich
2010-03-30  7:52     ` Pasi Kärkkäinen
2010-03-30 11:27     ` Guillaume Rousse
2010-03-30 12:10       ` Jan Beulich
2010-03-30  8:17   ` Guillaume Rousse
2010-03-30 10:04     ` Pasi Kärkkäinen
2010-04-05 16:50   ` Guillaume Rousse
2010-04-10 12:31     ` Andrew Lyon
2010-04-10 12:58       ` Pasi Kärkkäinen
2010-04-10 13:17         ` Andrew Lyon
2010-04-12 11:21           ` Pasi Kärkkäinen
2010-04-12 11:27         ` Guillaume Rousse
2010-04-12 15:40           ` Andrew Lyon
2010-04-12 15:50             ` Pasi Kärkkäinen
2010-04-12 16:05               ` Andrew Lyon
2010-04-12 16:51                 ` Pasi Kärkkäinen
2010-04-20 16:16                 ` Pasi Kärkkäinen
2010-04-20 16:52                   ` Stefano Stabellini
2010-04-20 17:05                     ` Pasi Kärkkäinen
2010-04-20 17:07                       ` Stefano Stabellini
2010-04-13 21:35             ` Guillaume Rousse [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4BC4E3AC.9050801@inria.fr \
    --to=guillaume.rousse@inria.fr \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).