From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:55690 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750922AbcJUCQh (ORCPT ); Thu, 20 Oct 2016 22:16:37 -0400 Date: Thu, 20 Oct 2016 22:16:35 -0400 From: Mike Snitzer To: tang.junhui@zte.com.cn Cc: agk@redhat.com, dm-devel@redhat.com, bmarzins@redhat.com, zhang.kai16@zte.com.cn, stable@vger.kernel.org Subject: Re: md/dm-table: tgt->type should be putted in dm_table_add_target() Message-ID: <20161021021635.GA7605@redhat.com> References: <1477013732-4800-1-git-send-email-tang.junhui@zte.com.cn> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1477013732-4800-1-git-send-email-tang.junhui@zte.com.cn> Sender: stable-owner@vger.kernel.org List-ID: On Thu, Oct 20 2016 at 9:35pm -0400, tang.junhui@zte.com.cn wrote: > From: "tang.junhui" > > tgt->type should be putted in dm_table_add_target() > when the target do not statisfy the needs of target type, > otherwise it would cause the module reference count > of this target type leakage. > > Signed-off-by: tang.junhui > Cc: stable@vger.kernel.org This issue, missing dm_put_target_type, dates back to 2011. See these 3 commits: $ git log --oneline 3791e2fc0^..36a0456fb 36a0456 dm table: add immutable feature cc6cbe1 dm table: add always writeable feature 3791e2f dm table: add singleton feature So are you having problems with failed DM table loads (due to the errors below) resulting in the inability to unload a DM module? (BTW, you don't need to cc stable when initially proposing a patch for stable. The stable@ maintainers will automatically pull such stable@ changes in once they land in Linus' tree.) Mike > --- > drivers/md/dm-table.c | 24 ++++++++++++------------ > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c > index 6554d91..4f56c38 100644 > --- a/drivers/md/dm-table.c > +++ b/drivers/md/dm-table.c > @@ -698,30 +698,30 @@ int dm_table_add_target(struct dm_table *t, const char *type, > > if (dm_target_needs_singleton(tgt->type)) { > if (t->num_targets) { > - DMERR("%s: target type %s must appear alone in table", > - dm_device_name(t->md), type); > - return -EINVAL; > + tgt->error = "target type must appear alone in table"; > + r = -EINVAL; > + goto bad; > } > t->singleton = 1; > } > > if (dm_target_always_writeable(tgt->type) && !(t->mode & FMODE_WRITE)) { > - DMERR("%s: target type %s may not be included in read-only tables", > - dm_device_name(t->md), type); > - return -EINVAL; > + tgt->error = "target type may not be included in read-only tables"; > + r = -EINVAL; > + goto bad; > } > > if (t->immutable_target_type) { > if (t->immutable_target_type != tgt->type) { > - DMERR("%s: immutable target type %s cannot be mixed with other target types", > - dm_device_name(t->md), t->immutable_target_type->name); > - return -EINVAL; > + tgt->error = "immutable target type cannot be mixed with other target types"; > + r = -EINVAL; > + goto bad; > } > } else if (dm_target_is_immutable(tgt->type)) { > if (t->num_targets) { > - DMERR("%s: immutable target type %s cannot be mixed with other target types", > - dm_device_name(t->md), tgt->type->name); > - return -EINVAL; > + tgt->error = "immutable target type cannot be mixed with other target types"; > + r = -EINVAL; > + goto bad; > } > t->immutable_target_type = tgt->type; > } > -- > 2.8.1.windows.1 >