[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals |
Date: |
Sat, 7 Jun 2014 09:55:01 +0100 |
On 7 June 2014 02:09, Eduardo Habkost <address@hidden> wrote:
> On Sat, Jun 07, 2014 at 12:45:26AM +0100, Peter Maydell wrote:
>> I'm with Eduardo on this one -- if the user passes us a bad
>> command line we should diagnose it helpfully and exit with
>> a failure code; abort() is for programming errors. (If nothing
>> else, using abort() for user-triggerable conditions tends to
>> mean we get bug reports about core dumps, so it's in our
>> own interest to not do that :-) )
>
> So you are agreeing with Igor, as well. He is just suggesting we keep
> terminating QEMU on that case, and we can use exit() for that. My patch,
> on the other hand, makes QEMU not terminate on those cases.
No, I'm happy with not terminating and handling the case
correctly if that makes more sense for this case; I'm just
saying that dumping core is definitely not OK.
thanks
-- PMM
- [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals, Eduardo Habkost, 2014/06/06
- Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals, Don Slutz, 2014/06/06
- Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals, Igor Mammedov, 2014/06/06
- Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals, Eduardo Habkost, 2014/06/06
- Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals, Igor Mammedov, 2014/06/06
- Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals, Peter Maydell, 2014/06/06
- Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals, Eduardo Habkost, 2014/06/06
- [Qemu-devel] [PATCH] qdev: Don't abort() in case globals can't be set, Eduardo Habkost, 2014/06/06
- Re: [Qemu-devel] [PATCH] qdev: Don't abort() in case globals can't be set, Michael S. Tsirkin, 2014/06/08
- Re: [Qemu-devel] [PATCH] qdev: Don't abort() in case globals can't be set, Igor Mammedov, 2014/06/09
- Re: [Qemu-devel] [PATCH] qdev: Skip non-existing properties when setting globals,
Peter Maydell <=