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

Re: [PATCH] CHANGELOG.md: Make PV shim smaller by factoring out HVM-specific shadow code


  • To: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
  • From: George Dunlap <George.Dunlap@xxxxxxxxxx>
  • Date: Tue, 30 Mar 2021 09:56:29 +0000
  • Accept-language: 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-SenderADCheck; bh=hnTQfbeHg9DlXTMhtf1bN2bhCY5ExhlaxHfBfwMaJH8=; b=NgVohyLJJarnyMKWvdqsSXPt4A6kUlLPwHbo/XG63ES873rYOzT8XD+ChWiCbPcbRop7ft83SNZiKseX2bifY4hy1xseSTgwv/bL94h0B+jVdi+KZhcP2zgJ+zTIdvIYYBSYHwjY96w2CPkzvIs/7/Wj0tEqc1tysTRrp0PeNdnbZl8Oai+t17hxebsbRD/+nF4NAmgYihRx+myM4DYr5nSBY6fS0ajPAq7k0o4ff8bun/g9gF4T8FtSrQqSeNsAQb0IZztwf8yZXSq8lps/v+GhMRBZfT6L8Bi7nOCkpmOPYcSj015XDGU4Bmn6YmWWq1OvmroUJOhIJ0+vi/YRnQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=l7Dz6Z9lfEUh2BvhZaWnrmSC0fi8nCs9gqZEocgaSS4/Iy4zGVzYQ6X+Khe2aGIYT0mK1qLklnO52HciukytRgADgPFpQPCfQvc+lvFOzbUyqxjKCtpNHMM7dZ69TjLj6vWXMueT1BJKcObV8NwqLTwm+a1LPaq5VkLO93n7QuSjB3FAuAW0fqCaAcOmgKTGfBWmnuj0BmBqFgRlfpCC+Imio5vFvI72WQfaZSi/VwTyZavk35gPVH7deIj1GzxAvmhGdB9hMJxCCiXVER84+l2J5WpZXCU5BQ/spc14wTnYTdVAFhvqwbt6dAh5K3m8oTUrCT5bDjuLkfjpYsbhbQ==
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Jan Beulich <jbeulich@xxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Paul Durrant <paul.durrant@xxxxxxxxxx>
  • Delivery-date: Tue, 30 Mar 2021 09:57:02 +0000
  • Ironport-hdrordr: A9a23:Pv4vIqliAN3VsUDVBHj0/E83GpfpDfP+imdD5ilNYBxZY6Wkvu izgfUW0gL1gj4NWHcm3euNIrWEXGm0z/NIyKMaVI3DYCDNvmy0IIZ+qbbz2jGIIVybysdx94 dFN5J/Btr5EERgga/BijWQPt48zLC8n5yAqvzZyx5WIz1CT4FFw0NHBh2AEktwLTM2YKYRMJ aH/MJIq36BVB0sH6eGL0IIVeTCuNHH/aiOCXI7LiUq9RWUineQ4KP6eiLy4j4lTzhNzb0+mF K18TDR26PLiZCG4y6Z7UD/xdB8mNztytxMbfb89/Q9G3HXpSuDIKhkU72GljgprO+o80ZCqq ixnz4Qe/5dxlmUUmapoQb8+wSI6kdQ11bSjWW2rFGmgcvlSCk0A8BM7LgpDCfx2g4bk/xXlI dotljp0KZ/PFf7swnWo+XsbVVMkHG5pHIz+NRj9EB3YM8lR5J66bAE8Fg9KuZnIAvKrLoJPc NJF8/m6PNfYTqhHgrkl1gq+tCqU3gpdy32O3Qqi4iQ2zhSqnhz01EV8swZhmsB75IwUfB/lp z5Dpg=
  • Ironport-sdr: 1tif6AZsf7cOzvaQR71sxL7vL8h5TXylpMPdhdLyZ+kPNsVOoOXeXW57R1PU3FKfvfmMP4Not0 +O2GmwGPyWRQ16ktGzbXgZWb/ZSyKum4kPuff745XpVr9Bfhv+v/wdn6/Hvp3eHKIULyN0JDBF AQlsdGc495NrIss/uF+q7WsGnjOAGvrgDl+AZNDd6xKE/eIR03ZvURGzuOZGVXZqm3jEoelUWI 7FZZwAETbs5ShIvZ8uty4nJMIoCOw7Pe4+bSEaUtcpEIjx9GsDvPcAzbrtevqbytW7/y5Vzrog WEw=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHXINLTS489UfL61EKD4pjlwXnfbqqUSZCAgAbMCwCAABhsgIAAEbyAgAEUfwA=
  • Thread-topic: [PATCH] CHANGELOG.md: Make PV shim smaller by factoring out HVM-specific shadow code


