qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: Aleksandr Bezzubikov <zuban32s@gmail.com>
Cc: Marcel Apfelbaum <marcel@redhat.com>,
	qemu-s390x@nongnu.org, Thomas Huth <thuth@redhat.com>,
	QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] Crash with pcie-pci-bridge on qemu-system-s390x
Date: Tue, 7 Nov 2017 11:34:46 +0100	[thread overview]
Message-ID: <20171107113446.6ac51e16.cohuck@redhat.com> (raw)
In-Reply-To: <CAKSfGUAvP+EB1J_4=-xAAdGuopHnHmjXC82BGx9tjjK=2_x_sg@mail.gmail.com>

On Thu, 2 Nov 2017 17:59:58 +0300
Aleksandr Bezzubikov <zuban32s@gmail.com> wrote:

> 2017-11-02 17:42 GMT+03:00 Marcel Apfelbaum <marcel@redhat.com>:
> > On 02/11/2017 16:19, Thomas Huth wrote:  
> >>
> >>   Hi,
> >>  
> >
> > Hi Thomas,
> >  
> 
> Hi Thomas, Marcel,
> 
> >> seems like there's a new way to crash QEMU with the pcie-pci-bridge
> >> device (using QEMU master branch of today):  
> >>  > $ s390x-softmmu/qemu-system-s390x -nographic -S  
> >> QEMU 2.10.50 monitor - type 'help' for more information
> >> (qemu) device_add pcie-pci-bridge,id=x
> >> Segmentation fault (core dumped)
> >>  
> >
> > Thanks for noticing it.
> >  
> >> Does anybody have a clue what might be wrong here?
> >>  
> >
> > It was tested with X86 and I think aarch64.
> > I am not sure how it would work with or if we
> > really need it for s390.
> >
> > We need the device to start a PCI hierarchy by plugging
> > it to PCIe Root Port. If I understand correctly s390
> > does not use PCIe Root Ports and may not need it.
> >  
> 
> Personally I can see 2 issues here:
> 1) [Marcel's already mentioned it] It seems that s390x machines have
> only pure PCI hosts,
> that's why it's a bit weird to put pcie-pci-bridge into the host
> 2) The actual segfault cause is bridge parent bus not having a parent
> device (somehow).
> Maybe it's a consequence of point 1)

Yes. PCI on s390x is weird; we basically have no topology and just make
one up (of which the guest sees nothing at all).

> 
> > If the assumption is correct I suppose we should only
> > be compiled into the above archs.
> > If you agree, I can post a patch for this.

I think disabling this for s390x makes sense.

      reply	other threads:[~2017-11-07 10:35 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-02 14:19 [Qemu-devel] Crash with pcie-pci-bridge on qemu-system-s390x Thomas Huth
2017-11-02 14:42 ` Marcel Apfelbaum
2017-11-02 14:59   ` Aleksandr Bezzubikov
2017-11-07 10:34     ` Cornelia Huck [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171107113446.6ac51e16.cohuck@redhat.com \
    --to=cohuck@redhat.com \
    --cc=marcel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-s390x@nongnu.org \
    --cc=thuth@redhat.com \
    --cc=zuban32s@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).