From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 38000C19759 for ; Mon, 5 Aug 2019 00:55:51 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 15D762133F for ; Mon, 5 Aug 2019 00:55:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726898AbfHEAzu (ORCPT ); Sun, 4 Aug 2019 20:55:50 -0400 Received: from mx1.redhat.com ([209.132.183.28]:39030 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726621AbfHEAzu (ORCPT ); Sun, 4 Aug 2019 20:55:50 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id ECB127FDCD; Mon, 5 Aug 2019 00:55:49 +0000 (UTC) Received: from ming.t460p (ovpn-8-20.pek2.redhat.com [10.72.8.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id BE24960922; Mon, 5 Aug 2019 00:55:38 +0000 (UTC) Date: Mon, 5 Aug 2019 08:55:33 +0800 From: Ming Lei To: Jens Axboe Cc: linux-block@vger.kernel.org, "Ewan D . Milne" , Bart Van Assche , Hannes Reinecke , Christoph Hellwig , Mike Snitzer , dm-devel@redhat.com, stable@vger.kernel.org Subject: Re: [PATCH V4 0/2] block/scsi/dm-rq: fix leak of request private data in dm-mpath Message-ID: <20190805005532.GA3449@ming.t460p> References: <20190725020500.4317-1-ming.lei@redhat.com> <20190730004359.GA28708@ming.t460p> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190730004359.GA28708@ming.t460p> User-Agent: Mutt/1.11.3 (2019-02-01) X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.27]); Mon, 05 Aug 2019 00:55:50 +0000 (UTC) Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org On Tue, Jul 30, 2019 at 08:43:59AM +0800, Ming Lei wrote: > On Thu, Jul 25, 2019 at 10:04:58AM +0800, Ming Lei wrote: > > Hi, > > > > When one request is dispatched to LLD via dm-rq, if the result is > > BLK_STS_*RESOURCE, dm-rq will free the request. However, LLD may allocate > > private data for this request, so this way will cause memory leak. > > > > Add .cleanup_rq() callback and implement it in SCSI for fixing the issue, > > since SCSI is the only driver which allocates private requst data in > > .queue_rq() path. > > > > Another use case of this callback is to free the request and re-submit > > bios during cpu hotplug when the hctx is dead, see the following link: > > > > https://lore.kernel.org/linux-block/f122e8f2-5ede-2d83-9ca0-bc713ce66d01@huawei.com/T/#t > > > > V4: > > - add more commit log on the new .cleanup_rq callback, as suggested > > by Mike > > > > V3: > > - run .cleanup_rq() from dm-rq because this issue is dm-rq specific, > > and even in future it should be still very unusual to free request > > in this way. If we call .cleanup_rq() in generic rq free code(fast > > path), cost will be introduced unnecessarily, also we have to > > consider related race. > > > > V2: > > - run .cleanup_rq() in blk_mq_free_request(), as suggested by Mike > > > > > > > > Ming Lei (2): > > blk-mq: add callback of .cleanup_rq > > scsi: implement .cleanup_rq callback > > > > drivers/md/dm-rq.c | 1 + > > drivers/scsi/scsi_lib.c | 13 +++++++++++++ > > include/linux/blk-mq.h | 13 +++++++++++++ > > 3 files changed, 27 insertions(+) > > > > Cc: Ewan D. Milne > > Cc: Bart Van Assche > > Cc: Hannes Reinecke > > Cc: Christoph Hellwig > > Cc: Mike Snitzer > > Cc: dm-devel@redhat.com > > Cc: > > Fixes: 396eaf21ee17 ("blk-mq: improve DM's blk-mq IO merging via blk_insert_cloned_request feedback") > > Hello Jens & guys, > > Ping on this fix. Hi Jens, Could you make the patcheset merged for 5.4? And it has been verified that big memory leak issue can be fixed by this patchset. thanks, Ming