Re: [PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-18 Thread Auger Eric
Hi Stefan, On 8/18/20 2:54 PM, Stefan Hajnoczi wrote: > On Thu, Aug 13, 2020 at 09:18:59PM +0200, Auger Eric wrote: >> Hi Alex, >> >> On 8/13/20 9:15 PM, Alex Williamson wrote: >>> On Thu, 13 Aug 2020 20:02:45 +0200 >>> Auger Eric wrote: >>> Hi Alex, On 8/13/20 6:59 PM, Alex Willia

Re: [PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-18 Thread Stefan Hajnoczi
On Thu, Aug 13, 2020 at 09:18:59PM +0200, Auger Eric wrote: > Hi Alex, > > On 8/13/20 9:15 PM, Alex Williamson wrote: > > On Thu, 13 Aug 2020 20:02:45 +0200 > > Auger Eric wrote: > > > >> Hi Alex, > >> > >> On 8/13/20 6:59 PM, Alex Williamson wrote: > >>> On Thu, 13 Aug 2020 15:37:08 +0800 > >>>

Re: [PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-13 Thread Auger Eric
Hi Alex, On 8/13/20 9:15 PM, Alex Williamson wrote: > On Thu, 13 Aug 2020 20:02:45 +0200 > Auger Eric wrote: > >> Hi Alex, >> >> On 8/13/20 6:59 PM, Alex Williamson wrote: >>> On Thu, 13 Aug 2020 15:37:08 +0800 >>> Chen Qun wrote: >>> Clang static code analyzer show warning: hw/vfi

Re: [PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-13 Thread Alex Williamson
On Thu, 13 Aug 2020 20:02:45 +0200 Auger Eric wrote: > Hi Alex, > > On 8/13/20 6:59 PM, Alex Williamson wrote: > > On Thu, 13 Aug 2020 15:37:08 +0800 > > Chen Qun wrote: > > > >> Clang static code analyzer show warning: > >> hw/vfio/platform.c:239:9: warning: Value stored to 'ret' is never r

Re: [PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-13 Thread Auger Eric
Hi Alex, On 8/13/20 6:59 PM, Alex Williamson wrote: > On Thu, 13 Aug 2020 15:37:08 +0800 > Chen Qun wrote: > >> Clang static code analyzer show warning: >> hw/vfio/platform.c:239:9: warning: Value stored to 'ret' is never read >> ret = event_notifier_test_and_clear(intp->interrupt); >>

Re: [PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-13 Thread Alex Williamson
On Thu, 13 Aug 2020 15:37:08 +0800 Chen Qun wrote: > Clang static code analyzer show warning: > hw/vfio/platform.c:239:9: warning: Value stored to 'ret' is never read > ret = event_notifier_test_and_clear(intp->interrupt); > ^ ~~ >

Re: [PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-13 Thread Auger Eric
Hi, On 8/13/20 9:37 AM, Chen Qun wrote: > Clang static code analyzer show warning: > hw/vfio/platform.c:239:9: warning: Value stored to 'ret' is never read > ret = event_notifier_test_and_clear(intp->interrupt); > ^ ~~ > > Reported-b

[PATCH 07/11] vfio/platform: Remove dead assignment in vfio_intp_interrupt()

2020-08-13 Thread Chen Qun
Clang static code analyzer show warning: hw/vfio/platform.c:239:9: warning: Value stored to 'ret' is never read ret = event_notifier_test_and_clear(intp->interrupt); ^ ~~ Reported-by: Euler Robot Signed-off-by: Chen Qun --- Cc: Alex