* [Qemu-devel] [PATCH v2 1/6] tcg: move tcg-runtime.c to tcg/ subdirectory
2017-05-31 6:28 [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Yang Zhong
@ 2017-05-31 6:28 ` Yang Zhong
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 2/6] tcg: move translate-all.c " Yang Zhong
` (5 subsequent siblings)
6 siblings, 0 replies; 13+ messages in thread
From: Yang Zhong @ 2017-05-31 6:28 UTC (permalink / raw)
To: eblake; +Cc: qemu-devel, pbonzini, anthony.xu, Yang Zhong
tcg: move tcg-runtime.c to tcg/ subdirectory
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 ce8dfe4..3ba2074 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] 13+ messages in thread* [Qemu-devel] [PATCH v2 2/6] tcg: move translate-all.c to tcg/ subdirectory
2017-05-31 6:28 [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Yang Zhong
2017-05-31 6:28 ` [Qemu-devel] [PATCH v2 1/6] tcg: move tcg-runtime.c to " Yang Zhong
@ 2017-05-31 6:29 ` Yang Zhong
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 3/6] tcg: move tcg header file Yang Zhong
` (4 subsequent siblings)
6 siblings, 0 replies; 13+ messages in thread
From: Yang Zhong @ 2017-05-31 6:29 UTC (permalink / raw)
To: eblake; +Cc: qemu-devel, pbonzini, anthony.xu, Yang Zhong
tcg: move translate-all.c to tcg/ subdirectory and
also update related 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 3ba2074..5d33174 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 433865f..a25db67 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] 13+ messages in thread* [Qemu-devel] [PATCH v2 3/6] tcg: move tcg header file
2017-05-31 6:28 [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Yang Zhong
2017-05-31 6:28 ` [Qemu-devel] [PATCH v2 1/6] tcg: move tcg-runtime.c to " Yang Zhong
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 2/6] tcg: move translate-all.c " Yang Zhong
@ 2017-05-31 6:29 ` Yang Zhong
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 4/6] tcg: move translate-common.c to tcg/ subdirectory Yang Zhong
` (3 subsequent siblings)
6 siblings, 0 replies; 13+ messages in thread
From: Yang Zhong @ 2017-05-31 6:29 UTC (permalink / raw)
To: eblake; +Cc: qemu-devel, pbonzini, anthony.xu, Yang Zhong
tcg: move translate-all.h to tcg/ subdirectory
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] 13+ messages in thread* [Qemu-devel] [PATCH v2 4/6] tcg: move translate-common.c to tcg/ subdirectory
2017-05-31 6:28 [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Yang Zhong
` (2 preceding siblings ...)
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 3/6] tcg: move tcg header file Yang Zhong
@ 2017-05-31 6:29 ` Yang Zhong
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 5/6] tcg: move tci.c " Yang Zhong
` (2 subsequent siblings)
6 siblings, 0 replies; 13+ messages in thread
From: Yang Zhong @ 2017-05-31 6:29 UTC (permalink / raw)
To: eblake; +Cc: qemu-devel, pbonzini, anthony.xu, Yang Zhong
tcg: move translate-common.c to tcg/ subdirectory
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 5d33174..648e61e 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] 13+ messages in thread* [Qemu-devel] [PATCH v2 5/6] tcg: move tci.c to tcg/ subdirectory
2017-05-31 6:28 [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Yang Zhong
` (3 preceding siblings ...)
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 4/6] tcg: move translate-common.c to tcg/ subdirectory Yang Zhong
@ 2017-05-31 6:29 ` Yang Zhong
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 6/6] tcg: change tcg related file's compile definition Yang Zhong
2017-05-31 12:54 ` [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Paolo Bonzini
6 siblings, 0 replies; 13+ messages in thread
From: Yang Zhong @ 2017-05-31 6:29 UTC (permalink / raw)
To: eblake; +Cc: qemu-devel, pbonzini, anthony.xu, Yang Zhong
tcg: move tci.c to tcg/ subdirectory
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 648e61e..8561132 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] 13+ messages in thread* [Qemu-devel] [PATCH v2 6/6] tcg: change tcg related file's compile definition
2017-05-31 6:28 [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Yang Zhong
` (4 preceding siblings ...)
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 5/6] tcg: move tci.c " Yang Zhong
@ 2017-05-31 6:29 ` Yang Zhong
2017-05-31 12:54 ` [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Paolo Bonzini
6 siblings, 0 replies; 13+ messages in thread
From: Yang Zhong @ 2017-05-31 6:29 UTC (permalink / raw)
To: eblake; +Cc: qemu-devel, pbonzini, anthony.xu, Yang Zhong
tcg: move tcg related 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 8561132..956aeff 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] 13+ messages in thread* Re: [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory.
2017-05-31 6:28 [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Yang Zhong
` (5 preceding siblings ...)
2017-05-31 6:29 ` [Qemu-devel] [PATCH v2 6/6] tcg: change tcg related file's compile definition Yang Zhong
@ 2017-05-31 12:54 ` Paolo Bonzini
2017-05-31 16:24 ` Laurent Desnogues
2017-06-01 5:27 ` Zhong Yang
6 siblings, 2 replies; 13+ messages in thread
From: Paolo Bonzini @ 2017-05-31 12:54 UTC (permalink / raw)
To: Yang Zhong, eblake; +Cc: qemu-devel, anthony.xu
On 31/05/2017 08:28, Yang Zhong wrote:
> Move the tcg related files into tcg/ subdirectory, which will make
> the code more clean in qemu. Next step, we will base on those changes
> to disable tcg.
You should also move cpu-exec.c, cpu-exec-common.c and cputlb.c.
Also, feel free to squash everything in a single patch.
Thanks,
Paolo
> Yang Zhong (6):
> tcg: move tcg-runtime.c to tcg/ subdirectory
> tcg: move translate-all.c to tcg/ subdirectory
> tcg: move tcg header file
> tcg: move translate-common.c to tcg/ subdirectory
> tcg: move tci.c to tcg/ subdirectory
> tcg: change tcg related 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 ---
> 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%)
>
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory.
2017-05-31 12:54 ` [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Paolo Bonzini
@ 2017-05-31 16:24 ` Laurent Desnogues
2017-05-31 19:05 ` Paolo Bonzini
2017-06-01 5:27 ` Zhong Yang
1 sibling, 1 reply; 13+ messages in thread
From: Laurent Desnogues @ 2017-05-31 16:24 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: Yang Zhong, Eric Blake, anthony.xu, qemu-devel@nongnu.org
Hello,
On Wed, May 31, 2017 at 2:54 PM, Paolo Bonzini <pbonzini@redhat.com> wrote:
>
>
> On 31/05/2017 08:28, Yang Zhong wrote:
>> Move the tcg related files into tcg/ subdirectory, which will make
>> the code more clean in qemu. Next step, we will base on those changes
>> to disable tcg.
>
> You should also move cpu-exec.c, cpu-exec-common.c and cputlb.c.
The LICENSE file in tcg directory contains this:
---------------------------------------------------------------------
All the files in this directory and subdirectories are released under
a BSD like license (see header in each file). No other license is
accepted.
---------------------------------------------------------------------
All source files have an explicit BSD license except tcg-runtime.h
which has none and aarch64 files which are licensed under GPL. So we
are already not following the rule expressed in LICENSE.
tcg-runtime.c is BSD
tci.c is GPL
translate-all.[ch] are GPL
translate-common.c is GPL
cpu-exec.c cpu-exec-common.c cputlb.c are GPL
I guess the LICENSE file has become obsolete and so should be removed
(and Fabrice Bellard should perhaps be let know about that just in
case).
Thanks,
Laurent
> Also, feel free to squash everything in a single patch.
> Thanks,
>
> Paolo
>
>> Yang Zhong (6):
>> tcg: move tcg-runtime.c to tcg/ subdirectory
>> tcg: move translate-all.c to tcg/ subdirectory
>> tcg: move tcg header file
>> tcg: move translate-common.c to tcg/ subdirectory
>> tcg: move tci.c to tcg/ subdirectory
>> tcg: change tcg related 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 ---
>> 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%)
>>
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory.
2017-05-31 16:24 ` Laurent Desnogues
@ 2017-05-31 19:05 ` Paolo Bonzini
0 siblings, 0 replies; 13+ messages in thread
From: Paolo Bonzini @ 2017-05-31 19:05 UTC (permalink / raw)
To: Laurent Desnogues
Cc: Yang Zhong, Eric Blake, anthony.xu, qemu-devel@nongnu.org
On 31/05/2017 18:24, Laurent Desnogues wrote:
> All source files have an explicit BSD license except tcg-runtime.h
> which has none and aarch64 files which are licensed under GPL. So we
> are already not following the rule expressed in LICENSE.
>
> tcg-runtime.c is BSD
> tci.c is GPL
> translate-all.[ch] are GPL
> translate-common.c is GPL
> cpu-exec.c cpu-exec-common.c cputlb.c are GPL
>
> I guess the LICENSE file has become obsolete and so should be removed
> (and Fabrice Bellard should perhaps be let know about that just in
> case).
We can change it to "The TCG backend files", followed by a list of
files. But even then, aarch64 files are GPL...
Paolo
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory.
2017-05-31 12:54 ` [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory Paolo Bonzini
2017-05-31 16:24 ` Laurent Desnogues
@ 2017-06-01 5:27 ` Zhong Yang
2017-06-01 8:14 ` Paolo Bonzini
1 sibling, 1 reply; 13+ messages in thread
From: Zhong Yang @ 2017-06-01 5:27 UTC (permalink / raw)
To: Paolo Bonzini
Cc: Eric Blake, Laurent Desnogues, anthony.xu, qemu-devel@nongnu.org
Wed, May 31, 2017 at 02:54:05PM +0200, Paolo Bonzini wrote:
>
>
> On 31/05/2017 08:28, Yang Zhong wrote:
> > Move the tcg related files into tcg/ subdirectory, which will make
> > the code more clean in qemu. Next step, we will base on those changes
> > to disable tcg.
>
> You should also move cpu-exec.c, cpu-exec-common.c and cputlb.c.
>
In fact, there are other two patchsets for those files. Since this patchset
is only for moving TCG backend files into tcg/ subdirectory, i firstly upload
this patchset.
Let me explain my plan here,
In order to disable tcg, the first phase is to split some
tcg code and separate those tcg related files into one
directory. The next phase will disable tcg more easily and
cleanly.
In the first phase, there are three patchsets to
deal with split code and separate the files.
1) split the tcg exec code.
a)split the tcg accelerators from accel.c file and re-name
tcg accelerator to tcg-all.c, like kvm-all.c did.
b)create one accel directory, which will include kvm,
tcg and general exec files.
c)move tcg exec files into ./accel/tcg directory.
2) move tcg related backend files into tcg/ directory
those files include translate-all.(ch), translate-common.c,
tci.c and tcg-runtime.c.
3) move kvm exec and exec.c file.
a)move kvm related exec files into ./accel/kvm directory.
b)move exec.c in ./accel directory.
after 1) and 3) done, the file tree like below:
accel/
├── accel.c
├── exec.c
├── kvm
│ ├── kvm-all.c
│ ├── kvm-stub.c
│ ├── Makefile.objs
│ └── trace-events
├── Makefile.objs
└── tcg
├── cpu-exec.c
├── cpu-exec-common.c
├── cputlb.c
├── Makefile.objs
├── tcg-all.c
└── trace-events
please help me check whether this kind of arrange is suitable?
> Also, feel free to squash everything in a single patch.
for the tcg/ subdirectory, i will make those 6 patchset to one single patch. thanks!
> Thanks,
>
> Paolo
>
> > Yang Zhong (6):
> > tcg: move tcg-runtime.c to tcg/ subdirectory
> > tcg: move translate-all.c to tcg/ subdirectory
> > tcg: move tcg header file
> > tcg: move translate-common.c to tcg/ subdirectory
> > tcg: move tci.c to tcg/ subdirectory
> > tcg: change tcg related 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 ---
> > 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%)
> >
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory.
2017-06-01 5:27 ` Zhong Yang
@ 2017-06-01 8:14 ` Paolo Bonzini
2017-06-01 12:04 ` Zhong Yang
0 siblings, 1 reply; 13+ messages in thread
From: Paolo Bonzini @ 2017-06-01 8:14 UTC (permalink / raw)
To: Zhong Yang
Cc: Eric Blake, Laurent Desnogues, anthony.xu, qemu-devel@nongnu.org
On 01/06/2017 07:27, Zhong Yang wrote:
> In the first phase, there are three patchsets to
> deal with split code and separate the files.
> 1) split the tcg exec code.
> a)split the tcg accelerators from accel.c file and re-name
> tcg accelerator to tcg-all.c, like kvm-all.c did.
> b)create one accel directory, which will include kvm,
> tcg and general exec files.
> c)move tcg exec files into ./accel/tcg directory.
> 2) move tcg related backend files into tcg/ directory
> those files include translate-all.(ch), translate-common.c,
> tci.c and tcg-runtime.c.
> 3) move kvm exec and exec.c file.
> a)move kvm related exec files into ./accel/kvm directory.
> b)move exec.c in ./accel directory.
>
> after 1) and 3) done, the file tree like below:
> accel/
> ├── accel.c
> ├── exec.c
> ├── kvm
> │ ├── kvm-all.c
> │ ├── kvm-stub.c
> │ ├── Makefile.objs
> │ └── trace-events
> ├── Makefile.objs
> └── tcg
> ├── cpu-exec.c
> ├── cpu-exec-common.c
> ├── cputlb.c
> ├── Makefile.objs
> ├── tcg-all.c
> └── trace-events
>
> please help me check whether this kind of arrange is suitable?
Oh, I see now. I think translate-all.[ch] and translate-common.c belong
in accel/tcg more than tcg/.
Thanks,
Paolo
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/6] tcg: move the tcg files into tcg/ subdirectory.
2017-06-01 8:14 ` Paolo Bonzini
@ 2017-06-01 12:04 ` Zhong Yang
0 siblings, 0 replies; 13+ messages in thread
From: Zhong Yang @ 2017-06-01 12:04 UTC (permalink / raw)
To: Paolo Bonzini
Cc: Eric Blake, Laurent Desnogues, anthony.xu, qemu-devel@nongnu.org
On Thu, Jun 01, 2017 at 10:14:34AM +0200, Paolo Bonzini wrote:
>
>
> On 01/06/2017 07:27, Zhong Yang wrote:
> > In the first phase, there are three patchsets to
> > deal with split code and separate the files.
> > 1) split the tcg exec code.
> > a)split the tcg accelerators from accel.c file and re-name
> > tcg accelerator to tcg-all.c, like kvm-all.c did.
> > b)create one accel directory, which will include kvm,
> > tcg and general exec files.
> > c)move tcg exec files into ./accel/tcg directory.
> > 2) move tcg related backend files into tcg/ directory
> > those files include translate-all.(ch), translate-common.c,
> > tci.c and tcg-runtime.c.
> > 3) move kvm exec and exec.c file.
> > a)move kvm related exec files into ./accel/kvm directory.
> > b)move exec.c in ./accel directory.
> >
> > after 1) and 3) done, the file tree like below:
> > accel/
> > ├── accel.c
> > ├── exec.c
> > ├── kvm
> > │ ├── kvm-all.c
> > │ ├── kvm-stub.c
> > │ ├── Makefile.objs
> > │ └── trace-events
> > ├── Makefile.objs
> > └── tcg
> > ├── cpu-exec.c
> > ├── cpu-exec-common.c
> > ├── cputlb.c
> > ├── Makefile.objs
> > ├── tcg-all.c
> > └── trace-events
> >
> > please help me check whether this kind of arrange is suitable?
>
> Oh, I see now. I think translate-all.[ch] and translate-common.c belong
> in accel/tcg more than tcg/.
okay, i will make those changes into one patchset and send v3 to all, thanks!
Yang.
>
> Thanks,
>
> Paolo
^ permalink raw reply [flat|nested] 13+ messages in thread