[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm



Dario Faggioli writes ("Re: [PATCH v2 5/5] libxl: Add explicit cast to 
libxl_psr_cat_set_cbm"):
> On Tue, 2016-01-19 at 14:21 +0000, Ian Campbell wrote:
> > libxl_shutdown_reason has the same issues, libxl_tsc_mode also might,
> > as might libxl_timer_mode. I'm not sure if there are others. 
> >
> (FTR) libxl_scheduler too, probably?

Quite possibly.  Hrm.

I see that for shutdown_reason Ian C did this deliberately in "libxl:
Remove xen/sched.h from public interface".

I am really not a fan of this approach.  I don't remember what I said
about that at the time.  The result is multiple enums with the same
values in that need to be kept in step.

However, I'm not sure I can persuade everyone to agree on an
alternative.  At the very least we should document the rules in
CODING_STYLE.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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