From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1752923AbdKWPu0 (ORCPT <rfc822;w@1wt.eu>);
        Thu, 23 Nov 2017 10:50:26 -0500
Received: from mx1.redhat.com ([209.132.183.28]:18140 "EHLO mx1.redhat.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1752204AbdKWPuY (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 23 Nov 2017 10:50:24 -0500
Date: Thu, 23 Nov 2017 15:49:41 +0000
From: Alasdair G Kergon <agk@redhat.com>
To: Elena Reshetova <elena.reshetova@intel.com>
Cc: dm-devel@redhat.com, keescook@chromium.org, snitzer@redhat.com,
        peterz@infradead.org, shli@kernel.org, koverstreet@google.com,
        linux-kernel@vger.kernel.org, linux-raid@vger.kernel.org,
        linux-bcache@vger.kernel.org, ejt@redhat.comg,
        kent.overstreet@gmail.com, agk@redhat.com,
        Zdenek Kabelac <zkabelac@redhat.com>
Subject: Re: [dm-devel] [PATCH 3/4] dm: convert dm_dev_internal.count from
        atomic_t to refcount_t
Message-ID: <20171123154941.GA15790@agk-dp.fab.redhat.com>
Mail-Followup-To: Elena Reshetova <elena.reshetova@intel.com>,
        dm-devel@redhat.com, keescook@chromium.org, snitzer@redhat.com,
        peterz@infradead.org, shli@kernel.org, koverstreet@google.com,
        linux-kernel@vger.kernel.org, linux-raid@vger.kernel.org,
        linux-bcache@vger.kernel.org, ejt@redhat.comg,
        kent.overstreet@gmail.com, agk@redhat.com,
        Zdenek Kabelac <zkabelac@redhat.com>
References: <1508485059-21881-1-git-send-email-elena.reshetova@intel.com> <1508485059-21881-4-git-send-email-elena.reshetova@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1508485059-21881-4-git-send-email-elena.reshetova@intel.com>
Organization: Red Hat UK Ltd. Registered in England and Wales, number
        03798903. Registered Office: Peninsular House, 30-36 Monument
        Street, 4th Floor, London, England, EC3R 8NB.
User-Agent: Mutt/1.5.18 (2008-05-17)
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.39]); Thu, 23 Nov 2017 15:50:24 +0000 (UTC)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Oct 20, 2017 at 10:37:38AM +0300, Elena Reshetova wrote:
>  	} else if (dd->dm_dev->mode != (mode | dd->dm_dev->mode)) {
>  		r = upgrade_mode(dd, mode, t->md);
>  		if (r)
>  			return r;
> +		refcount_inc(&dd->count);
>  	}

Missing here:

        else
		refcount_inc(&dd->count);

?

Alasdair