[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 0/3] dataplane: dataplane: more graceful err
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH RFC 0/3] dataplane: dataplane: more graceful error handling |
Date: |
Thu, 7 Aug 2014 16:39:01 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 25.07.2014 um 14:10 hat Cornelia Huck geschrieben:
> Currently, qemu will take a hard exit if it fails to set up guest or
> host notifiers, giving no real clue as to what went wrong (e.g., when
> out of file descriptors).
>
> This patchset tries to make this more manageable: Both by improving the
> error message and by gracefully falling back to non-dataplane in case of
> errors.
>
> Patches are also available on
>
> git://github.com/cohuck/qemu dataplane-graceful-fail
>
> Thoughts?
I think Stefan should comment on this, but I certainly welcome every
patch that fixes an exit(1) call.
I'm not entirely sure about the added fprintf(). It feels wrong, but of
course it's a lot less wrong than exiting. Ideally already adding the
device with dataplane enabled would fail so that we can return a proper
QMP error message instead of just dumping something on stderr. Not sure
if it's possible, though, I don't really know that code.
Nothing to stop this series anyway.
Kevin