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

Re: [Xen-devel] [PATCH v4 6/8] ioreq-server: remove p2m entries when server is enabled



>>> On 09.04.14 at 15:36, <Paul.Durrant@xxxxxxxxxx> wrote:
>> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
>> Also, you're pretty consistently naming the field/variable "enabled"
>> rather than "enable", despite it being a transition you're invoking
>> rather than obtaining state.
> 
> Yes, because I'm setting whether the server state is 'enabled' or not. The 
> value of the boolean is the end state not the transition, so it's correct to 
> use the adjective rather than the verb.

Hmm, that's not my way of thinking with operations like this. To me,
the operation is to enable (or disable) the server, not to set its state
to enabled (or disabled). But yes, one may view it your way too,
even if I would think that's not commonly done (and I think I saw
"enable"s too somewhere in the series, and maybe even in the same
patch, so I'd be inclined to ask for consistency even if our ways of
thinking of these operations differ).

Jan


_______________________________________________
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®.