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

[Mark Williamson <mark.williamson@xxxxxxxxxxxx>]

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.

Cheers,
Mark

-- 
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
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel