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

Re: [PATCH v8 0/7] Boot time cpupools


  • To: Julien Grall <julien@xxxxxxx>
  • From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • Date: Fri, 6 May 2022 11:43:27 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=WXy3JtLng7ohu1dj92FGp1hY+RNsJ9Vt+cO9IyT1eQI=; b=aIaM4xUA8WXI0pfJooriG/vtjSZ2n+1QZBkWUqbwMfo0c9KLcOKyauBw7j5xdj4wC+0nyBSvfBGe6xsWiEb1hS/52IOxq5M8Tw3wQ2Tq8flhAl6EtSZWO8PEBo09Pj9Zzetluy1npfqSPvBUxWRWqOraLLacJrtS0UZC1NtBEwQYCE1LuIIBDuIKrxnmXdbxGMy0Vr773iVBEv7km3iaw4AZ8XYAgkgrj8AtPbYGj2Qdc9aHU/x4P/puWmF36iS+iqyxEpq+MDE48dN1Ya3de2Z1PDYk8BiGMnE4Whs4QbT2ENVgfZtJUSfI6Escqd6v64QUdcl/bLS5rwZIME4Scg==
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=WXy3JtLng7ohu1dj92FGp1hY+RNsJ9Vt+cO9IyT1eQI=; b=SeTKjBGkC0T3d+dD4AECYiWq07SQc5dv8hdXa06IhZdNTCV/6EOJGfXqDDYdTL+bexK5xaWPDtV8ADQywTW5q08H663aYqpSlGc6flWierXen0/8n0yZ08ztTes+cytMX6JWB2b2aYCeCt0QuIWpIIM3iIrT6FrWNbq4ZMs9HAClEscb3+ACcWzHdExxqAUvzEP4AIwYl41igeUZeLAVcSzmvAaPsuZuwt/aRAjOp8CUg8EX8DZZsZ5CL6DfIHuBD9+m2L7Ur3G6XinZT7TVEJx5fNU7I1lq57NQQUeTOrhK3j8SUX8ps34yXye8OofEkxsCa5+vqmO1ldqSWepf4w==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=FckjDgDMccQb0C1CHBKH+5kGn0Nvni0IkkJuu9yC0xoyTMVoxjyaJxl0tj8oep4E98eOC9yoURlVzL12uSi+gMpHoSnw238WcYqC+QLe2kt3GakpGIVlq1plsnE1tLJHygbGEXR7GtK9Fj8SP+mD9ERvGpBN+c2/p1wJyFBPGMiM8rz4BX/74Ct7UuUurxq3F256K9n0FggGnBu10c/uEJx1o9jungbFO4KbNCaJwRwhvuKrQHVqvnKz3KD6sPQPZMzFyoC9NFZk/X7+V5RU4wHDWpccirpq02N/ZDTIhrnM73LjNbXIBupuEAtYvjrRYUcs+GlCgq26zblnmUNShQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lCRb3bXjg78sPFOwipwk3l/E0bnyQc5egu1o9UEETYTOJ+fwAS+HK2l5nZ+0FGUxNJ8AQg21sd7IahVOaqHNoshAsaA22ybfnM5ySP8qgar6c+RcAZ/8PKRfeJbwBKJ6wkK9zxUvbMSLVnLkbFdAw49XvIvmIaeGH8BTxQm+w/J8elmTQOBmUk2dg50aXXo0cpLkjaIk0majS5iV4Wff7XVPbVsXvi20rLcGHi4F4+a5zMUF1buL/DVvEqKcOtfK4dsC6Ph9AXBuv9nWFF+SaSzNBG9Y4HK/kvUtQs8pG/ecvNhmVB+IgplV2gjiC8gl+IeWg4MmvsMLpCGO7Z7TKw==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>
  • Delivery-date: Fri, 06 May 2022 11:43:50 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHYVVhdzvmSbSkA9kKhcVjYQ/ksyq0RzUeAgAAEKQCAAADfgA==
  • Thread-topic: [PATCH v8 0/7] Boot time cpupools

>>> 
>>> Luca Fancellu (7):
>>> tools/cpupools: Give a name to unnamed cpupools
>>> xen/sched: create public function for cpupools creation
>>> xen/sched: retrieve scheduler id by name
>>> xen/cpupool: Create different cpupools at boot time
>>> xen/cpupool: Don't allow removing cpu0 from cpupool0
>>> arm/dom0less: assign dom0less guests to cpupools
>>> xen/cpupool: Allow cpupool0 to use different scheduler
>>> 
>>> MAINTAINERS | 2 +-
>>> docs/misc/arm/device-tree/booting.txt | 5 +
>>> docs/misc/arm/device-tree/cpupools.txt | 140 +++++++++++++++
>>> tools/helpers/xen-init-dom0.c | 37 +++-
>>> tools/libs/light/libxl_utils.c | 3 +-
>>> xen/arch/arm/domain_build.c | 19 +-
>>> xen/arch/arm/include/asm/smp.h | 3 +
>>> xen/common/domain.c | 2 +-
>>> xen/common/sched/Kconfig | 7 +
>>> xen/common/sched/Makefile | 1 +
>>> xen/common/sched/boot-cpupool.c | 234 +++++++++++++++++++++++++
>>> xen/common/sched/core.c | 40 +++--
>>> xen/common/sched/cpupool.c | 43 ++++-
>>> xen/include/public/domctl.h | 5 +-
>>> xen/include/xen/sched.h | 53 ++++++
>>> 15 files changed, 563 insertions(+), 31 deletions(-)
>>> create mode 100644 docs/misc/arm/device-tree/cpupools.txt
>>> create mode 100644 xen/common/sched/boot-cpupool.c
>>> 
>>> -- 
>>> 2.17.1
>>> 
>> Ping?
> 
> You seemed to have forgotten to CC the relevant maintainers on each patch and 
> the cover letter. I only got this e-mail because I have a filter for some 
> keywords.
> 
>> The v8 seems to be reviewed for all patch
>> https://patchwork.kernel.org/project/xen-devel/list/?series=634064
> 
> AFAICT, patchwork only tells you the number of reviewed-by tag. It doesn't 
> tell you whether the patch was suitably reviewed.
> 
> Looking through the patches:
> #1: Ready
> #2: Ready
> #3: Ready
> #4: Missing review from the cpupool maintainers (Juergen or Dario)
> #5: Ready
> #6: Missing review from the cpupool maintainers
> #7: Ready
> 

Thanks Julien, indeed my ping didn’t CC the maintainers, I will ping separately 
on the #4 and #6 patches.

Cheers,
Luca

> Cheers,
> 
> -- 
> Julien Grall


 


Rackspace

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