Bug 1664364

Summary: qemu -device scsi-hd,help coredumps with upstream qemu ea9ce8934
Product: Red Hat Enterprise Linux Advanced Virtualization Reporter: Lukáš Doktor <ldoktor>
Component: qemu-kvmAssignee: Marc-Andre Lureau <marcandre.lureau>
Status: CLOSED NOTABUG QA Contact: Xueqiang Wei <xuwei>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: ---CC: chayang, coli, juzhang, marcandre.lureau, michen, rbalakri, ribarry, virt-maint
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-02-04 13:30:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Lukáš Doktor 2019-01-08 15:17:22 UTC
Description of problem:
Running with upstream qemu I noticed errors when running avocado. It was caused by running:

    qemu-system-x86_64  -device scsi-hd,help

which crashed. First bad commit is:

commit ea9ce8934c5d2cc8925359a6d8d45eb829a9f27f (HEAD, refs/bisect/bad)
Author: Marc-André Lureau <marcandre.lureau>
Date:   Mon Nov 26 22:04:32 2018 +0400

    hw: apply accel compat properties without touching globals
    
    Instead of registering compat properties as globals, let's keep them
    in their own array, to avoid mixing with user globals.
    
    Introduce object_apply_global_props() function, to apply compatibility
    properties from a GPtrArray.
    
    Signed-off-by: Marc-André Lureau <marcandre.lureau>
    Reviewed-by: Igor Mammedov <imammedo>
    Reviewed-by: Cornelia Huck <cohuck>
    Acked-by: Eduardo Habkost <ehabkost>


Version-Release number of selected component (if applicable):
upstream qemu ea9ce8934c5d2cc8925359a6d8d45eb829a9f27f

How reproducible:
always (tested on s390 and x86_64)

Steps to Reproduce:
1. clone qemu from git
2. git checkout ea9ce8934c5d2cc8925359a6d8d45eb829a9f27f
3. make
4. ./x86_64-softmmu/qemu-system-x86_64  -device scsi-hd,hel

Actual results:
Segmentation fault (core dumped [obraz paměti uložen])

Expected results:
scsi-hd options:
  min_io_size=<uint16>
  secs=<uint32>
  removable=<bool>       - on/off
  channel=<uint32>
...

Comment 3 Marc-Andre Lureau 2019-01-09 10:24:20 UTC
Sent to Qemu ML:
[PATCH] qdev: fix -device scsi-hd,help regression

Comment 4 Ademar Reis 2019-01-09 20:10:51 UTC
(In reply to Marc-Andre Lureau from comment #3)
> Sent to Qemu ML:
> [PATCH] qdev: fix -device scsi-hd,help regression

This is still a downstream BZ, so we can't move bugs to POST until it's either merged upstream (so we add "Fixed in version" here) or backported.