diff options
author | Sunil Mushran <sunil.mushran@oracle.com> | 2010-12-20 19:35:01 -0500 |
---|---|---|
committer | Joel Becker <joel.becker@oracle.com> | 2010-12-22 21:34:46 -0500 |
commit | 079ffb743c622fe2189b75614427c56e8391498b (patch) | |
tree | a6618e7e8bc95041d1ac6a4650be7c1840c71bd8 /fs | |
parent | 37096a7927decb0b1d3c2514b8adb4583a834112 (diff) |
ocfs2/dlm: Hard code the values for enums
In o2dlm, the enumerated message values are part of the protocol.
The patch hard codes each value so as to reduce the chance of an editing
error causing a protocol mismatch.
Signed-off-by: Sunil Mushran <sunil.mushran@oracle.com>
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ocfs2/dlm/dlmcommon.h | 86 |
1 files changed, 43 insertions, 43 deletions
diff --git a/fs/ocfs2/dlm/dlmcommon.h b/fs/ocfs2/dlm/dlmcommon.h index b36d0bf77a5a..4bdf7baee344 100644 --- a/fs/ocfs2/dlm/dlmcommon.h +++ b/fs/ocfs2/dlm/dlmcommon.h | |||
@@ -50,10 +50,10 @@ | |||
50 | #define dlm_lockid_hash(_n, _l) full_name_hash(_n, _l) | 50 | #define dlm_lockid_hash(_n, _l) full_name_hash(_n, _l) |
51 | 51 | ||
52 | enum dlm_mle_type { | 52 | enum dlm_mle_type { |
53 | DLM_MLE_BLOCK, | 53 | DLM_MLE_BLOCK = 0, |
54 | DLM_MLE_MASTER, | 54 | DLM_MLE_MASTER = 1, |
55 | DLM_MLE_MIGRATION, | 55 | DLM_MLE_MIGRATION = 2, |
56 | DLM_MLE_NUM_TYPES | 56 | DLM_MLE_NUM_TYPES = 3, |
57 | }; | 57 | }; |
58 | 58 | ||
59 | struct dlm_master_list_entry { | 59 | struct dlm_master_list_entry { |
@@ -82,8 +82,8 @@ struct dlm_master_list_entry { | |||
82 | 82 | ||
83 | enum dlm_ast_type { | 83 | enum dlm_ast_type { |
84 | DLM_AST = 0, | 84 | DLM_AST = 0, |
85 | DLM_BAST, | 85 | DLM_BAST = 1, |
86 | DLM_ASTUNLOCK | 86 | DLM_ASTUNLOCK = 2, |
87 | }; | 87 | }; |
88 | 88 | ||
89 | 89 | ||
@@ -119,9 +119,9 @@ struct dlm_recovery_ctxt | |||
119 | 119 | ||
120 | enum dlm_ctxt_state { | 120 | enum dlm_ctxt_state { |
121 | DLM_CTXT_NEW = 0, | 121 | DLM_CTXT_NEW = 0, |
122 | DLM_CTXT_JOINED, | 122 | DLM_CTXT_JOINED = 1, |
123 | DLM_CTXT_IN_SHUTDOWN, | 123 | DLM_CTXT_IN_SHUTDOWN = 2, |
124 | DLM_CTXT_LEAVING, | 124 | DLM_CTXT_LEAVING = 3, |
125 | }; | 125 | }; |
126 | 126 | ||
127 | struct dlm_ctxt | 127 | struct dlm_ctxt |
@@ -388,8 +388,8 @@ struct dlm_lock | |||
388 | 388 | ||
389 | enum dlm_lockres_list { | 389 | enum dlm_lockres_list { |
390 | DLM_GRANTED_LIST = 0, | 390 | DLM_GRANTED_LIST = 0, |
391 | DLM_CONVERTING_LIST, | 391 | DLM_CONVERTING_LIST = 1, |
392 | DLM_BLOCKED_LIST | 392 | DLM_BLOCKED_LIST = 2, |
393 | }; | 393 | }; |
394 | 394 | ||
395 | static inline int dlm_lvb_is_empty(char *lvb) | 395 | static inline int dlm_lvb_is_empty(char *lvb) |
@@ -427,27 +427,27 @@ struct dlm_node_iter | |||
427 | 427 | ||
428 | 428 | ||
429 | enum { | 429 | enum { |
430 | DLM_MASTER_REQUEST_MSG = 500, | 430 | DLM_MASTER_REQUEST_MSG = 500, |
431 | DLM_UNUSED_MSG1, /* 501 */ | 431 | DLM_UNUSED_MSG1 = 501, |
432 | DLM_ASSERT_MASTER_MSG, /* 502 */ | 432 | DLM_ASSERT_MASTER_MSG = 502, |
433 | DLM_CREATE_LOCK_MSG, /* 503 */ | 433 | DLM_CREATE_LOCK_MSG = 503, |
434 | DLM_CONVERT_LOCK_MSG, /* 504 */ | 434 | DLM_CONVERT_LOCK_MSG = 504, |
435 | DLM_PROXY_AST_MSG, /* 505 */ | 435 | DLM_PROXY_AST_MSG = 505, |
436 | DLM_UNLOCK_LOCK_MSG, /* 506 */ | 436 | DLM_UNLOCK_LOCK_MSG = 506, |
437 | DLM_DEREF_LOCKRES_MSG, /* 507 */ | 437 | DLM_DEREF_LOCKRES_MSG = 507, |
438 | DLM_MIGRATE_REQUEST_MSG, /* 508 */ | 438 | DLM_MIGRATE_REQUEST_MSG = 508, |
439 | DLM_MIG_LOCKRES_MSG, /* 509 */ | 439 | DLM_MIG_LOCKRES_MSG = 509, |
440 | DLM_QUERY_JOIN_MSG, /* 510 */ | 440 | DLM_QUERY_JOIN_MSG = 510, |
441 | DLM_ASSERT_JOINED_MSG, /* 511 */ | 441 | DLM_ASSERT_JOINED_MSG = 511, |
442 | DLM_CANCEL_JOIN_MSG, /* 512 */ | 442 | DLM_CANCEL_JOIN_MSG = 512, |
443 | DLM_EXIT_DOMAIN_MSG, /* 513 */ | 443 | DLM_EXIT_DOMAIN_MSG = 513, |
444 | DLM_MASTER_REQUERY_MSG, /* 514 */ | 444 | DLM_MASTER_REQUERY_MSG = 514, |
445 | DLM_LOCK_REQUEST_MSG, /* 515 */ | 445 | DLM_LOCK_REQUEST_MSG = 515, |
446 | DLM_RECO_DATA_DONE_MSG, /* 516 */ | 446 | DLM_RECO_DATA_DONE_MSG = 516, |
447 | DLM_BEGIN_RECO_MSG, /* 517 */ | 447 | DLM_BEGIN_RECO_MSG = 517, |
448 | DLM_FINALIZE_RECO_MSG, /* 518 */ | 448 | DLM_FINALIZE_RECO_MSG = 518, |
449 | DLM_QUERY_REGION, /* 519 */ | 449 | DLM_QUERY_REGION = 519, |
450 | DLM_QUERY_NODEINFO, /* 520 */ | 450 | DLM_QUERY_NODEINFO = 520, |
451 | }; | 451 | }; |
452 | 452 | ||
453 | struct dlm_reco_node_data | 453 | struct dlm_reco_node_data |
@@ -460,19 +460,19 @@ struct dlm_reco_node_data | |||
460 | enum { | 460 | enum { |
461 | DLM_RECO_NODE_DATA_DEAD = -1, | 461 | DLM_RECO_NODE_DATA_DEAD = -1, |
462 | DLM_RECO_NODE_DATA_INIT = 0, | 462 | DLM_RECO_NODE_DATA_INIT = 0, |
463 | DLM_RECO_NODE_DATA_REQUESTING, | 463 | DLM_RECO_NODE_DATA_REQUESTING = 1, |
464 | DLM_RECO_NODE_DATA_REQUESTED, | 464 | DLM_RECO_NODE_DATA_REQUESTED = 2, |
465 | DLM_RECO_NODE_DATA_RECEIVING, | 465 | DLM_RECO_NODE_DATA_RECEIVING = 3, |
466 | DLM_RECO_NODE_DATA_DONE, | 466 | DLM_RECO_NODE_DATA_DONE = 4, |
467 | DLM_RECO_NODE_DATA_FINALIZE_SENT, | 467 | DLM_RECO_NODE_DATA_FINALIZE_SENT = 5, |
468 | }; | 468 | }; |
469 | 469 | ||
470 | 470 | ||
471 | enum { | 471 | enum { |
472 | DLM_MASTER_RESP_NO = 0, | 472 | DLM_MASTER_RESP_NO = 0, |
473 | DLM_MASTER_RESP_YES, | 473 | DLM_MASTER_RESP_YES = 1, |
474 | DLM_MASTER_RESP_MAYBE, | 474 | DLM_MASTER_RESP_MAYBE = 2, |
475 | DLM_MASTER_RESP_ERROR | 475 | DLM_MASTER_RESP_ERROR = 3, |
476 | }; | 476 | }; |
477 | 477 | ||
478 | 478 | ||
@@ -649,9 +649,9 @@ struct dlm_proxy_ast | |||
649 | #define DLM_MOD_KEY (0x666c6172) | 649 | #define DLM_MOD_KEY (0x666c6172) |
650 | enum dlm_query_join_response_code { | 650 | enum dlm_query_join_response_code { |
651 | JOIN_DISALLOW = 0, | 651 | JOIN_DISALLOW = 0, |
652 | JOIN_OK, | 652 | JOIN_OK = 1, |
653 | JOIN_OK_NO_MAP, | 653 | JOIN_OK_NO_MAP = 2, |
654 | JOIN_PROTOCOL_MISMATCH, | 654 | JOIN_PROTOCOL_MISMATCH = 3, |
655 | }; | 655 | }; |
656 | 656 | ||
657 | struct dlm_query_join_packet { | 657 | struct dlm_query_join_packet { |