* [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
@ 2019-08-05 3:13 Oleinik, Alexander
2019-08-05 18:12 ` John Snow
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Oleinik, Alexander @ 2019-08-05 3:13 UTC (permalink / raw)
To: qemu-devel
Cc: Laurent Vivier, thuth, Oleinik, Alexander, bsd, stefanha, pbonzini
Both the qtest client, libqtest.c, and server, qtest.c, used the same
name for initialization functions which can cause confusion.
Signed-off-by: Alexander Oleinik <alxndr@bu.edu>
---
Thank you, Thomas Huth for the suggestion.
include/sysemu/qtest.h | 2 +-
qtest.c | 3 +--
vl.c | 2 +-
3 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/include/sysemu/qtest.h b/include/sysemu/qtest.h
index cd114b8d80..5ed09c80b1 100644
--- a/include/sysemu/qtest.h
+++ b/include/sysemu/qtest.h
@@ -24,6 +24,6 @@ static inline bool qtest_enabled(void)
bool qtest_driver(void);
-void qtest_init(const char *qtest_chrdev, const char *qtest_log, Error **errp);
+void qtest_server_init(const char *qtest_chrdev, const char *qtest_log, Error **errp);
#endif
diff --git a/qtest.c b/qtest.c
index 15e27e911f..c9681dbdf3 100644
--- a/qtest.c
+++ b/qtest.c
@@ -748,8 +748,7 @@ static void qtest_event(void *opaque, int event)
break;
}
}
-
-void qtest_init(const char *qtest_chrdev, const char *qtest_log, Error **errp)
+void qtest_server_init(const char *qtest_chrdev, const char *qtest_log, Error **errp)
{
Chardev *chr;
diff --git a/vl.c b/vl.c
index b426b32134..130a389712 100644
--- a/vl.c
+++ b/vl.c
@@ -4197,7 +4197,7 @@ int main(int argc, char **argv, char **envp)
migration_object_init();
if (qtest_chrdev) {
- qtest_init(qtest_chrdev, qtest_log, &error_fatal);
+ qtest_server_init(qtest_chrdev, qtest_log, &error_fatal);
}
machine_opts = qemu_get_machine_opts();
--
2.20.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
2019-08-05 3:13 [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init() Oleinik, Alexander
@ 2019-08-05 18:12 ` John Snow
2019-08-08 15:36 ` Stefan Hajnoczi
2019-08-13 8:48 ` Thomas Huth
2 siblings, 0 replies; 4+ messages in thread
From: John Snow @ 2019-08-05 18:12 UTC (permalink / raw)
To: Oleinik, Alexander, qemu-devel
Cc: Laurent Vivier, pbonzini, thuth, bsd, stefanha
On 8/4/19 11:13 PM, Oleinik, Alexander wrote:
> Both the qtest client, libqtest.c, and server, qtest.c, used the same
> name for initialization functions which can cause confusion.
>
> Signed-off-by: Alexander Oleinik <alxndr@bu.edu>
> ---
> Thank you, Thomas Huth for the suggestion.
Sometimes we use:
Suggested-by: Thomas Huth <thuth@redhat.com>
to give people credit in a way that our tooling has the ability to
track. We don't really rigorously check or enforce such things, though.
>
> include/sysemu/qtest.h | 2 +-
> qtest.c | 3 +--
> vl.c | 2 +-
> 3 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/include/sysemu/qtest.h b/include/sysemu/qtest.h
> index cd114b8d80..5ed09c80b1 100644
> --- a/include/sysemu/qtest.h
> +++ b/include/sysemu/qtest.h
> @@ -24,6 +24,6 @@ static inline bool qtest_enabled(void)
>
> bool qtest_driver(void);
>
> -void qtest_init(const char *qtest_chrdev, const char *qtest_log, Error **errp);
> +void qtest_server_init(const char *qtest_chrdev, const char *qtest_log, Error **errp);
>
> #endif
> diff --git a/qtest.c b/qtest.c
> index 15e27e911f..c9681dbdf3 100644
> --- a/qtest.c
> +++ b/qtest.c
> @@ -748,8 +748,7 @@ static void qtest_event(void *opaque, int event)
> break;
> }
> }
> -
> -void qtest_init(const char *qtest_chrdev, const char *qtest_log, Error **errp)
> +void qtest_server_init(const char *qtest_chrdev, const char *qtest_log, Error **errp)
> {
> Chardev *chr;
>
> diff --git a/vl.c b/vl.c
> index b426b32134..130a389712 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4197,7 +4197,7 @@ int main(int argc, char **argv, char **envp)
> migration_object_init();
>
> if (qtest_chrdev) {
> - qtest_init(qtest_chrdev, qtest_log, &error_fatal);
> + qtest_server_init(qtest_chrdev, qtest_log, &error_fatal);
> }
>
> machine_opts = qemu_get_machine_opts();
>
Makes good sense to me, that IS confusing.
Reviewed-by: John Snow <jsnow@redhat.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
2019-08-05 3:13 [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init() Oleinik, Alexander
2019-08-05 18:12 ` John Snow
@ 2019-08-08 15:36 ` Stefan Hajnoczi
2019-08-13 8:48 ` Thomas Huth
2 siblings, 0 replies; 4+ messages in thread
From: Stefan Hajnoczi @ 2019-08-08 15:36 UTC (permalink / raw)
To: Oleinik, Alexander; +Cc: bsd, pbonzini, thuth, qemu-devel, Laurent Vivier
[-- Attachment #1: Type: text/plain, Size: 529 bytes --]
On Mon, Aug 05, 2019 at 03:13:01AM +0000, Oleinik, Alexander wrote:
> Both the qtest client, libqtest.c, and server, qtest.c, used the same
> name for initialization functions which can cause confusion.
>
> Signed-off-by: Alexander Oleinik <alxndr@bu.edu>
> ---
> Thank you, Thomas Huth for the suggestion.
>
> include/sysemu/qtest.h | 2 +-
> qtest.c | 3 +--
> vl.c | 2 +-
> 3 files changed, 3 insertions(+), 4 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
2019-08-05 3:13 [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init() Oleinik, Alexander
2019-08-05 18:12 ` John Snow
2019-08-08 15:36 ` Stefan Hajnoczi
@ 2019-08-13 8:48 ` Thomas Huth
2 siblings, 0 replies; 4+ messages in thread
From: Thomas Huth @ 2019-08-13 8:48 UTC (permalink / raw)
To: Oleinik, Alexander, qemu-devel; +Cc: Laurent Vivier, pbonzini, bsd, stefanha
On 8/5/19 5:13 AM, Oleinik, Alexander wrote:
> Both the qtest client, libqtest.c, and server, qtest.c, used the same
> name for initialization functions which can cause confusion.
>
> Signed-off-by: Alexander Oleinik <alxndr@bu.edu>
> ---
> Thank you, Thomas Huth for the suggestion.
Thanks you for the patch, it's now queued in my qtest tree:
https://gitlab.com/huth/qemu/commits/qtest-next
Thomas
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-08-13 8:49 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-05 3:13 [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init() Oleinik, Alexander
2019-08-05 18:12 ` John Snow
2019-08-08 15:36 ` Stefan Hajnoczi
2019-08-13 8:48 ` Thomas Huth
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).