[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] credit: Limit load balancing to once per millisecond
- To: George Dunlap <george.dunlap@xxxxxxxxx>
- From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- Date: Wed, 5 Jul 2023 12:33:22 +0100
- 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=q68qehWTC/YAaN/i7zcOXWAiNaBe8hg/BHivHXV3zhk=; b=n1nEX2nk5F5tJXb5uHTUgdbcV9LkC3lUbQxt8Pu4ohe31KQ0Js1/mHnkjTauCZU4IJso0q8C87ZknZKf+zwCU8ZJzwHpdi5u6QaJnDSIfyojObUtNvCdLWBiwzgAZ0p8bq5VvAwK69CUXZKOiTGlMOwIF+pufHz1OHA0cmAqMcc46amZXuB8DgQ7VyOYycQahjarkcxmAebhrCb/NHrrOZLeknn/X7hlpth6XX+nm9EzKwXofWrPtEagTha3rlNqLXJV6dHXA4rTU6D3UkuFWJageUg/EjmIkcmRH9njDh71nsdSlydB/r+IznB9ArF8qkqZinC4HlZuFWYgNZWUHQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SPKueWkKlYd73LcWADI/Ms5j/ppSQPXYxpa8kppRsgt1e7v8gCipdxcLupmFRhwWv0iaRfzdgGXvsYSK4uwnOvZtAww2S++D3YxvLWs4fMNlFnCPQI8bzYJQTnZDtnxUT+9oiSulPydNiVE2be71Ns78IdCv5HN3sJweKrETv3AG/PlD5wgq4RFwk5yw+T2vK9DujuMZJilSz5sQfIXygVsK7hOj0vn+yE7NMLPAFd3lhIxAQU95GUkwRCxDb2TiDSyTmFjiBBEZZXuJShgEf+bAmAVIRyVFb3I6yIMxVea4gmX8SO7uQIwWJ91UknthYqj8/UwSdtBa3bljIFPtRQ==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
- Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Dario Faggioli <dfaggioli@xxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
- Delivery-date: Wed, 05 Jul 2023 11:33:57 +0000
- Ironport-data: A9a23:hes7kamIvMi7D4mrcLVqfrbo5gx5J0RdPkR7XQ2eYbSJt1+Wr1Gzt xJMXzvXP6uMMGr9et1xb9nioR4F6JeAmIJlGVQ/+yE1RiMWpZLJC+rCIxarNUt+DCFhoGFPt JxCN4aafKjYaleG+39B55C49SEUOZmgH+a6U6icfHgqH2eIcQ954Tp7gek1n4V0ttawBgKJq LvartbWfVSowFaYCEpNg064gE0p5K2aVA8w5ARkPqgU5AKGzhH5MbpETU2PByqgKmVrNrbSq 9brlNmR4m7f9hExPdKp+p6TnpoiG+O60aCm0xK6aoD66vRwjnVaPpUTbZLwXXx/mTSR9+2d/ f0W3XCGpaXFCYWX8AgVe0Ew/yiTpsSq8pefSZS0mZT7I0Er7xIAahihZa07FdRwxwp5PY1B3 aUWGDxOUT6Evd63wLm2Ee9jiOMcFPC+aevzulk4pd3YJdAPZMmbBo/suppf1jp2gd1SF/HDY cZfcSBocBnLfxxIPBEQFY46m+CrwHL4dlW0qnrM/fZxvzeVk1Q3ieC1WDbWUoXiqcF9t0CUv G/ZuU/+BQkXLoe3wjuZ6HO8wOTImEsXXapLTeDiraE00A37Kmo7FEU0eRjjr+mAtV+1C9BSN G054DI3lP1nnKCsZpynN/Gim1aLsxkGVNcWH/A87AiV4qHQ5BuVQGMDS1ZpeNEg8cM7WzEu/ luIhM/yQyxitqWPTnCQ/avSqim9UQAZI2YNai4DSQot+MT4rcc4iRenZtRpHbOxj9b1MSrt2 D3Mpy87750RkMoK2qOT7V3BxTW2qfDhVRUp7w/aWmak6AJRZ4O/YYGsr1/B4p5oM4KxXlSH+ n8elKCjAPsmCJiMkGmHRroLFbTwv/KdamSE2RhoAoUr8Cmr9zi7Z4dM7TpiJUBvdMEZZTvuZ 0yVsgRUjHNOAEaXgWZMS9rZI6wXIWLITLwJiti8ggJyX6VM
- Ironport-hdrordr: A9a23:wk44b6CXgzkCl4flHemi55DYdb4zR+YMi2TDtnoBLiC9F/by/f xG88526faZslsssQgb6LK90cq7MBfhHOBOj7X5VI3KNDUO3lHIEGhM1+vfKlbbdBEWmNQttp uJmMVFebjNMWQ=
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 05/07/2023 10:25 am, George Dunlap wrote:
> On Tue, Jul 4, 2023 at 6:34 PM Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> wrote:
>> On 30/06/2023 12:37 pm, George Dunlap wrote:
>>>
>>> docs/misc/xen-command-line.pandoc | 6 +++++
>>> xen/common/sched/credit.c | 40 ++++++++++++++++++++++++++-----
>>> xen/include/public/sysctl.h | 6 +++++
>
>> Given this filelist, why the sysctl change?
>
>> There's no logic to drive this parameter in the xc/libxl param get/set.
>>
>> The only two in-tree users I can see are xenpm, along with an
>> unconditional print to stderr saying it's deprecated and to use xl,
>> and xl.
>
>
> Yes. I think unless someone objects then, I'll drop the sysctl
> interface from this patch, and add it in a follow-up patch (perhaps at a
> later date, depending on how much time I have this week).
Yeah, that sounds fine.
~Andrew
|
