[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/11] pseries: Clean up error reporting in htab
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH 11/11] pseries: Clean up error reporting in htab migration functions |
Date: |
Mon, 14 Dec 2015 12:28:43 +1100 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Dec 11, 2015 at 08:22:59AM -0700, Eric Blake wrote:
> On 12/10/2015 05:11 PM, David Gibson wrote:
> > The functions for migrating the hash page table on pseries machine type
> > (htab_save_setup() and htab_load()) can report some errors with an
> > explicit fprintf() before returning an appropriate eror code. Change these
> > to use error_report() instead.
> >
> > Signed-off-by: David Gibson <address@hidden>
> > ---
> > hw/ppc/spapr.c | 17 +++++++++--------
> > 1 file changed, 9 insertions(+), 8 deletions(-)
> >
>
> >
> > @@ -1577,9 +1578,9 @@ static int htab_load(QEMUFile *f, void *opaque, int
> > version_id)
> > if ((index + n_valid + n_invalid) >
> > (HTAB_SIZE(spapr) / HASH_PTE_SIZE_64)) {
> > /* Bad index in stream */
> > - fprintf(stderr, "htab_load() bad index %d (%hd+%hd entries) "
> > - "in htab stream (htab_shift=%d)\n", index, n_valid,
> > n_invalid,
> > - spapr->htab_shift);
> > + error_report(
> > + "htab_load() bad index %d (%hd+%hd entries) in htab stream
> > (htab_shift=%d)\n",
> > + index, n_valid, n_invalid, spapr->htab_shift);
>
> No trailing newline to error_report().
Thanks. Changed that in most of the places I removed fprintf(), but
missed this one.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [PATCH 05/11] pseries: Cleanup error handling in spapr_vga_init(), (continued)
- [Qemu-devel] [PATCH 02/11] pseries: Cleanup error handling of spapr_cpu_init(), David Gibson, 2015/12/10
- [Qemu-devel] [PATCH 11/11] pseries: Clean up error reporting in htab migration functions, David Gibson, 2015/12/10
- [Qemu-devel] [PATCH 08/11] pseries: Clean up error handling in spapr_rtas_register(), David Gibson, 2015/12/10
- [Qemu-devel] [PATCH 09/11] pseries: Clean up error handling in xics_system_init(), David Gibson, 2015/12/10
- [Qemu-devel] [PATCH 10/11] pseries: Clean up error handling in ppc_spapr_init(), David Gibson, 2015/12/10
- [Qemu-devel] [PATCH 03/11] pseries: Clean up hash page table allocation error handling, David Gibson, 2015/12/10