[Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()

Oleinik, Alexander posted 1 patch 4 years, 7 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190805031240.6024-1-alxndr@bu.edu
Test checkpatch passed
Test s390x passed
Test asan passed
Test docker-mingw@fedora passed
Test FreeBSD passed
Test docker-clang@ubuntu passed
Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>
include/sysemu/qtest.h | 2 +-
qtest.c                | 3 +--
vl.c                   | 2 +-
3 files changed, 3 insertions(+), 4 deletions(-)
[Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
Posted by Oleinik, Alexander 4 years, 7 months ago
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


Re: [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
Posted by John Snow 4 years, 7 months ago

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>

Re: [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
Posted by Stefan Hajnoczi 4 years, 7 months ago
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>
Re: [Qemu-devel] [PATCH] qtest: Rename qtest.c:qtest_init()
Posted by Thomas Huth 4 years, 7 months ago
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