[Pvfs2-developers] concurrent sms

Sam Lang slang at mcs.anl.gov
Wed Jul 12 16:26:40 EDT 2006


Hi Walt,

The state machine stuff you've got in WALT3 branch looks like its  
coming along.  Nice to see those cleanups to the state_machine  
functions.  I have a minor nit about this:

/* Op State - this is the user-specific (client or server) information
* that must be maintained for each running state machine - if new
* "users" are created the need to be added to this union
*/
union PINT_OP_STATE
{
     struct PINT_server_op s; /* server specific fields */
     struct PINT_client_sm c; /* client specific fields */
};

The caps makes me think of #define, which I think it used to be.   
Could that be PINT_op_state instead?  Or is this the union that is  
going to be replaced with a void*?

-sam


More information about the Pvfs2-developers mailing list