From: Julien Grall <jgrall@amazon.com>
A prototype for dump_conn() has been present for quite a long time
but there are no implementation. Even, AFAICT in the patch that
introduced it. So drop it.
Signed-off-by: Julien Grall <jgrall@amazon.com>
---
tools/xenstore/xenstored_core.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c
index d54a6042a9f7..591b28e4552f 100644
--- a/tools/xenstore/xenstored_core.c
+++ b/tools/xenstore/xenstored_core.c
@@ -2085,7 +2085,6 @@ static struct option options[] = {
#endif
{ NULL, 0, NULL, 0 } };
-extern void dump_conn(struct connection *conn);
int dom0_domid = 0;
int dom0_event = 0;
int priv_domid = 0;
--
2.17.1
Hi Julien, On 20 Apr 2021, at 14:49, Julien Grall <julien@xen.org<mailto:julien@xen.org>> wrote: From: Julien Grall <jgrall@amazon.com<mailto:jgrall@amazon.com>> A prototype for dump_conn() has been present for quite a long time but there are no implementation. Even, AFAICT in the patch that introduced it. So drop it. Signed-off-by: Julien Grall <jgrall@amazon.com<mailto:jgrall@amazon.com>> Agree, no implementation of this exists so: Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com<mailto:bertrand.marquis@arm.com>> Cheers Bertrand --- tools/xenstore/xenstored_core.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c index d54a6042a9f7..591b28e4552f 100644 --- a/tools/xenstore/xenstored_core.c +++ b/tools/xenstore/xenstored_core.c @@ -2085,7 +2085,6 @@ static struct option options[] = { #endif { NULL, 0, NULL, 0 } }; -extern void dump_conn(struct connection *conn); int dom0_domid = 0; int dom0_event = 0; int priv_domid = 0; -- 2.17.1
On 20.04.21 15:49, Julien Grall wrote: > From: Julien Grall <jgrall@amazon.com> > > A prototype for dump_conn() has been present for quite a long time > but there are no implementation. Even, AFAICT in the patch that > introduced it. So drop it. > > Signed-off-by: Julien Grall <jgrall@amazon.com> Reviewed-by: Juergen Gross <jgross@suse.com> Juergen
Hi Juergen, On 21/04/2021 08:39, Juergen Gross wrote: > On 20.04.21 15:49, Julien Grall wrote: >> From: Julien Grall <jgrall@amazon.com> >> >> A prototype for dump_conn() has been present for quite a long time >> but there are no implementation. Even, AFAICT in the patch that >> introduced it. So drop it. >> >> Signed-off-by: Julien Grall <jgrall@amazon.com> > > Reviewed-by: Juergen Gross <jgross@suse.com> Thanks! I have committed it. Cheers, -- Julien Grall
© 2016 - 2024 Red Hat, Inc.