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

Re: [PATCH 0/2] xen: credit2: limit the number of CPUs per runqueue


  • To: Dario Faggioli <dfaggioli@xxxxxxxx>
  • From: George Dunlap <George.Dunlap@xxxxxxxxxx>
  • Date: Fri, 29 May 2020 16:15:35 +0000
  • Accept-language: en-GB, en-US
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: Juergen Gross <jgross@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
  • Delivery-date: Fri, 29 May 2020 16:15:41 +0000
  • Ironport-sdr: 38FOTJdBGyyLsqaMNQ7d+0f9HtyrVHXBruRG7Y9k/p1Z1xYWXr4ewkwKMatofCTKthXgRGUIE6 WDFmhXM3bs5sOn3KvXlP7c3wh4EAZR4IZf3IvOdtVEWos7Ox4/CdIe+v1YYBGyVtl2n890C6TP nV50LZzuPzbiKq7iQyJg2vlPQdETGpJtbdTnHbba77LRgQzyE7j1oCGg7sS/OUObr+OqhT1gnL HL4gjFhXQy/rvuUZpcGbolWvSyiCS8m92oGCt4k/NMe5Ike52y8lSnwvwKhNSSqWDrDGeVS29R MVY=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHWHky4XA5G+M3bUEaQnq/cZhz5cKi+/2qAgABLOYA=
  • Thread-topic: [PATCH 0/2] xen: credit2: limit the number of CPUs per runqueue


> On May 29, 2020, at 12:46 PM, Dario Faggioli <dfaggioli@xxxxxxxx> wrote:
> 
> So,
> 
> I felt like providing some additional thoughts about this series, from
> a release point of view (adding Paul).
> 
> Timing is *beyond tight* so if this series, entirely or partly, has any
> chance to go in, it would be through some form of exception, which of
> course comes with some risks, etc.
> 
> I did work hard to submit the full series, because I wanted people to
> be able to see the complete solution. However, I think the series
> itself can be logically split in two parts.
> 
> Basically, if we just consider patches 1 and 4 we will end up, right
> after boot, with a system that has smaller runqueues. They will most
> likely be balanced in terms of how many CPUs each one has, so a good
> setup. This will likely (actual differences seems to depend *quite a
> bit* on the actual workload) be an improvement for very large systems.

Fundamentally, I feel like the reason we have the feature freeze is exactly to 
have to avoid questions like this.  Something very much like patch 4 was posted 
before the last posting date; patches 1-4 received R-b’s before the feature 
freeze.  I think they should probably go in.

The rebalancing patches I’m inclined to say should wait until they’ve had a bit 
more time to be thought about.

 -George


 


Rackspace

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