From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.kernel.org ([198.145.29.99]:47552 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727999AbeKQARy (ORCPT ); Fri, 16 Nov 2018 19:17:54 -0500 Date: Fri, 16 Nov 2018 06:05:21 -0800 From: Greg Kroah-Hartman To: Ming Lei Cc: Jens Axboe , linux-block@vger.kernel.org, "jianchao.wang" , Guenter Roeck , stable@vger.kernel.org Subject: Re: [PATCH V2 for-4.21 1/2] blk-mq: not embed .mq_kobj and ctx->kobj into queue instance Message-ID: <20181116140521.GB4595@kroah.com> References: <20181116112311.4117-1-ming.lei@redhat.com> <20181116112311.4117-2-ming.lei@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181116112311.4117-2-ming.lei@redhat.com> Sender: stable-owner@vger.kernel.org List-ID: On Fri, Nov 16, 2018 at 07:23:10PM +0800, Ming Lei wrote: > @@ -456,7 +456,7 @@ struct request_queue { > /* > * mq queue kobject > */ > - struct kobject mq_kobj; > + struct kobject *mq_kobj; What is this kobject even used for? It wasn't obvious at all from this patch, why is it needed if you are not using it to reference count the larger structure here? thanks, greg k-h