From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH][RFC] Supporting Enlightened Windows 2008Server Date: Sat, 05 Apr 2008 10:21:35 +0100 Message-ID: References: <47F68017.E57C.0030.0@novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <47F68017.E57C.0030.0@novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ky Srinivasan , Tim Deegan Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On 5/4/08 00:24, "Ky Srinivasan" wrote: > Based on the feedback I got from you and Tim, I am enclosing the next version > of the patches to support enlightened win2008 server. Here are the changes I > have made: > > 1) I have put the shim on a low calorie diet - I have gotten rid of the > framework infrastructure and to the extent possible integrated the shim code > with xen. > > 2) I have tried to cleanup the code. I am sure more work will be needed here. > > 3) I am not advertising the TLB related enlightenments. We can revisit this > later if needed. It's certainly quite a bit shorter which is good. For the remaining stuff, do you have empirical evidence that performance is improved by it? Other more minor comments are that the coding style is still off (e.g., start braces should go on their own line, spaces inside () for if/for/while headers), you have at least one big switch statement where most of the cases could be collapsed to just one shared block of code, and indeed shouldn't the 'default' case in the hypercall demuxing switch statement be to return 'denied', and that would get rid of most of the individual cases altogether? -- Keir