[Qemu-devel] [PATCH] backends: remove empty trace-events file

Daniel P. Berrange posted 1 patch 6 years, 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170629162046.4135-1-berrange@redhat.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
Makefile.objs         | 1 -
backends/trace-events | 0
2 files changed, 1 deletion(-)
delete mode 100644 backends/trace-events
[Qemu-devel] [PATCH] backends: remove empty trace-events file
Posted by Daniel P. Berrange 6 years, 9 months ago
The content of the backends/trace-events file was entirely
removed in

  commit 6b10e573d15ef82dbc5c5b3726028e6642e134f6
  Author: Marc-André Lureau <marcandre.lureau@redhat.com>
  Date:   Mon May 29 12:39:42 2017 +0400

    char: move char devices to chardev/

Leaving the empty file around, causes tracetool to generate
an empty .dtrace file which makes the dtrace compiler throw
a syntax error.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
---
 Makefile.objs         | 1 -
 backends/trace-events | 0
 2 files changed, 1 deletion(-)
 delete mode 100644 backends/trace-events

diff --git a/Makefile.objs b/Makefile.objs
index b2e6322..8004d6d 100644
--- a/Makefile.objs
+++ b/Makefile.objs
@@ -122,7 +122,6 @@ trace-events-subdirs += crypto
 trace-events-subdirs += io
 trace-events-subdirs += migration
 trace-events-subdirs += block
-trace-events-subdirs += backends
 trace-events-subdirs += chardev
 trace-events-subdirs += hw/block
 trace-events-subdirs += hw/block/dataplane
diff --git a/backends/trace-events b/backends/trace-events
deleted file mode 100644
index e69de29..0000000
-- 
2.9.4


Re: [Qemu-devel] [PATCH] backends: remove empty trace-events file
Posted by Eric Blake 6 years, 9 months ago
On 06/29/2017 11:20 AM, Daniel P. Berrange wrote:
> The content of the backends/trace-events file was entirely
> removed in
> 
>   commit 6b10e573d15ef82dbc5c5b3726028e6642e134f6
>   Author: Marc-André Lureau <marcandre.lureau@redhat.com>
>   Date:   Mon May 29 12:39:42 2017 +0400
> 
>     char: move char devices to chardev/
> 
> Leaving the empty file around, causes tracetool to generate
> an empty .dtrace file which makes the dtrace compiler throw
> a syntax error.
> 
> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
> ---
>  Makefile.objs         | 1 -
>  backends/trace-events | 0
>  2 files changed, 1 deletion(-)
>  delete mode 100644 backends/trace-events

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [Qemu-trivial] [PATCH] backends: remove empty trace-events file
Posted by Philippe Mathieu-Daudé 6 years, 9 months ago
On 06/29/2017 01:20 PM, Daniel P. Berrange wrote:
> The content of the backends/trace-events file was entirely
> removed in
> 
>    commit 6b10e573d15ef82dbc5c5b3726028e6642e134f6
>    Author: Marc-André Lureau <marcandre.lureau@redhat.com>
>    Date:   Mon May 29 12:39:42 2017 +0400
> 
>      char: move char devices to chardev/
> 
> Leaving the empty file around, causes tracetool to generate
> an empty .dtrace file which makes the dtrace compiler throw
> a syntax error.
> 
> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

> ---
>   Makefile.objs         | 1 -
>   backends/trace-events | 0
>   2 files changed, 1 deletion(-)
>   delete mode 100644 backends/trace-events
> 
> diff --git a/Makefile.objs b/Makefile.objs
> index b2e6322..8004d6d 100644
> --- a/Makefile.objs
> +++ b/Makefile.objs
> @@ -122,7 +122,6 @@ trace-events-subdirs += crypto
>   trace-events-subdirs += io
>   trace-events-subdirs += migration
>   trace-events-subdirs += block
> -trace-events-subdirs += backends
>   trace-events-subdirs += chardev
>   trace-events-subdirs += hw/block
>   trace-events-subdirs += hw/block/dataplane
> diff --git a/backends/trace-events b/backends/trace-events
> deleted file mode 100644
> index e69de29..0000000
> 

Re: [Qemu-devel] [PATCH] backends: remove empty trace-events file
Posted by Stefan Hajnoczi 6 years, 9 months ago
On Thu, Jun 29, 2017 at 05:20:46PM +0100, Daniel P. Berrange wrote:
> The content of the backends/trace-events file was entirely
> removed in
> 
>   commit 6b10e573d15ef82dbc5c5b3726028e6642e134f6
>   Author: Marc-André Lureau <marcandre.lureau@redhat.com>
>   Date:   Mon May 29 12:39:42 2017 +0400
> 
>     char: move char devices to chardev/
> 
> Leaving the empty file around, causes tracetool to generate
> an empty .dtrace file which makes the dtrace compiler throw
> a syntax error.
> 
> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
> ---
>  Makefile.objs         | 1 -
>  backends/trace-events | 0
>  2 files changed, 1 deletion(-)
>  delete mode 100644 backends/trace-events

Thanks, applied to my tracing tree:
https://github.com/stefanha/qemu/commits/tracing

Stefan