From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Burakov, Anatoly" Subject: Re: [PATCH v3 0/4] examples/power: allow use of more than 64 cores Date: Fri, 14 Dec 2018 13:37:37 +0000 Message-ID: <36e6f6f0-8632-96d5-697f-905745778bbe@intel.com> References: <20181214114946.21570-1-david.hunt@intel.com> <20181214133137.23516-1-david.hunt@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: lei.a.yao@intel.com To: David Hunt , dev@dpdk.org Return-path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id 0EDC91BB3B for ; Fri, 14 Dec 2018 14:37:41 +0100 (CET) In-Reply-To: <20181214133137.23516-1-david.hunt@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 14-Dec-18 1:31 PM, David Hunt wrote: > First of all we convert all the relevant uint64_t's to char arrays. Then > we remove the unneeded mask functions that were limited to 64 cores. Also > extend the guest functionality and finally rause the number of supported > cores to something more sensible, i.e. 256. > > v2 changes: > * Updates out of Review by Anatoly. > * Indentation fixes > * resolved missing spinlock around config updates > * simplified some loops > * fixed bug in non-equivalent code > > v3 changes: > * Moved a spinlock to a more appropriate place. Was in a for-loop, made > more sense to have it outside. > > [1/4] examples/power: change 64-bit masks to arrays > [2/4] examples/power: remove mask functions > [3/4] examples/power: allow vms to use lcores over 63 > [4/4] examples/power: increase max cores to 256 > > Reviewed-by: Anatoly Burakov -- Thanks, Anatoly