Re: [Qemu-devel] [PATCH 0/2] m68k: 680x0 processors family support

[Laurent Vivier]

Le 22/06/2015 10:33, Peter Maydell a écrit :
> On 21 June 2015 at 23:35, Laurent Vivier <address@hidden> wrote:
>> I carry this series for several years now.
>>
>> I was reluctant to merge it with the mainstream because
>> I was sure it was breaking Coldfire support.
>>
>> But with the kernel provided by Greg Ungerer I was able
>> to check and correct the support of ColdFire family
>> in my tree.
>>
>> This series allows:
>> - to start a ColdFire semi-hosted kernel (m68k-softmmu)
>> - chroot or start a container of debian etch-m68k (m68k-linux-user)
>>
>> I have another series providing privileged instructions and
>> allowing to boot a 680x0 debian kernel (Macintosh Quadra 800),
>> but this one needs more work (and love). Based on this work,
>> we will also be able to boot a NextStation (Bryce Lanham, GSoC)
>>
>> This work is based on the work of Andreas Schwab.
>>
>> Laurent Vivier (2):
>>   m68k: Add compatibility with 680x0 processors family
>>   m68k: Implement 680x0 processors family 96 bit FPU
>>
>>  configure                  |    2 +-
>>  cpu-exec.c                 |    6 -
>>  disas.c                    |    4 +
>>  fpu/softfloat-specialize.h |   48 +-
>>  fpu/softfloat.c            |   38 +-
>>  gdb-xml/m68k-fp.xml        |   21 +
>>  include/fpu/softfloat.h    |   11 +-
>>  target-m68k/cpu.c          |   67 +-
>>  target-m68k/cpu.h          |   76 +-
>>  target-m68k/helper.c       | 1691 ++++++++++++++++++--
>>  target-m68k/helper.h       |  100 +-
>>  target-m68k/m68k-qreg.h    |   11 -
>>  target-m68k/op_helper.c    |  182 ++-
>>  target-m68k/qregs.def      |    6 +-
>>  target-m68k/translate.c    | 3682 
>> +++++++++++++++++++++++++++++++++++---------
>>  15 files changed, 5013 insertions(+), 932 deletions(-)
>>  create mode 100644 gdb-xml/m68k-fp.xml
>>  delete mode 100644 target-m68k/m68k-qreg.h
> 
> Thanks for sending this. My initial feeling is that this would
> really benefit from being split up into more patches to make
> it easier to review.

Thank you Peter, I understand.

In fact, in my tree, I have 160 commits I have merged in two big ones
;-) (some are new features, other bug fixes).

So, no problem, just define the granularity, I will create the patches
accordingly.

Is it possible to integrate them little by little ?
I really want to see my patches stack to decrease.

Laurent