Re: [RFC PATCH v3 00/24] ARM: Add Xen NUMA support

From: <vkilari@codeaurora.org>
To: julien.grall@arm.com, xen-devel@lists.xen.org
Cc: vkilari@codeaurora.org, kevin.tian@intel.com,
	sstabellini@kernel.org, wei.liu2@citrix.com,
	konrad.wilk@oracle.com, George.Dunlap@eu.citrix.com,
	andrew.cooper3@citrix.com, dario.faggioli@citrix.com,
	ian.jackson@eu.citrix.com, tim@xen.org, jbeulich@suse.com,
	sgoel@codeaurora.org, nd@arm.com, shankerd@codeaurora.org
Subject: Re: [RFC PATCH v3 00/24] ARM: Add Xen NUMA support
Date: Fri, 6 Oct 2017 22:39:19 +0530	[thread overview]
Message-ID: <000001d33ec5$de5a36d0$9b0ea470$@codeaurora.org> (raw)
In-Reply-To: <CALicx6svuo3JXik=8bYuciFzWDu6qmwVi1VXdBgjLp_f_YUhqQ@mail.gmail.com>

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:vijay.kilari@gmail.com]
> Sent: Friday, October 6, 2017 7:33 PM
> To: vkilari@codeaurora.org
> Subject: Fwd: [RFC PATCH v3 00/24] ARM: Add Xen NUMA support
> 
> ---------- Forwarded message ----------
> From: Julien Grall <julien.grall@arm.com>
> Date: Wed, Jul 19, 2017 at 12:48 PM
> Subject: Re: [RFC PATCH v3 00/24] ARM: Add Xen NUMA support
> To: Vijay Kilari <vijay.kilari@gmail.com>
> Cc: nd@arm.com, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
> Andrew Cooper <andrew.cooper3@citrix.com>, George Dunlap
> <George.Dunlap@eu.citrix.com>, Ian Jackson <ian.jackson@eu.citrix.com>, Jan
> Beulich <jbeulich@suse.com>, Konrad Rzeszutek Wilk
> <konrad.wilk@oracle.com>, Stefano Stabellini <sstabellini@kernel.org>, Tim
> Deegan <tim@xen.org>, Wei Liu <wei.liu2@citrix.com>, Dario Faggioli
> <dario.faggioli@citrix.com>, kevin.tian@intel.com, Vijaya Kumar K
> <Vijaya.Kumar@cavium.com>
> 
> 
> 
> 
> On 19/07/2017 07:31, Vijay Kilari wrote:
> >
> > On Tue, Jul 18, 2017 at 9:48 PM, Julien Grall <julien.grall@arm.com> wrote:
> >>
> >> Hi,
> >>
> >> On 18/07/17 12:41, vijay.kilari@gmail.com 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
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel