On Wed, 26 Jul 2017 09:07:52 -0700
Alistair Francis <alistair.fran...@xilinx.com> wrote:
> In a previous patch (3dc6f8693694a649a9c83f1e2746565b47683923) we
> converted uses of error_report("warning:"... to use warn_report()
> instead. This was to help standardise on a single method of printing
> warnings to the user.
>
> There appears to have been some cases that slipped through in patch sets
> applied around the same time, this patch catches the few remaining
> cases.
>
> All of the warnings were changed using this command:
> find ./* -type f -exec sed -i \
> 's|error_report(".*warning[,:] |warn_report("|Ig' {} +
>
> Indentation fixed up manually afterwards.
>
> Signed-off-by: Alistair Francis <alistair.fran...@xilinx.com>
> Cc: Kevin Wolf <kw...@redhat.com>
> Cc: Max Reitz <mre...@redhat.com>
> Cc: Christian Borntraeger <borntrae...@de.ibm.com>
> Cc: Cornelia Huck <coh...@redhat.com>
> Cc: Alexander Graf <ag...@suse.de>
> Cc: Richard Henderson <r...@twiddle.net>
> Cc: Stefan Hajnoczi <stefa...@redhat.com>
Hm, the automatic cc:ing does not seem to have worked.
> ---
>
> block/qcow2.c | 10 +++++-----
> target/s390x/kvm.c | 4 ++--
> trace/control.c | 4 ++--
> 3 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> index c4c5791d27..1084923adb 100644
> --- a/target/s390x/kvm.c
> +++ b/target/s390x/kvm.c
> @@ -222,8 +222,8 @@ static void kvm_s390_enable_cmma(void)
> };
>
> if (mem_path) {
> - error_report("Warning: CMM will not be enabled because it is not "
> - "compatible to hugetlbfs.");
> + warn_report("CMM will not be enabled because it is not "
> + "compatible to hugetlbfs.");
> return;
> }
> rc = kvm_vm_ioctl(kvm_state, KVM_SET_DEVICE_ATTR, &attr);
s390x part:
Acked-by: Cornelia Huck <coh...@redhat.com>
