From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [PATCH 1/2] x86: identifying the boot cpu Date: Fri, 13 Mar 2015 19:06:54 +0000 Message-ID: <5503354E.1070103@eu.citrix.com> References: <20150313180757.GA3155@gmail.com> <5503296C.7000903@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5503296C.7000903@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Andrew Cooper , Uma Sharma , xen-devel@lists.xen.org Cc: dario.faggioli@citrix.com, keir@xen.org, George.Dunlap@citrix.com, jbeulich@suse.com List-Id: xen-devel@lists.xenproject.org On 03/13/2015 06:16 PM, Andrew Cooper wrote: > Please use git-send-email and thread your patch series properly. You > are still submitting 3 independent emails. > > On 13/03/15 18:07, Uma Sharma wrote: >> Provide helpers to access the socket and core IDs, resulting from >> identification phase. >> Initialize socket and core ID to -1 i.e invalid instead of 0. Having >> that field in all elements set to 0 would induce credit2 to think that >> the pCPU have already been initialized, and that all are on socket 0 >> in case of credit2 socket scheduler and on core 0 in case of credit2 >> core scheduler. >> >> Signed-off-by: Uma Sharma > > You have not addressed the cpu onlining/offlining problem which Jan > asked you about. > > Furthermore, you don't make any justification as to why it is safe to > change the defaults under all the other users of cpu_data. That's because Dario said these were his patches and that he would be doing the revision based on comments. That said, Uma, if it was your intention not to actually suggest this be accepted now, then you probably should have put "RFC" in the subject line, and mentioned that this is a patch that Dario is going to work on a replacement for, so that people know not to review it. -George