Bug 1805334 - vhost-user/50-qemu-gpu.json is not valid JSON
Summary: vhost-user/50-qemu-gpu.json is not valid JSON
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux Advanced Virtualization
Classification: Red Hat
Component: qemu-kvm
Version: 8.2
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: rc
: 8.2
Assignee: Ján Tomko
QA Contact: Guo, Zhiyi
URL:
Whiteboard:
Depends On:
Blocks: 1805324
TreeView+ depends on / blocked
 
Reported: 2020-02-20 16:49 UTC by Ján Tomko
Modified: 2020-05-05 09:58 UTC (History)
5 users (show)

Fixed In Version: qemu-kvm-4.2.0-12.module+el8.2.0+5858+afd073bc
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-05-05 09:57:51 UTC
Type: Bug
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2020:2017 0 None None None 2020-05-05 09:58:51 UTC

Description Ján Tomko 2020-02-20 16:49:04 UTC
Description of problem:
    Trailing comma is not valid json:
    
    $ cat contrib/vhost-user-gpu/50-qemu-gpu.json.in | jq
    parse error: Expected another key-value pair at line 5, column 1

Version-Release number of selected component (if applicable):
qemu-kvm-core-4.2.0-1.module+el8.2.0+4793+b09dd2fb.x86_64

How reproducible:
0144 %


Additional info:
Fixed upstream by:

commit ca26b032e5a0e8a190c763ce828a8740d24b9b65
Author:     Cole Robinson <crobinso>
CommitDate: 2019-12-20 06:56:37 +0100

    vhost-user-gpu: Drop trailing json comma
    
    Trailing comma is not valid json:
    
    $ cat contrib/vhost-user-gpu/50-qemu-gpu.json.in | jq
    parse error: Expected another key-value pair at line 5, column 1
    
    Signed-off-by: Cole Robinson <crobinso>
    Reviewed-by: Marc-André Lureau <marcandre.lureau>
    Reviewed-by: Li Qiang <liq3ea>
    Reviewed-by: Philippe Mathieu-Daudé <philmd>
    Message-id: 7f5dd2ac9f3504e2699f23e69bc3d8051b729832.1568925097.git.crobinso
    Signed-off-by: Gerd Hoffmann <kraxel>

git describe: v4.2.0-199-gca26b032e5

Comment 1 Guo, Zhiyi 2020-02-21 00:03:53 UTC
Hi Gerd,

   I think the 50-qemu-gpu.json is part of using virgl with vhost-user, right? Is this vhost-user-gpu already testable on rhel 8.2? I don't see any RFE bugs point to the support to vhost-user-gpu, what's the plan to support vhost-user-gpu? Thanks!


BR/
Zhiyi

Comment 2 Ján Tomko 2020-02-21 08:03:52 UTC
Zhiyi,

my motivation is virtiofsd. To find out which vhost-user binary offers "type": "fs",
libvirt needs to parse all the vhost-user json files, which it cannot if they aren't valid.

Comment 7 Guo, Zhiyi 2020-02-28 08:17:04 UTC
Verify this issue against qemu-kvm-core-4.2.0-12.module+el8.2.0+5858+afd073bc.x86_64
# cat /usr/share/qemu-kvm/vhost-user/50-qemu-gpu.json | jq
{
  "description": "QEMU vhost-user-gpu",
  "type": "gpu",
  "binary": "/usr/libexec/vhost-user-gpu"
}

Comment 9 errata-xmlrpc 2020-05-05 09:57:51 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:2017


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