Bug 218486
| Summary: | Need to update libvirt PV-FB support to work with latest upstream SEXPR format | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 5 | Reporter: | Daniel Berrangé <berrange> | ||||||||
| Component: | libvirt | Assignee: | Daniel Veillard <veillard> | ||||||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |||||||||
| Severity: | medium | Docs Contact: | |||||||||
| Priority: | high | ||||||||||
| Version: | 5.0 | CC: | konradr, xen-maint | ||||||||
| Target Milestone: | --- | ||||||||||
| Target Release: | --- | ||||||||||
| Hardware: | All | ||||||||||
| OS: | Linux | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | RC | Doc Type: | Bug Fix | ||||||||
| Doc Text: | Story Points: | --- | |||||||||
| Clone Of: | Environment: | ||||||||||
| Last Closed: | 2007-02-08 01:23:12 UTC | Type: | --- | ||||||||
| Regression: | --- | Mount Type: | --- | ||||||||
| Documentation: | --- | CRM: | |||||||||
| Verified Versions: | Category: | --- | |||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||
| Embargoed: | |||||||||||
| Bug Depends On: | |||||||||||
| Bug Blocks: | 218048, 218050 | ||||||||||
| Attachments: |
|
||||||||||
Blocker justification: bug 218050 (already a GA blocker) will break libvirt & related tools, without this ticket also being fixed. Status of having a patch? Created attachment 142879 [details]
Patch to look for new SEXPR format and generate <graphics> tag as needed
QE ack for RHEL5. 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
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.
457216 build (dist-5E, RHEL-5:libvirt-0_1_8-9_el5) A build with the 2 patches is available, Daniel 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. |
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.