Attachment 13602 Details for Bug 13029 – Patches for v4-6

[patch] Patches for v4-6

BZ13029-v4-6.patch (text/plain), 1.30 KB, created by Amitay Isaacs on 2017-09-15 01:16:26 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Amitay Isaacs

Created: 2017-09-15 01:16:26 UTC

Size: 1.30 KB

patch

obsolete

>From 507686740e629e2bc6e54efc6354102d106d3168 Mon Sep 17 00:00:00 2001
>From: Amitay Isaacs <amitay@gmail.com>
>Date: Mon, 11 Sep 2017 15:59:19 +1000
>Subject: [PATCH] ctdb-daemon: Free up record data if a call request is
> deferred
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=13029
>
>If a call request for a key (migration request) is in flight, then all
>the subsequent call requests for the same key are deferred.  In that case,
>the data corresponding to key read from the local tdb is useless and there
>is no need to keep it around.  Once the deferred call is reprocessed,
>the data corresponding to that key will be fetched again.
>
>Signed-off-by: Amitay Isaacs <amitay@gmail.com>
>Reviewed-by: Martin Schwenke <martin@meltin.net>
>(cherry picked from commit 77c17b03cfc4734142fd86ba3cdd9663e75f34e3)
>---
> ctdb/server/ctdb_daemon.c | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/ctdb/server/ctdb_daemon.c b/ctdb/server/ctdb_daemon.c
>index 122d884a0c9..4d6987a121e 100644
>--- a/ctdb/server/ctdb_daemon.c
>+++ b/ctdb/server/ctdb_daemon.c
>@@ -675,6 +675,7 @@ static void daemon_request_call_from_client(struct ctdb_client *client,
> 				DEBUG(DEBUG_ERR,(__location__ " ctdb_ltdb_unlock() failed with error %d\n", ret));
> 			}
> 			CTDB_DECREMENT_STAT(ctdb, pending_calls);
>+			talloc_free(data.dptr);
> 			return;
> 		}
> 	}
>-- 
>2.13.5
>

Flags:

martins: review+

Actions: View

Attachments on bug 13029: 13602 | 13603