On Thu, Oct 2, 2014 at 6:24 PM, Hitoshi Mitake
<[email protected]> wrote:
> Previous tgtd could cause double locking as an ordinal action. But
> recent change
> (https://github.com/fujita/tgt/commit/744aa51940c10c7986fc3911eb3ea44d6116ed45)
> removed this confusing action. So this patch let sheep forbid the
> double locking.
>
> Signed-off-by: Hitoshi Mitake <[email protected]>
> ---
> sheep/vdi.c | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)
Applied.
Thanks,
Hitoshi
>
> diff --git a/sheep/vdi.c b/sheep/vdi.c
> index 5874c29..c8fdf4a 100644
> --- a/sheep/vdi.c
> +++ b/sheep/vdi.c
> @@ -363,13 +363,9 @@ static bool add_new_participant(struct vdi_state_entry
> *entry,
> if (node_id_cmp(&entry->participants[i], owner))
> continue;
>
> - /*
> - * already exist, VDI locking with reloading inode causes
> - * this case
> - */
> - sd_debug("%s is already locking %"PRIx32,
> node_id_to_str(owner),
> + sd_err("%s is already locking %"PRIx32, node_id_to_str(owner),
> entry->vid);
> - return true;
> + return false;
> }
>
> idx = entry->nr_participants++;
> --
> 1.8.3.2
>
--
sheepdog mailing list
[email protected]
http://lists.wpkg.org/mailman/listinfo/sheepdog
