Re: [Minios-devel] [PATCH 00/40] MINI-OS: enable the arm64 support

[Julien Grall <julien.grall@xxxxxxxxxx>]

Hi Shijie,

On 07/11/2017 08:08, Huang Shijie wrote:
> On Fri, Nov 03, 2017 at 06:59:24PM +0100, Samuel Thibault wrote:
> > Hello,
> >
> > Some of the patches can be committed right away:
> >
> > [PATCH 01/40] mini-os: fix the wrong parameter for map_free() in 
> > init_page_allocator()
> > [PATCH 14/40] arm64: mm.c: fix the compiler error
> > [PATCH 15/40] arm64: refine the arch_init_mm
> > [PATCH 22/40] arm64: change sp to "unsigned long" type
> > [PATCH 23/40] arm64: fix the wrong size of the register
> >
> > and these can't hurt applying already
> > [PATCH 10/40] arm64: define the quad_t for arm64
> > [PATCH 12/40] mini-os: define ULONG_MAX/LONG_MAX for arm64
> Thanks Samuel, I hope the maintainer can merge these patches.
> And my next version will become smaller.
>
> It is a too big patch set for us now.
The patch series is too big because you decided to rework, do the 
preparation and add arm64 implementation within the same series.
I went through the series and I think it is fairly easy to split it 
(though you may have to re-order patches).
This will be much easier if you follow one the path I suggested in the 
cover letter. BTW, I haven't had any feedback from you there...
Cheers,

--
Julien Grall

_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/minios-devel