Hello George,

On 5/30/19 17:05, George Dunlap wrote:
On May 30, 2019, at 6:47 AM, Baodong Chen <chenbaod...@mxnavi.com> wrote:

So remove 'keyhandler.h' include.
Also add 'static' prefix for 'schud_bull_def'

Signed-off-by: Baodong Chen <chenbaod...@mxnavi.com>
Thanks for the patch — these changes look good.  I think the title would be 
better something like:

xen/sched_null: Superficial clean-ups

Then just list both in bullet points; something like:

* Remove unused dependency ‘keyhandler.’h
* Make sched_null_def static

Would you mind re-sending the patch?  You can add:

Reviewed-by: George Dunlap <george.dun...@citrix.com>
Thanks for your review, resent.
Thanks,
  -George


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to