[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.