Bug 218486 - Need to update libvirt PV-FB support to work with latest upstream SEXPR format
Summary: Need to update libvirt PV-FB support to work with latest upstream SEXPR format
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: libvirt
Version: 5.0
Hardware: All
OS: Linux
high
medium
Target Milestone: ---
: ---
Assignee: Daniel Veillard
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 218048 218050
TreeView+ depends on / blocked
 
Reported: 2006-12-05 18:33 UTC by Daniel Berrangé
Modified: 2007-11-30 22:07 UTC (History)
2 users (show)

Fixed In Version: RC
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-02-08 01:23:12 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Patch to look for new SEXPR format and generate <graphics> tag as needed (2.57 KB, patch)
2006-12-05 19:14 UTC, Daniel Berrangé
no flags Details | Diff
Patch to make new PFVB config parsing work (1.08 KB, patch)
2006-12-08 19:47 UTC, Daniel Berrangé
no flags Details | Diff
Support generation of new style SEXR (7.81 KB, patch)
2006-12-11 20:48 UTC, Daniel Berrangé
no flags Details | Diff

Comment 1 Daniel Berrangé 2006-12-05 18:38:42 UTC
Original way of configuring PVFB was to specify the config file options:

vnc=1
vncunused=1
vnclisten="127.0.0.1"
vncpasswd="123456"

This translated to an SEXPR of

    (image
        (linux
            (vnc 1)
            (vncunused 1)
            (vnclisten 127.0.0.1)
            (vncpasswd 123456)
        )
    )


The new way is to specify

vfb = [ "type=vnc,vncunused=1,vnclisten=127.0.0.1,vncpasswd=123456"]


Which translates to an SEXPR of

    (device (vkbd (backend 0)))
    (device
        (vfb
            (vncunused 1)
            (vnclisten 127.0.0.1)
            (vncpasswd 123456)
            (type vnc)
        )
    )


XenD in RHEL-5 will be able to read the original config file format & convert to
the new style config format on the fly. This does not, however, address the
issue of changed SEXPR format. We need to patch libvirt to understand the new
format, otherwise the XML dump for guest domains will be broken, which impacts
the whole xen userspace toolchain.

The patch should be pretty straightforward.


Comment 2 Daniel Berrangé 2006-12-05 18:41:07 UTC
Blocker justification: bug 218050 (already a GA blocker) will break libvirt &
related tools, without this ticket also being fixed.


Comment 3 Brian Stein 2006-12-05 18:56:01 UTC
Status of having a patch?

Comment 5 Daniel Berrangé 2006-12-05 19:14:50 UTC
Created attachment 142879 [details]
Patch to look for new SEXPR format and generate <graphics> tag as needed

Comment 6 Jay Turner 2006-12-07 03:53:02 UTC
QE ack for RHEL5.

Comment 8 Daniel Berrangé 2006-12-08 19:47:48 UTC
Created attachment 143189 [details]
Patch to make new PFVB config  parsing  work

The previous patch didn't apply cleanly against libvirt version in RHEL-5. This
patch is a backport to 0.1.8

Comment 9 Daniel Berrangé 2006-12-11 20:48:38 UTC
Created attachment 143324 [details]
Support generation of new style SEXR

The previous patch supporting  parsing the new style SEXPR.
This patch provides the 2nd 1/2  of the solution, generatring the new style
SEXPR.

Comment 11 Daniel Veillard 2006-12-13 14:17:17 UTC
457216 build (dist-5E, RHEL-5:libvirt-0_1_8-9_el5)

A build with the 2 patches is available,

Daniel

Comment 12 RHEL Program Management 2007-02-08 01:23:13 UTC
A package has been built which should help the problem described in 
this bug report. This report is therefore being closed with a resolution 
of CURRENTRELEASE. You may reopen this bug report if the solution does 
not work for you.



Note You need to log in before you can comment on or make changes to this bug.