* [Qemu-devel] [PATCH] target/m68k: add monitor.c
@ 2017-12-20 20:35 Laurent Vivier
2017-12-20 20:54 ` Philippe Mathieu-Daudé
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Laurent Vivier @ 2017-12-20 20:35 UTC (permalink / raw)
To: qemu-devel; +Cc: Thomas Huth, Philippe Mathieu-Daudé, Laurent Vivier
This allows to use registers content in the monitor.
Example:
BEFORE:
(qemu) print $d0
unknown register
AFTER:
(qemu) print $d0
0
(qemu) print $sr
0x2000
(qemu) x/10i $pc
0x40010a2a: movew %sr,%d0
0x40010a2c: oril #1792,%d0
0x40010a32: movew %d0,%sr
0x40010a34: movel %a0@,%d0
0x40010a36: btst #3,%d0
0x40010a3a: beqs 0x40010a26
0x40010a3c: movew %sr,%d0
0x40010a3e: andil #63743,%d0
0x40010a44: movew %d0,%sr
0x40010a46: rts
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
target/m68k/Makefile.objs | 1 +
target/m68k/monitor.c | 55 +++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 56 insertions(+)
create mode 100644 target/m68k/monitor.c
diff --git a/target/m68k/Makefile.objs b/target/m68k/Makefile.objs
index 39141ab93d..d143f20270 100644
--- a/target/m68k/Makefile.objs
+++ b/target/m68k/Makefile.objs
@@ -1,3 +1,4 @@
obj-y += m68k-semi.o
obj-y += translate.o op_helper.o helper.o cpu.o fpu_helper.o
obj-y += gdbstub.o
+obj-$(CONFIG_SOFTMMU) += monitor.o
diff --git a/target/m68k/monitor.c b/target/m68k/monitor.c
new file mode 100644
index 0000000000..03d037ccab
--- /dev/null
+++ b/target/m68k/monitor.c
@@ -0,0 +1,55 @@
+/*
+ * QEMU monitor
+ *
+ * Copyright (c) 2003-2004 Fabrice Bellard
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+#include "qemu/osdep.h"
+#include "cpu.h"
+#include "monitor/hmp-target.h"
+
+const MonitorDef monitor_defs[] = {
+ { "d0", offsetof(CPUM68KState, dregs[0]) },
+ { "d1", offsetof(CPUM68KState, dregs[1]) },
+ { "d2", offsetof(CPUM68KState, dregs[2]) },
+ { "d3", offsetof(CPUM68KState, dregs[3]) },
+ { "d4", offsetof(CPUM68KState, dregs[4]) },
+ { "d5", offsetof(CPUM68KState, dregs[5]) },
+ { "d6", offsetof(CPUM68KState, dregs[6]) },
+ { "d7", offsetof(CPUM68KState, dregs[7]) },
+ { "a0", offsetof(CPUM68KState, aregs[0]) },
+ { "a1", offsetof(CPUM68KState, aregs[1]) },
+ { "a2", offsetof(CPUM68KState, aregs[2]) },
+ { "a3", offsetof(CPUM68KState, aregs[3]) },
+ { "a4", offsetof(CPUM68KState, aregs[4]) },
+ { "a5", offsetof(CPUM68KState, aregs[5]) },
+ { "a6", offsetof(CPUM68KState, aregs[6]) },
+ { "a7", offsetof(CPUM68KState, aregs[7]) },
+ { "pc", offsetof(CPUM68KState, pc) },
+ { "sr", offsetof(CPUM68KState, sr) },
+ { "ssp", offsetof(CPUM68KState, sp[0]) },
+ { "usp", offsetof(CPUM68KState, sp[1]) },
+ { NULL },
+};
+
+const MonitorDef *target_monitor_defs(void)
+{
+ return monitor_defs;
+}
--
2.14.3
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] target/m68k: add monitor.c
2017-12-20 20:35 [Qemu-devel] [PATCH] target/m68k: add monitor.c Laurent Vivier
@ 2017-12-20 20:54 ` Philippe Mathieu-Daudé
2017-12-21 7:57 ` Laurent Vivier
2017-12-20 21:01 ` Richard Henderson
2017-12-20 21:22 ` Thomas Huth
2 siblings, 1 reply; 7+ messages in thread
From: Philippe Mathieu-Daudé @ 2017-12-20 20:54 UTC (permalink / raw)
To: Laurent Vivier; +Cc: qemu-devel@nongnu.org Developers, Thomas Huth
On Wed, Dec 20, 2017 at 5:35 PM, Laurent Vivier <laurent@vivier.eu> wrote:
> This allows to use registers content in the monitor.
>
> Example:
>
> BEFORE:
> (qemu) print $d0
> unknown register
>
> AFTER:
> (qemu) print $d0
> 0
> (qemu) print $sr
> 0x2000
> (qemu) x/10i $pc
> 0x40010a2a: movew %sr,%d0
> 0x40010a2c: oril #1792,%d0
> 0x40010a32: movew %d0,%sr
> 0x40010a34: movel %a0@,%d0
> 0x40010a36: btst #3,%d0
> 0x40010a3a: beqs 0x40010a26
> 0x40010a3c: movew %sr,%d0
> 0x40010a3e: andil #63743,%d0
> 0x40010a44: movew %d0,%sr
> 0x40010a46: rts
>
> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
> ---
> target/m68k/Makefile.objs | 1 +
> target/m68k/monitor.c | 55 +++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 56 insertions(+)
> create mode 100644 target/m68k/monitor.c
>
> diff --git a/target/m68k/Makefile.objs b/target/m68k/Makefile.objs
> index 39141ab93d..d143f20270 100644
> --- a/target/m68k/Makefile.objs
> +++ b/target/m68k/Makefile.objs
> @@ -1,3 +1,4 @@
> obj-y += m68k-semi.o
> obj-y += translate.o op_helper.o helper.o cpu.o fpu_helper.o
> obj-y += gdbstub.o
> +obj-$(CONFIG_SOFTMMU) += monitor.o
> diff --git a/target/m68k/monitor.c b/target/m68k/monitor.c
> new file mode 100644
> index 0000000000..03d037ccab
> --- /dev/null
> +++ b/target/m68k/monitor.c
> @@ -0,0 +1,55 @@
> +/*
> + * QEMU monitor
> + *
> + * Copyright (c) 2003-2004 Fabrice Bellard
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a copy
> + * of this software and associated documentation files (the "Software"), to deal
> + * in the Software without restriction, including without limitation the rights
> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
> + * copies of the Software, and to permit persons to whom the Software is
> + * furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
> + * THE SOFTWARE.
> + */
> +#include "qemu/osdep.h"
> +#include "cpu.h"
> +#include "monitor/hmp-target.h"
> +
> +const MonitorDef monitor_defs[] = {
adding 'static':
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> + { "d0", offsetof(CPUM68KState, dregs[0]) },
> + { "d1", offsetof(CPUM68KState, dregs[1]) },
> + { "d2", offsetof(CPUM68KState, dregs[2]) },
> + { "d3", offsetof(CPUM68KState, dregs[3]) },
> + { "d4", offsetof(CPUM68KState, dregs[4]) },
> + { "d5", offsetof(CPUM68KState, dregs[5]) },
> + { "d6", offsetof(CPUM68KState, dregs[6]) },
> + { "d7", offsetof(CPUM68KState, dregs[7]) },
> + { "a0", offsetof(CPUM68KState, aregs[0]) },
> + { "a1", offsetof(CPUM68KState, aregs[1]) },
> + { "a2", offsetof(CPUM68KState, aregs[2]) },
> + { "a3", offsetof(CPUM68KState, aregs[3]) },
> + { "a4", offsetof(CPUM68KState, aregs[4]) },
> + { "a5", offsetof(CPUM68KState, aregs[5]) },
> + { "a6", offsetof(CPUM68KState, aregs[6]) },
> + { "a7", offsetof(CPUM68KState, aregs[7]) },
> + { "pc", offsetof(CPUM68KState, pc) },
> + { "sr", offsetof(CPUM68KState, sr) },
> + { "ssp", offsetof(CPUM68KState, sp[0]) },
> + { "usp", offsetof(CPUM68KState, sp[1]) },
> + { NULL },
> +};
> +
> +const MonitorDef *target_monitor_defs(void)
> +{
> + return monitor_defs;
> +}
> --
> 2.14.3
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] target/m68k: add monitor.c
2017-12-20 20:35 [Qemu-devel] [PATCH] target/m68k: add monitor.c Laurent Vivier
2017-12-20 20:54 ` Philippe Mathieu-Daudé
@ 2017-12-20 21:01 ` Richard Henderson
2017-12-20 21:22 ` Thomas Huth
2 siblings, 0 replies; 7+ messages in thread
From: Richard Henderson @ 2017-12-20 21:01 UTC (permalink / raw)
To: Laurent Vivier, qemu-devel; +Cc: Thomas Huth, Philippe Mathieu-Daudé
On 12/20/2017 12:35 PM, Laurent Vivier wrote:
> +const MonitorDef monitor_defs[] = {
static. Otherwise,
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] target/m68k: add monitor.c
2017-12-20 20:35 [Qemu-devel] [PATCH] target/m68k: add monitor.c Laurent Vivier
2017-12-20 20:54 ` Philippe Mathieu-Daudé
2017-12-20 21:01 ` Richard Henderson
@ 2017-12-20 21:22 ` Thomas Huth
2017-12-21 7:56 ` Laurent Vivier
2 siblings, 1 reply; 7+ messages in thread
From: Thomas Huth @ 2017-12-20 21:22 UTC (permalink / raw)
To: Laurent Vivier; +Cc: qemu-devel, Philippe Mathieu-Daudé
Am Wed, 20 Dec 2017 21:35:28 +0100
schrieb Laurent Vivier <laurent@vivier.eu>:
> This allows to use registers content in the monitor.
>
> Example:
>
> BEFORE:
> (qemu) print $d0
> unknown register
>
> AFTER:
> (qemu) print $d0
> 0
> (qemu) print $sr
> 0x2000
> (qemu) x/10i $pc
> 0x40010a2a: movew %sr,%d0
> 0x40010a2c: oril #1792,%d0
> 0x40010a32: movew %d0,%sr
> 0x40010a34: movel %a0@,%d0
> 0x40010a36: btst #3,%d0
> 0x40010a3a: beqs 0x40010a26
> 0x40010a3c: movew %sr,%d0
> 0x40010a3e: andil #63743,%d0
> 0x40010a44: movew %d0,%sr
> 0x40010a46: rts
>
> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
> ---
> target/m68k/Makefile.objs | 1 +
> target/m68k/monitor.c | 55
> +++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 56
> insertions(+) create mode 100644 target/m68k/monitor.c
>
> diff --git a/target/m68k/Makefile.objs b/target/m68k/Makefile.objs
> index 39141ab93d..d143f20270 100644
> --- a/target/m68k/Makefile.objs
> +++ b/target/m68k/Makefile.objs
> @@ -1,3 +1,4 @@
> obj-y += m68k-semi.o
> obj-y += translate.o op_helper.o helper.o cpu.o fpu_helper.o
> obj-y += gdbstub.o
> +obj-$(CONFIG_SOFTMMU) += monitor.o
> diff --git a/target/m68k/monitor.c b/target/m68k/monitor.c
> new file mode 100644
> index 0000000000..03d037ccab
> --- /dev/null
> +++ b/target/m68k/monitor.c
> @@ -0,0 +1,55 @@
> +/*
> + * QEMU monitor
> + *
> + * Copyright (c) 2003-2004 Fabrice Bellard
Why this copyright line (and the non-GPL license code)?
Apart from that, the patch looks fine to me.
Thomas
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] target/m68k: add monitor.c
2017-12-20 21:22 ` Thomas Huth
@ 2017-12-21 7:56 ` Laurent Vivier
2017-12-21 8:04 ` Thomas Huth
0 siblings, 1 reply; 7+ messages in thread
From: Laurent Vivier @ 2017-12-21 7:56 UTC (permalink / raw)
To: Thomas Huth; +Cc: qemu-devel, Philippe Mathieu-Daudé
Le 20/12/2017 à 22:22, Thomas Huth a écrit :
> Am Wed, 20 Dec 2017 21:35:28 +0100
> schrieb Laurent Vivier <laurent@vivier.eu>:
>
>> This allows to use registers content in the monitor.
>>
>> Example:
>>
>> BEFORE:
>> (qemu) print $d0
>> unknown register
>>
>> AFTER:
>> (qemu) print $d0
>> 0
>> (qemu) print $sr
>> 0x2000
>> (qemu) x/10i $pc
>> 0x40010a2a: movew %sr,%d0
>> 0x40010a2c: oril #1792,%d0
>> 0x40010a32: movew %d0,%sr
>> 0x40010a34: movel %a0@,%d0
>> 0x40010a36: btst #3,%d0
>> 0x40010a3a: beqs 0x40010a26
>> 0x40010a3c: movew %sr,%d0
>> 0x40010a3e: andil #63743,%d0
>> 0x40010a44: movew %d0,%sr
>> 0x40010a46: rts
>>
>> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
>> ---
>> target/m68k/Makefile.objs | 1 +
>> target/m68k/monitor.c | 55
>> +++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 56
>> insertions(+) create mode 100644 target/m68k/monitor.c
>>
>> diff --git a/target/m68k/Makefile.objs b/target/m68k/Makefile.objs
>> index 39141ab93d..d143f20270 100644
>> --- a/target/m68k/Makefile.objs
>> +++ b/target/m68k/Makefile.objs
>> @@ -1,3 +1,4 @@
>> obj-y += m68k-semi.o
>> obj-y += translate.o op_helper.o helper.o cpu.o fpu_helper.o
>> obj-y += gdbstub.o
>> +obj-$(CONFIG_SOFTMMU) += monitor.o
>> diff --git a/target/m68k/monitor.c b/target/m68k/monitor.c
>> new file mode 100644
>> index 0000000000..03d037ccab
>> --- /dev/null
>> +++ b/target/m68k/monitor.c
>> @@ -0,0 +1,55 @@
>> +/*
>> + * QEMU monitor
>> + *
>> + * Copyright (c) 2003-2004 Fabrice Bellard
>
> Why this copyright line (and the non-GPL license code)?
I don't know. I've checked all the others monitor.c and they have the
same header. An y suggestions?
Thanks,
Laurent
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] target/m68k: add monitor.c
2017-12-20 20:54 ` Philippe Mathieu-Daudé
@ 2017-12-21 7:57 ` Laurent Vivier
0 siblings, 0 replies; 7+ messages in thread
From: Laurent Vivier @ 2017-12-21 7:57 UTC (permalink / raw)
To: Philippe Mathieu-Daudé; +Cc: qemu-devel@nongnu.org Developers, Thomas Huth
Le 20/12/2017 à 21:54, Philippe Mathieu-Daudé a écrit :
> On Wed, Dec 20, 2017 at 5:35 PM, Laurent Vivier <laurent@vivier.eu> wrote:
>> This allows to use registers content in the monitor.
>>
>> Example:
>>
>> BEFORE:
>> (qemu) print $d0
>> unknown register
>>
>> AFTER:
>> (qemu) print $d0
>> 0
>> (qemu) print $sr
>> 0x2000
>> (qemu) x/10i $pc
>> 0x40010a2a: movew %sr,%d0
>> 0x40010a2c: oril #1792,%d0
>> 0x40010a32: movew %d0,%sr
>> 0x40010a34: movel %a0@,%d0
>> 0x40010a36: btst #3,%d0
>> 0x40010a3a: beqs 0x40010a26
>> 0x40010a3c: movew %sr,%d0
>> 0x40010a3e: andil #63743,%d0
>> 0x40010a44: movew %d0,%sr
>> 0x40010a46: rts
>>
>> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
>> ---
>> target/m68k/Makefile.objs | 1 +
>> target/m68k/monitor.c | 55 +++++++++++++++++++++++++++++++++++++++++++++++
>> 2 files changed, 56 insertions(+)
>> create mode 100644 target/m68k/monitor.c
>>
>> diff --git a/target/m68k/Makefile.objs b/target/m68k/Makefile.objs
>> index 39141ab93d..d143f20270 100644
>> --- a/target/m68k/Makefile.objs
>> +++ b/target/m68k/Makefile.objs
>> @@ -1,3 +1,4 @@
>> obj-y += m68k-semi.o
>> obj-y += translate.o op_helper.o helper.o cpu.o fpu_helper.o
>> obj-y += gdbstub.o
>> +obj-$(CONFIG_SOFTMMU) += monitor.o
>> diff --git a/target/m68k/monitor.c b/target/m68k/monitor.c
>> new file mode 100644
>> index 0000000000..03d037ccab
>> --- /dev/null
>> +++ b/target/m68k/monitor.c
>> @@ -0,0 +1,55 @@
>> +/*
>> + * QEMU monitor
>> + *
>> + * Copyright (c) 2003-2004 Fabrice Bellard
>> + *
>> + * Permission is hereby granted, free of charge, to any person obtaining a copy
>> + * of this software and associated documentation files (the "Software"), to deal
>> + * in the Software without restriction, including without limitation the rights
>> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
>> + * copies of the Software, and to permit persons to whom the Software is
>> + * furnished to do so, subject to the following conditions:
>> + *
>> + * The above copyright notice and this permission notice shall be included in
>> + * all copies or substantial portions of the Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
>> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
>> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
>> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
>> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
>> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
>> + * THE SOFTWARE.
>> + */
>> +#include "qemu/osdep.h"
>> +#include "cpu.h"
>> +#include "monitor/hmp-target.h"
>> +
>> +const MonitorDef monitor_defs[] = {
>
> adding 'static':
ok
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Thanks,
Laurent
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] target/m68k: add monitor.c
2017-12-21 7:56 ` Laurent Vivier
@ 2017-12-21 8:04 ` Thomas Huth
0 siblings, 0 replies; 7+ messages in thread
From: Thomas Huth @ 2017-12-21 8:04 UTC (permalink / raw)
To: qemu-devel
On 21.12.2017 08:56, Laurent Vivier wrote:
> Le 20/12/2017 à 22:22, Thomas Huth a écrit :
>> Am Wed, 20 Dec 2017 21:35:28 +0100
>> schrieb Laurent Vivier <laurent@vivier.eu>:
>>
>>> This allows to use registers content in the monitor.
>>>
>>> Example:
>>>
>>> BEFORE:
>>> (qemu) print $d0
>>> unknown register
>>>
>>> AFTER:
>>> (qemu) print $d0
>>> 0
>>> (qemu) print $sr
>>> 0x2000
>>> (qemu) x/10i $pc
>>> 0x40010a2a: movew %sr,%d0
>>> 0x40010a2c: oril #1792,%d0
>>> 0x40010a32: movew %d0,%sr
>>> 0x40010a34: movel %a0@,%d0
>>> 0x40010a36: btst #3,%d0
>>> 0x40010a3a: beqs 0x40010a26
>>> 0x40010a3c: movew %sr,%d0
>>> 0x40010a3e: andil #63743,%d0
>>> 0x40010a44: movew %d0,%sr
>>> 0x40010a46: rts
>>>
>>> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
>>> ---
>>> target/m68k/Makefile.objs | 1 +
>>> target/m68k/monitor.c | 55
>>> +++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 56
>>> insertions(+) create mode 100644 target/m68k/monitor.c
>>>
>>> diff --git a/target/m68k/Makefile.objs b/target/m68k/Makefile.objs
>>> index 39141ab93d..d143f20270 100644
>>> --- a/target/m68k/Makefile.objs
>>> +++ b/target/m68k/Makefile.objs
>>> @@ -1,3 +1,4 @@
>>> obj-y += m68k-semi.o
>>> obj-y += translate.o op_helper.o helper.o cpu.o fpu_helper.o
>>> obj-y += gdbstub.o
>>> +obj-$(CONFIG_SOFTMMU) += monitor.o
>>> diff --git a/target/m68k/monitor.c b/target/m68k/monitor.c
>>> new file mode 100644
>>> index 0000000000..03d037ccab
>>> --- /dev/null
>>> +++ b/target/m68k/monitor.c
>>> @@ -0,0 +1,55 @@
>>> +/*
>>> + * QEMU monitor
>>> + *
>>> + * Copyright (c) 2003-2004 Fabrice Bellard
>>
>> Why this copyright line (and the non-GPL license code)?
>
> I don't know. I've checked all the others monitor.c and they have the
> same header. An y suggestions?
Put at least your name in there - Fabrice did not write the new
monitor.c, but it was you instead!
Concerning the license text, this seems to be the MIT license. I guess
that's OK, just wanted to make you aware of it to consider whether you
rather want to use GPL instead or not.
Thomas
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-12-21 8:05 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-20 20:35 [Qemu-devel] [PATCH] target/m68k: add monitor.c Laurent Vivier
2017-12-20 20:54 ` Philippe Mathieu-Daudé
2017-12-21 7:57 ` Laurent Vivier
2017-12-20 21:01 ` Richard Henderson
2017-12-20 21:22 ` Thomas Huth
2017-12-21 7:56 ` Laurent Vivier
2017-12-21 8:04 ` 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).