Re: [Qemu-devel] [PATCH v4 0/3] qapi/error: converts error_setg(_fatal) to error_report() + exit()

[Philippe Mathieu-Daudé]

On 06/28/2018 10:51 AM, Peter Maydell wrote:
> On 25 June 2018 at 17:57, Philippe Mathieu-Daudé  wrote:
>> Hi,
>>
>> This series converts error_setg(_fatal) to error_report() + exit() as
>> suggested by the "qapi/error.h" documentation.
>>
>> This reduce Coverity and Clang static analyzer positive falses.
>>
>> See http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg07585.html:
>>
>> On 07/24/2017 04:52 PM, Eric Blake wrote:
>> That's a shame.  Rather, we should patch this file (and others) to avoid
>> all the inconsistent uses of error_setg(_*), to comply with the
>> error.h documentation.
>>
> 
> Applied to target-arm.next, thanks. (Not all arm code, but mostly
> close enough that that seems the easiest way to deal with it.)

Thanks!

Re: [Qemu-devel] [PATCH v4 0/3] qapi/error: converts error_setg(_fatal) to error_report() + exit()

[Peter Maydell]

On 25 June 2018 at 17:57, Philippe Mathieu-Daudé  wrote:
> Hi,
>
> This series converts error_setg(_fatal) to error_report() + exit() as
> suggested by the "qapi/error.h" documentation.
>
> This reduce Coverity and Clang static analyzer positive falses.
>
> See http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg07585.html:
>
> On 07/24/2017 04:52 PM, Eric Blake wrote:
> That's a shame.  Rather, we should patch this file (and others) to avoid
> all the inconsistent uses of error_setg(_*), to comply with the
> error.h documentation.
>

Applied to target-arm.next, thanks. (Not all arm code, but mostly
close enough that that seems the easiest way to deal with it.)

thanks
-- PMM

[Qemu-devel] [PATCH v4 0/3] qapi/error: converts error_setg(_fatal) to error_report() + exit()

[Philippe Mathieu-Daudé]

Hi,

This series converts error_setg(_fatal) to error_report() + exit() as
suggested by the "qapi/error.h" documentation.

This reduce Coverity and Clang static analyzer positive falses.

See http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg07585.html:

On 07/24/2017 04:52 PM, Eric Blake wrote:
That's a shame.  Rather, we should patch this file (and others) to avoid
all the inconsistent uses of error_setg(_*), to comply with the
error.h documentation.

Since v3:
- corrected error display in copy_properties_from_host (Eric)
  display error if property required and returned -FDT_ERR_NOTFOUND
- resetted Markus R-b tag

Since v2:
- added R-b, A-b (Markus, John, David)
- fixed incorrect update of sysbus-fdt patch between v1->v2 (Markus)

Since v1:
- patch #1: use assert() directly (Markus explanation)
- patch #2: use abort() without error_report() (Markus 'no lipstick')
- patch #3: replaced exit() by assert() (Markus)
- patch #4: no change, added R-b

Regards,

Phil.

[] : patches are identical
[] : number of functional differences between upstream/downstream patch
The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively

001/3:[] [--] 'hw/block/fdc: Replace error_setg(_abort) by assert()'
002/3:[0010] [FC] 'hw/arm/sysbus-fdt: Replace error_setg(_fatal) by 
error_report() + exit()'
003/3:[] [--] 'device_tree: Replace error_setg(_fatal) by 
error_report() + exit()'

Philippe Mathieu-Daudé (3):
  hw/block/fdc: Replace error_setg(_abort) by assert()
  hw/arm/sysbus-fdt: Replace error_setg(_fatal) by error_report() + exit()
  device_tree: Replace error_setg(_fatal) by error_report() + exit()

 device_tree.c   | 23 +++-
 hw/arm/sysbus-fdt.c | 53 +
 hw/block/fdc.c  |  9 +---
 3 files changed, 44 insertions(+), 41 deletions(-)

-- 
2.18.0