[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



On 31.03.2021 15:52, Ian Jackson wrote:
> George Dunlap writes ("Re: [PATCH] CHANGELOG.md: Make PV shim smaller by 
> factoring out HVM-specific shadow code"):
>> 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 agree with George here.
> 
> There ae a number of reasons why behind-the-scenes work with little
> (intentional) user-visible impact are useful to note in the
> CHANGELOG.md.  With my Release Manager hat on I would like to see, for
> example,
> 
>>> + - Factored out HVM-specific shadow code, allowing PV shim to be slimmer
> 
> something about htis work in the CHANGELOG.md.
> 
> IDK precisely, and I don't think George does either, what a good and
> accurate statement is.  But I guess we will go with the text above if
> we don't get something better.

At the very least the part after the comma ought to be deleted. As
said in an earlier reply, at least the shim default config disables
shadow code anyway, so the factoring out has no effect there.

Jan



 


Rackspace

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