[Qemu-devel] [PATCH] spapr: return from post_load method when RTC import fails

Cédric Le Goater posted 1 patch 5 years, 2 months ago
Test asan passed
Test checkpatch passed
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test docker-clang@ubuntu passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190104090610.652-1-clg@kaod.org
There is a newer version of this series
hw/ppc/spapr.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
[Qemu-devel] [PATCH] spapr: return from post_load method when RTC import fails
Posted by Cédric Le Goater 5 years, 2 months ago
The error value can be squashed by the section handling radix migration.
Simply bail out if an error occurs when the RTC offset is imported.

Fixes: d39c90f5f3ae ("spapr: Fix migration of Radix guests")

Signed-off-by: Cédric Le Goater <clg@kaod.org>
---
 hw/ppc/spapr.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index f1725313e979..64397ee91ef0 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -1748,12 +1748,17 @@ static int spapr_post_load(void *opaque, int version_id)
         return err;
     }
 
-    /* In earlier versions, there was no separate qdev for the PAPR
+    /*
+     * In earlier versions, there was no separate qdev for the PAPR
      * RTC, so the RTC offset was stored directly in sPAPREnvironment.
      * So when migrating from those versions, poke the incoming offset
-     * value into the RTC device */
+     * value into the RTC device
+     */
     if (version_id < 3) {
         err = spapr_rtc_import_offset(&spapr->rtc, spapr->rtc_offset);
+        if (err) {
+            return err;
+        }
     }
 
     if (kvm_enabled() && spapr->patb_entry) {
-- 
2.20.1


Re: [Qemu-devel] [PATCH] spapr: return from post_load method when RTC import fails
Posted by Peter Maydell 5 years, 2 months ago
On Fri, 4 Jan 2019 at 09:07, Cédric Le Goater <clg@kaod.org> wrote:
>
> The error value can be squashed by the section handling radix migration.
> Simply bail out if an error occurs when the RTC offset is imported.
>
> Fixes: d39c90f5f3ae ("spapr: Fix migration of Radix guests")

You could also mention that this fixes the Coverity
issue CID 1398591.

thanks
-- PMM

Re: [Qemu-devel] [PATCH] spapr: return from post_load method when RTC import fails
Posted by Cédric Le Goater 5 years, 2 months ago
On 1/4/19 11:08 AM, Peter Maydell wrote:
> On Fri, 4 Jan 2019 at 09:07, Cédric Le Goater <clg@kaod.org> wrote:
>>
>> The error value can be squashed by the section handling radix migration.
>> Simply bail out if an error occurs when the RTC offset is imported.
>>
>> Fixes: d39c90f5f3ae ("spapr: Fix migration of Radix guests")
> 
> You could also mention that this fixes the Coverity
> issue CID 1398591.

ok. I will in a v2.

Is there a tag for it ? 

or adding a comment "this fixes the Coverity issue CID 1398591." 
is considered sufficient ? 

Thanks,

C. 


Re: [Qemu-devel] [PATCH] spapr: return from post_load method when RTC import fails
Posted by Peter Maydell 5 years, 2 months ago
On Fri, 4 Jan 2019 at 12:31, Cédric Le Goater <clg@kaod.org> wrote:
>
> On 1/4/19 11:08 AM, Peter Maydell wrote:
> > On Fri, 4 Jan 2019 at 09:07, Cédric Le Goater <clg@kaod.org> wrote:
> >>
> >> The error value can be squashed by the section handling radix migration.
> >> Simply bail out if an error occurs when the RTC offset is imported.
> >>
> >> Fixes: d39c90f5f3ae ("spapr: Fix migration of Radix guests")
> >
> > You could also mention that this fixes the Coverity
> > issue CID 1398591.
>
> ok. I will in a v2.
>
> Is there a tag for it ?
>
> or adding a comment "this fixes the Coverity issue CID 1398591."
> is considered sufficient ?


No, we don't have a formal tag for that kind of thing.

thanks
-- PMM