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 <da...@gibson.dropbear.id.au> > --- > 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().
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
