On 19.09.2014 05:01, Jim Fehlig wrote:
Stefan Bader wrote:
Re-pushing this as the old thread got rather stale.
Thanks.
Some of the
VFB setup went in a bug fix. Not sure I missed a detail in rebasing
bug the keyboard setting may be the only thing missing...
Yes, agreed.
-Stefan
Re-pushing this as the old thread got rather stale. Some of the
VFB setup went in a bug fix. Not sure I missed a detail in rebasing
bug the keyboard setting may be the only thing missing...
-Stefan
[v2: Check return code of VIR_STRDUP and fix indentation]
[v3: Split out VRAM fixup and return
Stefan Bader wrote:
Re-pushing this as the old thread got rather stale.
Thanks.
Some of the
VFB setup went in a bug fix. Not sure I missed a detail in rebasing
bug the keyboard setting may be the only thing missing...
Yes, agreed.
-Stefan
[v2: Check return code of VIR_STRDUP and
On 16.07.2014 23:05, Jim Fehlig wrote:
Stefan Bader wrote:
being as bad with timely responses. Ok, so how about the following?
One note: it could be the STRDUP's are not strictly needed. But
to me it felt wrong to have two places refer to the same strings
(as MakeVFB copies the struct
Stefan Bader wrote:
On 16.07.2014 23:05, Jim Fehlig wrote:
While testing this, I noticed that libvirt will set vram to 9216 if not
specified. E.g.
# cat test.xml
...
video
model type='vga'/
/video
...
# virsh define test.xml
# virsh dumpxml test
...
video
Stefan Bader wrote:
being as bad with timely responses. Ok, so how about the following?
One note: it could be the STRDUP's are not strictly needed. But
to me it felt wrong to have two places refer to the same strings
(as MakeVFB copies the struct containing the pointers).
Agreed. Without
On 07/16/2014 03:05 PM, Jim Fehlig wrote:
+b_info-video_memkb = def-videos[0]-vram ?
+ def-videos[0]-vram :
+ LIBXL_MEMKB_DEFAULT;
While testing this, I noticed that libvirt will set vram to 9216 if not
specified. E.g.
being as bad with timely responses. Ok, so how about the following?
One note: it could be the STRDUP's are not strictly needed. But
to me it felt wrong to have two places refer to the same strings
(as MakeVFB copies the struct containing the pointers). If this
is not needed, then all changes now