Re: [Qemu-devel] [PATCH for-3.2 28/41] slirp: replace error_report() with g_critical()

2018-11-19 Thread Samuel Thibault
Marc-André Lureau, le mer. 14 nov. 2018 16:36:30 +0400, a ecrit:
> Reduce dependency on QEMU. QEMU could use a custom log handler if it
> wants to redirect/filter it.
> 
> Signed-off-by: Marc-André Lureau 

Applied to my tree, thanks!



[Qemu-devel] [PATCH for-3.2 28/41] slirp: replace error_report() with g_critical()

2018-11-14 Thread Marc-André Lureau
Reduce dependency on QEMU. QEMU could use a custom log handler if it
wants to redirect/filter it.

Signed-off-by: Marc-André Lureau 
---
 slirp/ip6_icmp.c | 2 +-
 slirp/misc.c | 4 ++--
 slirp/slirp.c| 6 +++---
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/slirp/ip6_icmp.c b/slirp/ip6_icmp.c
index 71d95daef0..e2e23d57bd 100644
--- a/slirp/ip6_icmp.c
+++ b/slirp/ip6_icmp.c
@@ -418,7 +418,7 @@ void icmp6_input(struct mbuf *m)
 icmp6_send_echoreply(m, slirp, ip, icmp);
 } else {
 /* TODO */
-error_report("external icmpv6 not supported yet");
+g_critical("external icmpv6 not supported yet");
 }
 break;
 
diff --git a/slirp/misc.c b/slirp/misc.c
index dd2b3512a8..17361b79a4 100644
--- a/slirp/misc.c
+++ b/slirp/misc.c
@@ -119,7 +119,7 @@ slirp_socketpair_with_oob(int sv[2])
 return 0;
 
 err:
-error_report("Error: slirp_socketpair(): %s", strerror(errno));
+g_critical("slirp_socketpair(): %s", strerror(errno));
 if (s >= 0) {
 closesocket(s);
 }
@@ -162,7 +162,7 @@ fork_exec(struct socket *so, const char *ex)
 g_strfreev(argv);
 
 if (err) {
-error_report("%s", err->message);
+g_critical("fork_exec: %s", err->message);
 g_error_free(err);
 closesocket(sp[0]);
 closesocket(sp[1]);
diff --git a/slirp/slirp.c b/slirp/slirp.c
index 864a9b5a7a..b1270a0309 100644
--- a/slirp/slirp.c
+++ b/slirp/slirp.c
@@ -1215,8 +1215,8 @@ static int sbuf_tmp_post_load(void *opaque, int version)
 }
 if (tmp->woff >= requested_len ||
 tmp->roff >= requested_len) {
-error_report("invalid sbuf offsets r/w=%u/%u len=%u",
- tmp->roff, tmp->woff, requested_len);
+g_critical("invalid sbuf offsets r/w=%u/%u len=%u",
+   tmp->roff, tmp->woff, requested_len);
 return -EINVAL;
 }
 
@@ -1324,7 +1324,7 @@ static int ss_family_post_load(void *opaque, int 
version_id)
 tss->parent->ss.ss_family = AF_INET6;
 break;
 default:
-error_report("invalid ss_family type %x", tss->portable_family);
+g_critical("invalid ss_family type %x", tss->portable_family);
 return -EINVAL;
 }
 
-- 
2.19.1.708.g4ede3d42df