Message ID | 1480583289-6380-1-git-send-email-mukesh02@linux.vnet.ibm.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers | show |
Hi Michael, Can you please have a look at this patchset as there is no functional changes involve with this? Thanks, Mukesh On Thursday 01 December 2016 02:38 PM, Mukesh Ojha wrote: > Moves the return value check of 'opal_dump_info' to a proper place which > was previously unnecessarily filling all the dump info even on failure. > > Signed-off-by: Mukesh Ojha <mukesh02@linux.vnet.ibm.com> > --- > arch/powerpc/platforms/powernv/opal-dump.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/arch/powerpc/platforms/powernv/opal-dump.c b/arch/powerpc/platforms/powernv/opal-dump.c > index 4c82782..ae32212 100644 > --- a/arch/powerpc/platforms/powernv/opal-dump.c > +++ b/arch/powerpc/platforms/powernv/opal-dump.c > @@ -225,13 +225,16 @@ static int64_t dump_read_info(uint32_t *dump_id, uint32_t *dump_size, uint32_t * > if (rc == OPAL_PARAMETER) > rc = opal_dump_info(&id, &size); > > + if (rc) { > + pr_warn("%s: Failed to get dump info (%d)\n", > + __func__, rc); > + return rc; > + } > + > *dump_id = be32_to_cpu(id); > *dump_size = be32_to_cpu(size); > *dump_type = be32_to_cpu(type); > > - if (rc) > - pr_warn("%s: Failed to get dump info (%d)\n", > - __func__, rc); > return rc; > } >
Hi Micheal, Can you please look at this patchset? -Mukesh On Tuesday 06 December 2016 12:07 PM, Mukesh Ojha wrote: > Hi Michael, > > Can you please have a look at this patchset as there is no > > functional changes involve with this? > > Thanks, > Mukesh > > > > > On Thursday 01 December 2016 02:38 PM, Mukesh Ojha wrote: >> Moves the return value check of 'opal_dump_info' to a proper place which >> was previously unnecessarily filling all the dump info even on failure. >> >> Signed-off-by: Mukesh Ojha <mukesh02@linux.vnet.ibm.com> >> --- >> arch/powerpc/platforms/powernv/opal-dump.c | 9 ++++++--- >> 1 file changed, 6 insertions(+), 3 deletions(-) >> >> diff --git a/arch/powerpc/platforms/powernv/opal-dump.c >> b/arch/powerpc/platforms/powernv/opal-dump.c >> index 4c82782..ae32212 100644 >> --- a/arch/powerpc/platforms/powernv/opal-dump.c >> +++ b/arch/powerpc/platforms/powernv/opal-dump.c >> @@ -225,13 +225,16 @@ static int64_t dump_read_info(uint32_t >> *dump_id, uint32_t *dump_size, uint32_t * >> if (rc == OPAL_PARAMETER) >> rc = opal_dump_info(&id, &size); >> >> + if (rc) { >> + pr_warn("%s: Failed to get dump info (%d)\n", >> + __func__, rc); >> + return rc; >> + } >> + >> *dump_id = be32_to_cpu(id); >> *dump_size = be32_to_cpu(size); >> *dump_type = be32_to_cpu(type); >> >> - if (rc) >> - pr_warn("%s: Failed to get dump info (%d)\n", >> - __func__, rc); >> return rc; >> } >> >
Hi Mukesh, > Moves the return value check of 'opal_dump_info' to a proper place which > was previously unnecessarily filling all the dump info even on failure. Acked-by: Jeremy Kerr <jk@ozlabs.org> Thanks! Jeremy
Mukesh Ojha <mukesh02@linux.vnet.ibm.com> writes: > Moves the return value check of 'opal_dump_info' to a proper place which > was previously unnecessarily filling all the dump info even on failure. > > Signed-off-by: Mukesh Ojha <mukesh02@linux.vnet.ibm.com> > --- > arch/powerpc/platforms/powernv/opal-dump.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) Acked-by: Stewart Smith <stewart@linux.vnet.ibm.com>
diff --git a/arch/powerpc/platforms/powernv/opal-dump.c b/arch/powerpc/platforms/powernv/opal-dump.c index 4c82782..ae32212 100644 --- a/arch/powerpc/platforms/powernv/opal-dump.c +++ b/arch/powerpc/platforms/powernv/opal-dump.c @@ -225,13 +225,16 @@ static int64_t dump_read_info(uint32_t *dump_id, uint32_t *dump_size, uint32_t * if (rc == OPAL_PARAMETER) rc = opal_dump_info(&id, &size); + if (rc) { + pr_warn("%s: Failed to get dump info (%d)\n", + __func__, rc); + return rc; + } + *dump_id = be32_to_cpu(id); *dump_size = be32_to_cpu(size); *dump_type = be32_to_cpu(type); - if (rc) - pr_warn("%s: Failed to get dump info (%d)\n", - __func__, rc); return rc; }
Moves the return value check of 'opal_dump_info' to a proper place which was previously unnecessarily filling all the dump info even on failure. Signed-off-by: Mukesh Ojha <mukesh02@linux.vnet.ibm.com> --- arch/powerpc/platforms/powernv/opal-dump.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)