> On Mar 29, 2021, at 6:26 PM, Andrew Cooper <andrew.cooper3@xxxxxxxxxx> wrote:
> 
> On 29/03/2021 17:23, Jan Beulich wrote:
>> On 29.03.2021 18:12, George Dunlap wrote:
>>>> On Mar 25, 2021, at 7:08 AM, Jan Beulich <JBeulich@xxxxxxxx> wrote:
>>>> On 24.03.2021 18:26, George Dunlap wrote:
>>>>> Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxx>
>>>>> ---
>>>>> Missed one from my list when creating the other series
>>>>> 
>>>>> CC: Ian Jackson <ian.jackson@xxxxxxxxxx>
>>>>> CC: Jan Beulich <jbeulich@xxxxxxxx>
>>>>> CC: Roger Pau Monne <roger.pau@xxxxxxxxxx>
>>>>> ---
>>>>> CHANGELOG.md | 1 +
>>>>> 1 file changed, 1 insertion(+)
>>>>> 
>>>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>>>> index 15a22d6bde..49832ae017 100644
>>>>> --- a/CHANGELOG.md
>>>>> +++ b/CHANGELOG.md
>>>>> @@ -18,6 +18,7 @@ The format is based on [Keep a 
>>>>> Changelog](https://keepachangelog.com/en/1.0.0/)
>>>>> - x86_emulate: Expanded testing for several instruction classes
>>>>> - CI loop: Add Alpine Linux, Ubuntu Focal targets; drop CentOS 6
>>>>> - CI loop: Add dom0less aarch64 smoke test
>>>>> + - Factored out HVM-specific shadow code, allowing PV shim to be slimmer
>>>> But shadow code doesn't get included by default in shim-exclusive
>>>> builds (and others are unlikely to disable HVM).
>>> Can you propose some better text please?
>> Does this need mentioning here in the first place?
> 
> I would recommend not.
> 
> We've been doing incremental improvements for the shim for several
> releases now, and in this case, we're literally talking a few kb of
> code.  As we already align to 2M boundaries for superpage reasons, there
> almost certainly isn't actually a reduction in runtime size.

I don’t understand why the two of you are downplaying your work so much.  Yes, 
these are all only incremental improvements; but they are improvements; and the 
cumulative effect of loads of incremental improvements can be significant.  
Communicating to people just what the nature of all these incremental 
improvements are is important.

I mean, look at the release notes for Go 1.15 [1].  It includes things like 
this:

"JSEscape now consistently uses Unicode escapes (\u00XX), which are compatible 
with JSON."

"go test -v now groups output by test name, rather than printing the test name 
on each line."

Those sound far more trivial than “Even more shadow code has been moved to an 
HVM-specific file”.

If the approach is going to be “SUPER IMPORTANT SPECIAL STUFF ONLY", I’d 
recommend removing CHANGELOG.md.  Having an official list that says, “Well, 
really, we only did 2 things this release” is going to be actively harmful.  

 -George

[1] https://golang.org/doc/go1.15

 


Rackspace

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