[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [for-4.17] automation: Do not use null scheduler for boot cpupools test
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
- Date: Mon, 24 Oct 2022 10:09:18 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
- 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=grys+4P5GcmRdhTc6X0D2ylf+A8Zr53c44RRir2OzAM=; b=nPzjEUDyGUObRjBuIQMV/E1zLdFZ/O9jQxmBdqp0vCSChtU8okU/3ToATKGp+nNvjxO6qkDGvCk0N3BFvvRh6RO0oxpqZYsnoqlt1DNqq3kvL1BOL9I38uCSCzun7yUs2+5HnuypaAmgzdQSCUgHdKag+hOa9sqBE3Gp663x5EOt/RULFIzbyRoJezur50pqYrZdFqMAX4yLyDUpBT3kRuec8Y+9BM6VFTPDn4m5dpnkTKJBfU1PRtjeSQmOH/vPHqn84ZCOSYOvMlgDvQZ5e7UTeIduL3dyjYLnE11Dd2IJv4RH+7Ycc+LxMwjRI2IrmwI0KEEGNdP/QZwlZlrUpQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BVNtmvCm3dzWdtiTSiz5bjjd6/FqVx5kzhjgGIwKJoINKu5eaeu+UMHaw5CwPOZxcL8aNQT1PFLX3mMTRwAmey2lR690gx5l4IG1UWvZkYC4rsHn8JiYj9IvdO52iiVvYm8LrkIDiwZzQ09Bm6Fu+BYfZpGrBOqjRgIitc9yhrmx8kyojlOfgH2A5WdNQtmOc1i2Iw+Na0aDQWZ3nDNVTg/Nj/+R+miqOwtgJgg0SaO4eUTP3xQ6Rwh4CoWpX/dWXihWrCZodLO7fydNdEQFy8iGs9SFrsrv7xmFU6qRPyhT4DnNfBk/hKOWL4ZNxKvoTs41cleLSozzKR9Xjuaiwg==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
- Cc: Doug Goldstein <cardoe@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Henry Wang <Henry.Wang@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Mon, 24 Oct 2022 10:09:33 +0000
- Ironport-data: A9a23:aTtMjqAxoTXaohVW/8Diw5YqxClBgxIJ4kV8jS/XYbTApG9z3zBWy jQYUWyFb/mKYTf3KIogPt+08k9Q6J/VzYUxQQY4rX1jcSlH+JHPbTi7wuUcHAvJd5GeExg3h yk6QoOdRCzhZiaE/n9BCpC48T8mk/ngqoPUUIbsIjp2SRJvVBAvgBdin/9RqoNziJ2yDhjlV ena+qUzA3f4nW8pWo4ow/jb8kk25K6u4GlwUmEWPpingnePzxH5M7pHTU2BByOQapVZGOe8W 9HCwNmRlo8O105wYj8Nuu+TnnwiGtY+DyDX4pZlc/HKbix5jj4zys4G2M80Mi+7vdkrc+dZk 72hvbToIesg0zaldO41C3G0GAkmVUFKFSOuzdFSfqV/wmWfG0YAzcmCA2kqZ7AeufxoIlpu1 r9ECGpTZBqFvuaplefTpulE3qzPLeHNFaZG4jRK626cCvwrB5feX6/N+NlUmi8qgdxDFurfY MxfbidzaBPHYFtEPVJ/5JAWxb/0wCWgNWIA7gvN+MLb4ECKpOB1+JHrPMDYZZqhQsJNk1zDj mnH4374ElcRM9n3JT+toijw2LWex3qTtIQ6N7yUptpmumaq7DItMAUmc0uphfO+oxvrMz5YA wlOksY0loAw/kG2Stj2XzWjvWWJ+BUbXrJ4GOQg5AaA4qHd+QqeCy4PSTspQMc9qMY8SDgu1 1mIt9DkHzpitPuSU3313peZqymjfxccK2AqbDUBCwAC5rHLpYA4jhvLCNF5AqOxptTwFXf7x DXihDgzgfAfgNAG042//EvbmHS8q57RVAk36w7LGGW/4WtEiJWNYoWp7R3R66lGJYPAFl2Z5 iFcxo6Z8fwECoyLmGqVWuIREbq15vGDdjrBnVpoGJpn/DOok5K+Qb1tDPhFDB8BGq45lfXBO Sc/ZSs5CEdvAUaX
- Ironport-hdrordr: A9a23:v+g6GarbXQuOlbQT7ZtK6S4aV5uIL9V00zEX/kB9WHVpm5Oj+v xGzc5w6farsl0ssSkb6Ku90KnpewK+yXbsibNhcItKLzOWwldAS7sSobcKogeQUREWk9Qw6U 4OSdkYNDSdNzlHZIPBkXGF+rUbsZa6GcKT9IHjJh5WJGkEBZ2IrT0JczpzeXcGJjWucKBJcK Z0kfA3wgZIF052Uu2LQl0+G8TTrdzCk5zrJTQcAQQ81QWIhTS0rJbnDhmxxH4lInJy6IZn1V KAvx3y562lvf3+4ATbzXXv45Nfn8ak4sdfBfaLltMeJlzX+0aVjcVaKv6/VQIO0aSSAWUR4Z 3xStAbToNOAkbqDyOISN3Wqk/dOXgVmibfIBSj8AbeSITCNU4H4ox69Mxkm1LimjQdVJsX6t M140uJ85VQFh/OhyL7+pzBUAxrjFO9pT44nfcUlGE3a/pWVFZ9l/1pwKpuKuZ3IAvqrIQ8VO V+BsDV4/hbNVuccnDCp2FqhNihRG46EBuKSlUL/pX96UkcoFlpi08DgMAPlHYJ85wwD5FC+u TfK6xt0LVDVNUfY65xDPoIBcG3FmvOSxTRN3/6GyWuKIgXf3bW75Ln6rQ84++nPJQO0ZspgZ zEFEhVsGYjEnieffFmHKc7hywlbF/NLggFkPsulqSRkoeMNIbDIGmEVE0kldemrrEWHtDbMs zDSq5rPw==
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Thread-index: AQHY5W3J3NgQRSiY5kiY+kL0DoKe0q4ZGCaAgAQA44CAAD12AA==
- Thread-topic: [for-4.17] automation: Do not use null scheduler for boot cpupools test
On 24/10/2022 07:29, Jan Beulich wrote:
> On 21.10.2022 19:21, Andrew Cooper wrote:
>> On 21/10/2022 17:53, Michal Orzel wrote:
>>> Null scheduler is not enabled on non-debug Xen builds so the current
>>> test can lead to a failure on such jobs. We still want to test that we
>>> can assign the cpupool to a domU with a different scheduler than default
>>> one (credit2). Switch to credit as it is enabled by default.
>>>
>>> Fixes: 36e3f4158778 ("automation: Add a new job for testing boot time
>>> cpupools on arm64")
>>> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
>> /sigh - I'm sure I nacked that stupidity to begin with. apparently not...
>>
>> It is totally bogus for CONFIG_DEBUG to influence logical chunks of
>> functionality like this. The CI script is good in its current form.
> Assuming you mean defaults of settings, I'm afraid I see nothing bogus
> there at all.
It's a complete violation of any reasonable interpretation of DEBUG.
Apart from creating the bug at the centre of this thread, one does not
turn on DEBUG to get at this functionality in the first place, so the
options should not be interlinked.
>> RTDS and ARINC should be default n.
>>
>> NULL is more tricky. PV_SHIM is explicitly security supported, and has
>> been for years, so the "UNSUPPORTED" is bogus, whatever the default is.
>>
>> As NULL is explicitly tested in CI, it's clearly supported, and probably
>> ought to be on default.
> ... the state of the NULL scheduler wrt its use by the shim has been
> puzzling me before.
NULL is exactly what the shim wants, and it gets thorough testing.
The only remaining problem is the paperwork. NULL is already "security
supported in pv shim", hence the unsupported tag is bogus.
~Andrew
|
