Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
SUSE:SLE-15-SP2:Update
pacemaker.26925
bsc#1181744-0004-Refactor-fencer-optimize-mergi...
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File bsc#1181744-0004-Refactor-fencer-optimize-merging-of-fencing-history-.patch of Package pacemaker.26925
From dbc27b2ac63ab6b7d2385da384c70d1c13e93369 Mon Sep 17 00:00:00 2001 From: "Gao,Yan" <ygao@suse.com> Date: Wed, 17 Feb 2021 11:36:50 +0100 Subject: [PATCH 4/4] Refactor: fencer: optimize merging of fencing history by removing unneeded entries on creation of history diff --- daemons/fenced/fenced_history.c | 161 ++++++++++++++++---------------- 1 file changed, 82 insertions(+), 79 deletions(-) Index: pacemaker-2.0.4+20200616.2deceaa3a/daemons/fenced/fenced_history.c =================================================================== --- pacemaker-2.0.4+20200616.2deceaa3a.orig/daemons/fenced/fenced_history.c +++ pacemaker-2.0.4+20200616.2deceaa3a/daemons/fenced/fenced_history.c @@ -260,7 +260,7 @@ stonith_xml_history_to_list(xmlNode *his /*! * \internal * \brief Craft xml difference between local fence-history and a history - * coming from remote + * coming from remote, and merge the remote history into the local * * \param[in] remote_history Fence-history as hash-table (may be NULL) * \param[in] add_id If crafting the answer for an API @@ -270,21 +270,23 @@ stonith_xml_history_to_list(xmlNode *his * \return The fence-history as xml */ static xmlNode * -stonith_local_history_diff(GHashTable *remote_history, +stonith_local_history_diff_and_merge(GHashTable *remote_history, gboolean add_id, const char *target) { xmlNode *history = NULL; + GHashTableIter iter; + remote_fencing_op_t *op = NULL; + gboolean updated = FALSE; int cnt = 0; if (stonith_remote_op_list) { - GHashTableIter iter; - remote_fencing_op_t *op = NULL; + char *id = NULL; history = create_xml_node(NULL, F_STONITH_HISTORY_LIST); g_hash_table_iter_init(&iter, stonith_remote_op_list); - while (g_hash_table_iter_next(&iter, NULL, (void **)&op)) { + while (g_hash_table_iter_next(&iter, (void **)&id, (void **)&op)) { xmlNode *entry = NULL; if (remote_history) { @@ -292,10 +294,25 @@ stonith_local_history_diff(GHashTable *r g_hash_table_lookup(remote_history, op->id); if (remote_op) { - if (remote_op->state != st_failed - && remote_op->state != st_done - && (op->state == st_failed - || op->state == st_done)) { + if (stonith__op_state_pending(op->state) + && !stonith__op_state_pending(remote_op->state)) { + + crm_debug("Updating outdated pending operation %.8s " + "(state=%d) according to the one (state=%d) from " + "remote peer history", + op->id, op->state, + remote_op->state); + + g_hash_table_steal(remote_history, op->id); + op->id = remote_op->id; + remote_op->id = id; + g_hash_table_iter_replace(&iter, remote_op); + + updated = TRUE; + continue; /* skip outdated entries */ + + } else if (!stonith__op_state_pending(op->state) + && stonith__op_state_pending(remote_op->state)) { crm_debug("Broadcasting operation %.8s (state=%d) to " "update the outdated pending one " @@ -303,7 +320,10 @@ stonith_local_history_diff(GHashTable *r op->id, op->state, remote_op->state); + g_hash_table_remove(remote_history, op->id); + } else { + g_hash_table_remove(remote_history, op->id); continue; /* skip entries broadcasted already */ } } @@ -329,6 +349,47 @@ stonith_local_history_diff(GHashTable *r } } + if (remote_history) { + init_stonith_remote_op_hash_table(&stonith_remote_op_list); + + updated |= g_hash_table_size(remote_history); + + g_hash_table_iter_init(&iter, remote_history); + while (g_hash_table_iter_next(&iter, NULL, (void **)&op)) { + + if (stonith__op_state_pending(op->state) && + safe_str_eq(op->originator, stonith_our_uname)) { + crm_warn("Failing pending operation %.8s originated by us but " + "known only from peer history", op->id); + op->state = st_failed; + op->completed = time(NULL); + /* use -EHOSTUNREACH to not introduce a new return-code that might + trigger unexpected results at other places and to prevent + remote_op_done from setting the delegate if not present + */ + stonith_bcast_result_to_peers(op, -EHOSTUNREACH, FALSE); + } + + g_hash_table_iter_steal(&iter); + g_hash_table_replace(stonith_remote_op_list, op->id, op); + /* we could trim the history here but if we bail + * out after trim we might miss more recent entries + * of those that might still be in the list + * if we don't bail out trimming once is more + * efficient and memory overhead is minimal as + * we are just moving pointers from one hash to + * another + */ + } + + g_hash_table_destroy(remote_history); /* remove what is left */ + } + + if (updated) { + stonith_fence_history_trim(); + do_stonith_notify(0, T_STONITH_NOTIFY_HISTORY, 0, NULL); + } + if (cnt == 0) { free_xml(history); return NULL; @@ -339,75 +400,18 @@ stonith_local_history_diff(GHashTable *r /*! * \internal - * \brief Merge fence-history coming from remote into local history + * \brief Craft xml from the local fence-history + * + * \param[in] add_id If crafting the answer for an API + * history-request there is no need for the id + * \param[in] target Optionally limit to certain fence-target * - * \param[in] history Hash-table holding remote history to be merged in + * \return The fence-history as xml */ -static void -stonith_merge_in_history_list(GHashTable *history) +static xmlNode * +stonith_local_history(gboolean add_id, const char *target) { - GHashTableIter iter; - remote_fencing_op_t *op = NULL; - gboolean updated = FALSE; - - if (!history) { - return; - } - - init_stonith_remote_op_hash_table(&stonith_remote_op_list); - - g_hash_table_iter_init(&iter, history); - while (g_hash_table_iter_next(&iter, NULL, (void **)&op)) { - remote_fencing_op_t *stored_op = - g_hash_table_lookup(stonith_remote_op_list, op->id); - - if (stored_op) { - if (stored_op->state != st_failed - && stored_op->state != st_done - && (op->state == st_failed || op->state == st_done)) { - crm_debug("Updating outdated pending operation %.8s " - "(state=%d) according to the one (state=%d) from " - "remote peer history", - op->id, stored_op->state, - op->state); - - } else { - continue; // Skip existent - } - } - - updated = TRUE; - g_hash_table_iter_steal(&iter); - - if ((op->state != st_failed) && - (op->state != st_done) && - safe_str_eq(op->originator, stonith_our_uname)) { - crm_warn("received pending action we are supposed to be the " - "owner but it's not in our records -> fail it"); - op->state = st_failed; - op->completed = time(NULL); - /* use -EHOSTUNREACH to not introduce a new return-code that might - trigger unexpected results at other places and to prevent - remote_op_done from setting the delegate if not present - */ - stonith_bcast_result_to_peers(op, -EHOSTUNREACH, FALSE); - } - - g_hash_table_insert(stonith_remote_op_list, op->id, op); - /* we could trim the history here but if we bail - * out after trim we might miss more recent entries - * of those that might still be in the list - * if we don't bail out trimming once is more - * efficient and memory overhead is minimal as - * we are just moving pointers from one hash to - * another - */ - } - stonith_fence_history_trim(); - if (updated) { - do_stonith_notify(0, T_STONITH_NOTIFY_HISTORY, 0, NULL); - } - g_hash_table_destroy(history); /* remove what is left */ + return stonith_local_history_diff_and_merge(NULL, add_id, target); } /*! @@ -463,7 +467,7 @@ stonith_fence_history(xmlNode *msg, xmlN * so that every node can merge and broadcast * what it has on top */ - out_history = stonith_local_history_diff(NULL, TRUE, NULL); + out_history = stonith_local_history(TRUE, NULL); crm_trace("Broadcasting history to peers"); stonith_send_broadcast_history(out_history, st_opt_broadcast | st_opt_discard_reply, @@ -488,7 +492,7 @@ stonith_fence_history(xmlNode *msg, xmlN !crm_is_true(crm_element_value(history, F_STONITH_DIFFERENTIAL))) { out_history = - stonith_local_history_diff(received_history, TRUE, NULL); + stonith_local_history_diff_and_merge(received_history, TRUE, NULL); if (out_history) { crm_trace("Broadcasting history-diff to peers"); crm_xml_add(out_history, F_STONITH_DIFFERENTIAL, @@ -500,7 +504,6 @@ stonith_fence_history(xmlNode *msg, xmlN crm_trace("History-diff is empty - skip broadcast"); } } - stonith_merge_in_history_list(received_history); } else { crm_trace("Skipping history-query-broadcast (%s%s)" " we sent ourselves", @@ -511,7 +514,7 @@ stonith_fence_history(xmlNode *msg, xmlN /* plain history request */ crm_trace("Looking for operations on %s in %p", target, stonith_remote_op_list); - *output = stonith_local_history_diff(NULL, FALSE, target); + *output = stonith_local_history(FALSE, target); } free_xml(out_history); return rc;
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor
