Hi Julien,

  Sorry for not being active on this topic for some time. I was on vacation and 
 have left cavium.
Note: My email has been changed

After going through the comments and fixing some of them, I see that clean up 
and code movement
patches are ~20+. So I would like to split this series into 2 patch series.

1) Separate patch series with x86 clean up and code movement.
2) Separate patch series for ARM NUMA patches

This helps to ease in review and ease in rebasing the cleanup and code movement 
patches.

Let me know if this ok. I will work and post the patches accordingly.

Regards
Vijay
> -----Original Message-----
> From: Vijay Kilari [mailto:[email protected]]
> Sent: Friday, October 6, 2017 7:33 PM
> To: [email protected]
> Subject: Fwd: [RFC PATCH v3 00/24] ARM: Add Xen NUMA support
> 
> ---------- Forwarded message ----------
> From: Julien Grall <[email protected]>
> Date: Wed, Jul 19, 2017 at 12:48 PM
> Subject: Re: [RFC PATCH v3 00/24] ARM: Add Xen NUMA support
> To: Vijay Kilari <[email protected]>
> Cc: [email protected], "[email protected]" <[email protected]>,
> Andrew Cooper <[email protected]>, George Dunlap
> <[email protected]>, Ian Jackson <[email protected]>, Jan
> Beulich <[email protected]>, Konrad Rzeszutek Wilk
> <[email protected]>, Stefano Stabellini <[email protected]>, Tim
> Deegan <[email protected]>, Wei Liu <[email protected]>, Dario Faggioli
> <[email protected]>, [email protected], Vijaya Kumar K
> <[email protected]>
> 
> 
> 
> 
> On 19/07/2017 07:31, Vijay Kilari wrote:
> >
> > On Tue, Jul 18, 2017 at 9:48 PM, Julien Grall <[email protected]> wrote:
> >>
> >> Hi,
> >>
> >> On 18/07/17 12:41, [email protected] wrote:
> >>>
> >>>
> >>> This patch is tested on Thunderx platform.
> >>> No changes are made to x86 implementation only code is sanitized and
> >>> refactored. Hence only compilation tested for x86.
> >>>
> >>> This series is posted as RFC for the reason that it is not tested on
> >>> x86. Request some help from community in testing this series on x86.
> >>>
> >>> Code is shared at
> >>> https://github.com/vijaykilari/xen-numa/commits/rfc_v3
> >>
> >>
> >>
> >> Few months ago you sent a patch that was a pre-requisite for booting
> >> NUMA (see [1]). It has never been upstreamed, so is it still required?
> >
> >
> > yes, it is required and is merged
> 
> 
> Hmmm, somehow I was not able to find it in the tree yesterday. Found it know.
> 
> Cheers,
> 
> 
> >
> > https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=c6fdc9696a6a6eac5
> > 9bf9c81121d1f1cd5b88dcd
> >
> >>
> >> Cheers,
> >>
> >> [1]
> >> https://lists.xenproject.org/archives/html/xen-devel/2017-03/msg03823
> >> .html
> >>
> >> --
> >> Julien Grall
> 
> 
> --
> Julien Grall


_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xen.org/xen-devel

Reply via email to