This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH] 2/4 "nemesis" scheduling domains for Xen

To: xen-devel@xxxxxxxxxxxxxxxxxxx, ncmike@xxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] 2/4 "nemesis" scheduling domains for Xen
From: Mark Williamson <mark.williamson@xxxxxxxxxxxx>
Date: Mon, 7 May 2007 20:58:00 +0100
Delivery-date: Mon, 07 May 2007 12:57:27 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20070504222432.GC26054@xxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20070504222432.GC26054@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: KMail/1.9.6
Hi Mike,

A couple of minor coding style quibbles.  I'll leave actually design comments 
to those who've worked on the scheduler more recently than I!

As Keir said, there's a few minor formatting differences from the Xen code 
(e.g. spacing around the brackets in if statements).  I'd like to add that in 
Xen code that isn't shared with Linux, NULL is preferred to 0 when assigning 
to a pointer.  Applying this patch and running it through sparse through 
sparse gave me:

> sched_credit.c:457:18: warning: Using plain integer as NULL pointer
> sched_credit.c:998:18: warning: Using plain integer as NULL pointer

Only a minor nit.


Dave: Just a question. What use is a unicyle with no seat?  And no pedals!
Mark: To answer a question with a question: What use is a skateboard?
Dave: Skateboards have wheels.
Mark: My wheel has a wheel!

Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>