* [Qemu-devel] [PATCH v0 1/6] move tcg relative files into tcg directory
2017-05-19 7:30 [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Yang Zhong
@ 2017-05-19 7:30 ` Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 2/6] " Yang Zhong
` (5 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Yang Zhong @ 2017-05-19 7:30 UTC (permalink / raw)
To: pbonzini; +Cc: qemu-devel, anthony.xu, Yang Zhong
move tcg-runtime.c to ./tcg directory
Signed-off-by: Yang Zhong <yang.zhong@intel.com>
---
Makefile.target | 2 +-
tcg-runtime.c => tcg/tcg-runtime.c | 0
2 files changed, 1 insertion(+), 1 deletion(-)
rename tcg-runtime.c => tcg/tcg-runtime.c (100%)
diff --git a/Makefile.target b/Makefile.target
index 465a633..c388fd6 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -98,7 +98,7 @@ obj-$(CONFIG_TCG_INTERPRETER) += disas/tci.o
obj-y += fpu/softfloat.o
obj-y += target/$(TARGET_BASE_ARCH)/
obj-y += disas.o
-obj-y += tcg-runtime.o
+obj-y += tcg/tcg-runtime.o
obj-$(call notempty,$(TARGET_XML_FILES)) += gdbstub-xml.o
obj-$(call lnot,$(CONFIG_HAX)) += hax-stub.o
obj-$(call lnot,$(CONFIG_KVM)) += kvm-stub.o
diff --git a/tcg-runtime.c b/tcg/tcg-runtime.c
similarity index 100%
rename from tcg-runtime.c
rename to tcg/tcg-runtime.c
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Qemu-devel] [PATCH v0 2/6] move tcg relative files into tcg directory
2017-05-19 7:30 [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 1/6] move tcg relative " Yang Zhong
@ 2017-05-19 7:30 ` Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 3/6] move tcg header file Yang Zhong
` (4 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Yang Zhong @ 2017-05-19 7:30 UTC (permalink / raw)
To: pbonzini; +Cc: qemu-devel, anthony.xu, Yang Zhong
move translate-all.c file into tcg directory and update
trace-events file
Signed-off-by: Yang Zhong <yang.zhong@intel.com>
---
Makefile.objs | 1 +
Makefile.target | 3 ++-
tcg/trace-events | 6 ++++++
translate-all.c => tcg/translate-all.c | 2 +-
trace-events | 3 ---
5 files changed, 10 insertions(+), 5 deletions(-)
create mode 100644 tcg/trace-events
rename translate-all.c => tcg/translate-all.c (99%)
diff --git a/Makefile.objs b/Makefile.objs
index 2100845..1de6462 100644
--- a/Makefile.objs
+++ b/Makefile.objs
@@ -163,6 +163,7 @@ trace-events-subdirs += target/ppc
trace-events-subdirs += qom
trace-events-subdirs += linux-user
trace-events-subdirs += qapi
+trace-events-subdirs += tcg
trace-events-files = $(SRC_PATH)/trace-events $(trace-events-subdirs:%=$(SRC_PATH)/%/trace-events)
diff --git a/Makefile.target b/Makefile.target
index c388fd6..c96ebd2 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -88,7 +88,8 @@ all: $(PROGS) stap
#########################################################
# cpu emulator library
-obj-y = exec.o translate-all.o cpu-exec.o
+obj-y = exec.o cpu-exec.o
+obj-y += tcg/translate-all.o
obj-y += translate-common.o
obj-y += cpu-exec-common.o
obj-y += tcg/tcg.o tcg/tcg-op.o tcg/optimize.o
diff --git a/tcg/trace-events b/tcg/trace-events
new file mode 100644
index 0000000..989a06e
--- /dev/null
+++ b/tcg/trace-events
@@ -0,0 +1,6 @@
+# Trace events for debugging and performance instrumentation
+#
+# TCG related tracing (mostly disabled by default)
+# translate-all.c
+translate_block(void *tb, uintptr_t pc, uint8_t *tb_code) "tb:%p, pc:0x%"PRIxPTR", tb_code:%p"
+
diff --git a/translate-all.c b/tcg/translate-all.c
similarity index 99%
rename from translate-all.c
rename to tcg/translate-all.c
index b3ee876..7b25a16 100644
--- a/translate-all.c
+++ b/tcg/translate-all.c
@@ -25,7 +25,7 @@
#include "qemu-common.h"
#define NO_CPU_IO_DEFS
#include "cpu.h"
-#include "trace-root.h"
+#include "trace.h"
#include "disas/disas.h"
#include "exec/exec-all.h"
#include "tcg.h"
diff --git a/trace-events b/trace-events
index e582d63..d89380e 100644
--- a/trace-events
+++ b/trace-events
@@ -78,9 +78,6 @@ disable exec_tb(void *tb, uintptr_t pc) "tb:%p pc=0x%"PRIxPTR
disable exec_tb_nocache(void *tb, uintptr_t pc) "tb:%p pc=0x%"PRIxPTR
disable exec_tb_exit(void *last_tb, unsigned int flags) "tb:%p flags=%x"
-# translate-all.c
-translate_block(void *tb, uintptr_t pc, uint8_t *tb_code) "tb:%p, pc:0x%"PRIxPTR", tb_code:%p"
-
# memory.c
memory_region_ops_read(int cpu_index, void *mr, uint64_t addr, uint64_t value, unsigned size) "cpu %d mr %p addr %#"PRIx64" value %#"PRIx64" size %u"
memory_region_ops_write(int cpu_index, void *mr, uint64_t addr, uint64_t value, unsigned size) "cpu %d mr %p addr %#"PRIx64" value %#"PRIx64" size %u"
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Qemu-devel] [PATCH v0 3/6] move tcg header file
2017-05-19 7:30 [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 1/6] move tcg relative " Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 2/6] " Yang Zhong
@ 2017-05-19 7:30 ` Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 4/6] move tcg relative files into tcg directory Yang Zhong
` (3 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Yang Zhong @ 2017-05-19 7:30 UTC (permalink / raw)
To: pbonzini; +Cc: qemu-devel, anthony.xu, Yang Zhong
move translate-all.h to ./tcg
Signed-off-by: Yang Zhong <yang.zhong@intel.com>
---
translate-all.h => tcg/translate-all.h | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename translate-all.h => tcg/translate-all.h (100%)
diff --git a/translate-all.h b/tcg/translate-all.h
similarity index 100%
rename from translate-all.h
rename to tcg/translate-all.h
--
1.9.1
^ permalink raw reply [flat|nested] 9+ messages in thread* [Qemu-devel] [PATCH v0 4/6] move tcg relative files into tcg directory
2017-05-19 7:30 [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Yang Zhong
` (2 preceding siblings ...)
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 3/6] move tcg header file Yang Zhong
@ 2017-05-19 7:30 ` Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 5/6] " Yang Zhong
` (2 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Yang Zhong @ 2017-05-19 7:30 UTC (permalink / raw)
To: pbonzini; +Cc: qemu-devel, anthony.xu, Yang Zhong
move translate-common.c to ./tcg
Signed-off-by: Yang Zhong <yang.zhong@intel.com>
---
Makefile.target | 2 +-
translate-common.c => tcg/translate-common.c | 0
2 files changed, 1 insertion(+), 1 deletion(-)
rename translate-common.c => tcg/translate-common.c (100%)
diff --git a/Makefile.target b/Makefile.target
index c96ebd2..7ee6cb3 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -90,7 +90,7 @@ all: $(PROGS) stap
# cpu emulator library
obj-y = exec.o cpu-exec.o
obj-y += tcg/translate-all.o
-obj-y += translate-common.o
+obj-y += tcg/translate-common.o
obj-y += cpu-exec-common.o
obj-y += tcg/tcg.o tcg/tcg-op.o tcg/optimize.o
obj-$(CONFIG_TCG_INTERPRETER) += tci.o
diff --git a/translate-common.c b/tcg/translate-common.c
similarity index 100%
rename from translate-common.c
rename to tcg/translate-common.c
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Qemu-devel] [PATCH v0 5/6] move tcg relative files into tcg directory
2017-05-19 7:30 [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Yang Zhong
` (3 preceding siblings ...)
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 4/6] move tcg relative files into tcg directory Yang Zhong
@ 2017-05-19 7:30 ` Yang Zhong
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 6/6] change tcg relative file's compile definition Yang Zhong
2017-05-19 22:14 ` [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Eric Blake
6 siblings, 0 replies; 9+ messages in thread
From: Yang Zhong @ 2017-05-19 7:30 UTC (permalink / raw)
To: pbonzini; +Cc: qemu-devel, anthony.xu, Yang Zhong
move tci.c to ./tcg
Signed-off-by: Yang Zhong <yang.zhong@intel.com>
---
Makefile.target | 2 +-
tci.c => tcg/tci.c | 0
2 files changed, 1 insertion(+), 1 deletion(-)
rename tci.c => tcg/tci.c (100%)
diff --git a/Makefile.target b/Makefile.target
index 7ee6cb3..489a325 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -93,7 +93,7 @@ obj-y += tcg/translate-all.o
obj-y += tcg/translate-common.o
obj-y += cpu-exec-common.o
obj-y += tcg/tcg.o tcg/tcg-op.o tcg/optimize.o
-obj-$(CONFIG_TCG_INTERPRETER) += tci.o
+obj-$(CONFIG_TCG_INTERPRETER) += tcg/tci.o
obj-y += tcg/tcg-common.o
obj-$(CONFIG_TCG_INTERPRETER) += disas/tci.o
obj-y += fpu/softfloat.o
diff --git a/tci.c b/tcg/tci.c
similarity index 100%
rename from tci.c
rename to tcg/tci.c
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Qemu-devel] [PATCH v0 6/6] change tcg relative file's compile definition
2017-05-19 7:30 [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Yang Zhong
` (4 preceding siblings ...)
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 5/6] " Yang Zhong
@ 2017-05-19 7:30 ` Yang Zhong
2017-05-19 22:14 ` [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Eric Blake
6 siblings, 0 replies; 9+ messages in thread
From: Yang Zhong @ 2017-05-19 7:30 UTC (permalink / raw)
To: pbonzini; +Cc: qemu-devel, anthony.xu, Yang Zhong
move tcg relative compile definitions form Makefile.target
to tcg/Makefile.objs
Signed-off-by: Yang Zhong <yang.zhong@intel.com>
---
Makefile.target | 7 +------
tcg/Makefile.objs | 2 ++
2 files changed, 3 insertions(+), 6 deletions(-)
create mode 100644 tcg/Makefile.objs
diff --git a/Makefile.target b/Makefile.target
index 489a325..c0272b4 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -89,17 +89,12 @@ all: $(PROGS) stap
#########################################################
# cpu emulator library
obj-y = exec.o cpu-exec.o
-obj-y += tcg/translate-all.o
-obj-y += tcg/translate-common.o
obj-y += cpu-exec-common.o
-obj-y += tcg/tcg.o tcg/tcg-op.o tcg/optimize.o
-obj-$(CONFIG_TCG_INTERPRETER) += tcg/tci.o
-obj-y += tcg/tcg-common.o
+obj-y += tcg/
obj-$(CONFIG_TCG_INTERPRETER) += disas/tci.o
obj-y += fpu/softfloat.o
obj-y += target/$(TARGET_BASE_ARCH)/
obj-y += disas.o
-obj-y += tcg/tcg-runtime.o
obj-$(call notempty,$(TARGET_XML_FILES)) += gdbstub-xml.o
obj-$(call lnot,$(CONFIG_HAX)) += hax-stub.o
obj-$(call lnot,$(CONFIG_KVM)) += kvm-stub.o
diff --git a/tcg/Makefile.objs b/tcg/Makefile.objs
new file mode 100644
index 0000000..f4dcc3b
--- /dev/null
+++ b/tcg/Makefile.objs
@@ -0,0 +1,2 @@
+obj-y += translate-all.o translate-common.o tcg.o tcg-op.o optimize.o tcg-common.o tcg-runtime.o
+obj-$(CONFIG_TCG_INTERPRETER) += tci.o
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory.
2017-05-19 7:30 [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Yang Zhong
` (5 preceding siblings ...)
2017-05-19 7:30 ` [Qemu-devel] [PATCH v0 6/6] change tcg relative file's compile definition Yang Zhong
@ 2017-05-19 22:14 ` Eric Blake
2017-05-22 17:31 ` Zhong Yang
6 siblings, 1 reply; 9+ messages in thread
From: Eric Blake @ 2017-05-19 22:14 UTC (permalink / raw)
To: Yang Zhong, pbonzini; +Cc: anthony.xu, qemu-devel
[-- Attachment #1: Type: text/plain, Size: 2544 bytes --]
On 05/19/2017 02:30 AM, Yang Zhong wrote:
> Move the tcg relative files into tcg directory, which will make
> the code more clean in qemu.
Titling a patch series v0 is a bit unusual (typically, the first version
is untitled, and the second version is titled v2; 'git send-email -v2'
can help). But since 0 is less than 1 or 2, I don't think it breaks any
tools if you use that as your explicit title for a first submission,
even if you have to respin.
>
> Yang Zhong (6):
> move tcg relative files into tcg directory
> move tcg relative files into tcg directory
> move tcg header file
> move tcg relative files into tcg directory
> move tcg relative files into tcg directory
None of your patches match the usual "topic: Short description" of other
patches. Furthermore, having identical titles on 4 out of 6 distinct
patches is a nightmare for downstream backporters (which "move tcg
relative files into tcg directory" do I have to backport to fix the bug,
again?). I highly suggest that every patch you submit have enough
details in the subject line that the subject is distinct (we don't
always succeed, but it's usually quite easy to avoid duplicates).
So, as an example, I might title a patch:
"tcg: Move tcg-runtime.c to tcg/ subdirectory"
> change tcg relative file's compile definition
>
> Makefile.objs | 1 +
> Makefile.target | 8 ++------
> tcg/Makefile.objs | 2 ++
> tcg-runtime.c => tcg/tcg-runtime.c | 0
> tci.c => tcg/tci.c | 0
> tcg/trace-events | 6 ++++++
> translate-all.c => tcg/translate-all.c | 2 +-
> translate-all.h => tcg/translate-all.h | 0
> translate-common.c => tcg/translate-common.c | 0
> trace-events | 3 ---
Thankfully, you've got git rename detection turned on, which makes for
much nicer reviews.
> 10 files changed, 12 insertions(+), 10 deletions(-)
> create mode 100644 tcg/Makefile.objs
> rename tcg-runtime.c => tcg/tcg-runtime.c (100%)
> rename tci.c => tcg/tci.c (100%)
> create mode 100644 tcg/trace-events
> rename translate-all.c => tcg/translate-all.c (99%)
> rename translate-all.h => tcg/translate-all.h (100%)
> rename translate-common.c => tcg/translate-common.c (100%)
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory.
2017-05-19 22:14 ` [Qemu-devel] [PATCH v0 0/6] move the tcg files into tcg directory Eric Blake
@ 2017-05-22 17:31 ` Zhong Yang
0 siblings, 0 replies; 9+ messages in thread
From: Zhong Yang @ 2017-05-22 17:31 UTC (permalink / raw)
To: Eric Blake; +Cc: pbonzini, anthony.xu, qemu-devel
O Fri, May 19, 2017 at 05:14:19PM -0500, Eric Blake wrote:
> On 05/19/2017 02:30 AM, Yang Zhong wrote:
> > Move the tcg relative files into tcg directory, which will make
> > the code more clean in qemu.
>
> Titling a patch series v0 is a bit unusual (typically, the first version
> is untitled, and the second version is titled v2; 'git send-email -v2'
> can help). But since 0 is less than 1 or 2, I don't think it breaks any
> tools if you use that as your explicit title for a first submission,
> even if you have to respin.
>
> >
> > Yang Zhong (6):
> > move tcg relative files into tcg directory
> > move tcg relative files into tcg directory
> > move tcg header file
> > move tcg relative files into tcg directory
> > move tcg relative files into tcg directory
>
> None of your patches match the usual "topic: Short description" of other
> patches. Furthermore, having identical titles on 4 out of 6 distinct
> patches is a nightmare for downstream backporters (which "move tcg
> relative files into tcg directory" do I have to backport to fix the bug,
> again?). I highly suggest that every patch you submit have enough
> details in the subject line that the subject is distinct (we don't
> always succeed, but it's usually quite easy to avoid duplicates).
>
> So, as an example, I might title a patch:
>
> "tcg: Move tcg-runtime.c to tcg/ subdirectory"
>
Thanks for your comments, and i will send v2 patchset to commnunity accroding to your suggestions.
Please also help review my tcg accelator patchset,many thanks!
> > change tcg relative file's compile definition
> >
> > Makefile.objs | 1 +
> > Makefile.target | 8 ++------
> > tcg/Makefile.objs | 2 ++
> > tcg-runtime.c => tcg/tcg-runtime.c | 0
> > tci.c => tcg/tci.c | 0
> > tcg/trace-events | 6 ++++++
> > translate-all.c => tcg/translate-all.c | 2 +-
> > translate-all.h => tcg/translate-all.h | 0
> > translate-common.c => tcg/translate-common.c | 0
> > trace-events | 3 ---
>
> Thankfully, you've got git rename detection turned on, which makes for
> much nicer reviews.
>
> > 10 files changed, 12 insertions(+), 10 deletions(-)
> > create mode 100644 tcg/Makefile.objs
> > rename tcg-runtime.c => tcg/tcg-runtime.c (100%)
> > rename tci.c => tcg/tci.c (100%)
> > create mode 100644 tcg/trace-events
> > rename translate-all.c => tcg/translate-all.c (99%)
> > rename translate-all.h => tcg/translate-all.h (100%)
> > rename translate-common.c => tcg/translate-common.c (100%)
> >
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3266
> Virtualization: qemu.org | libvirt.org
>
^ permalink raw reply [flat|nested] 9+ messages in thread