From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7D551311C01 for ; Thu, 7 May 2026 21:41:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190107; cv=none; b=c7aOW4ducqcBOEpcG6jp+k82T45md+AHAMAO+DlVJrl6sNa1mpVrs5tPGdpt76gnl3GhQ6H/TlgfwLE7MEJjpdhecxdwdWCSqvxvQFYT5yqc7OdFMeTQMSLW/mKR/i6L3JzyC+Cf+8VNSi39d+OOXockgU2aG0IKDy6tYAsuDvg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778190107; c=relaxed/simple; bh=U3a1Rvop2Bo7hkyN/DK56QhgCdGnBNh1cl29APUanZU=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=M+LvW9m7TWDLjGg/k8o3fcReOQXZHXcP92XyWJK81KQWWoyPhkg2k9Dj/2qOFatK2Z2AyqynLVfuDV5qA7IKRwBMhkiTWfJY3XHnIrFz4E1Y4wN1L7MyXGy8m1SooXGM9/AeS9rVhoSmMAfrMWQUxk8oZ+4kB9wfrf5b7AV0S+E= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=gJqPteKR; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=TgW/D+mB; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="gJqPteKR"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="TgW/D+mB" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1778190105; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=wm30c8FokIh/dw9WAqFPgr9dxyPUU9O9WsqvHE9ngCQ=; b=gJqPteKROkqsCWlRyXhZoC7NrQzTl5Ya7gKu8Unh5nGtPB5Z5r3ziOjzDP0DUknLv45us6 blGvH3odtjFZGj539527W8LXozstkxq8/NT2cb6dv04WC6QIHUEt8nOA8hLVZHhSxlPrzj Bhs0p/f8A7mDOAOTdU49A8MDN66cIeE= Received: from mail-yw1-f198.google.com (mail-yw1-f198.google.com [209.85.128.198]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-541-w4if1QPfNkqhjezm8w4Hdw-1; Thu, 07 May 2026 17:41:44 -0400 X-MC-Unique: w4if1QPfNkqhjezm8w4Hdw-1 X-Mimecast-MFC-AGG-ID: w4if1QPfNkqhjezm8w4Hdw_1778190104 Received: by mail-yw1-f198.google.com with SMTP id 00721157ae682-7bd5c267082so26386627b3.0 for ; Thu, 07 May 2026 14:41:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1778190103; x=1778794903; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=wm30c8FokIh/dw9WAqFPgr9dxyPUU9O9WsqvHE9ngCQ=; b=TgW/D+mBxPQJ1XkGnmUzUwSlrbtXO3cINEbqHoSkHJ2L8d2eeQlafXxZ/brPHfurSN ifBM6AzUzbyrNPR+Tbmfb1TdAdrDvK+GoiS/cWj30nuCL38cvtEqUgNqIZwEhEMcXYjn jQda/XXwFhZrTJTYaS/CrXC53H5NuMJuTsLIZEph6S0ZgKedbTHeDRkMhBfXiUjhjOs4 5vL9aAumwP98k/R4OYFkoPjT3wG1k9cXFOKwVxzPtiHYoygQdwaLeo2KM4meTQWU1Zbv rKnB+nwnsqEvxmzs0vGE7Jp7H9wNbwS0YcfFlPFACdIQ2bWc6EeuHCIUUm6bpBwVVoIj w3nQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778190103; x=1778794903; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=wm30c8FokIh/dw9WAqFPgr9dxyPUU9O9WsqvHE9ngCQ=; b=pPBUT9pYuBIPsdXkJ007hX3QyLkEBzYiaCVFt/Wjjj/feChHuvVfLZKiRZIT/DtOda dNzTvpDzT1CV5BBW5llom5YRDzqp6l9fX+faNqEe5g4ocThIKYuPZi/tl8fpNytAV1TU eZZ5UGWU2IjpzjXuyBlhlTRjIat1kpwR1T3rgoIxIj0gr+06kBPXDl33yyvojd+GhUKZ 2bsVHmOGQYQA9Oj7NzbkPtApHe8PwIn4w3cNZeRY3qy4vLcYumEGjA38v+4BwEobm1Qv YmS7f6ZqsZ7Qc6kgSdflHxpKguXzYw1Rh8GeikqKsdclZJQ6mXe5c5vZLLjYgRUqvXYj C8tw== X-Forwarded-Encrypted: i=1; AFNElJ81NpDS2Kt7mzDvMFRSPSgMfYN9MI6yH6mBgKLV5x/lsQkG1x7gGkx9y+/exzpVZlGBHMHF1YW62h16+QM=@vger.kernel.org X-Gm-Message-State: AOJu0Yws/KU9ihbvCzXr8WR3WX3TaimGvolLu5HX2liePPrOTCE7/vsj 7kzbetP8Ka5Uu4aF63Kpzf77ZnUy49FXLum5RxPMfJsuz8N+HED1aRAovZGujtxv8vHi2gIyjiV 6VYSQd/lNhStZXTw53vPDxOYwHiwYwQ9hWzwk1YZ8fziyEHZzN97/m/KAxR4kJZT7To4P243uHg == X-Gm-Gg: Acq92OFLDW2+/Fh5qip9SXfyQhGCRZQBdqj6NCx/1SXSXYGeHnufe56YTkz0aDNXMVC J9y2YuVOQ9tWRn4Y37nyCAF9QXEUpEfv0fkVCJxx9ZS9lGWJxAD8eERRh2MTc4XSxlZSCNO+fLR uDNz9gqnp7stMxYeqSr1xgl6jMMqStfQerfhLLZ4GtJprFv8JTCuw0Ele5+Mbd7UpOB/FRDLsg+ aEJ/4413Qqy5IqTTOJveoRCL5DCAlkRjs6AaR4GI8iE/dtNqiMq6aNCzIBOBGwDwoXshy41NdH3 K4fhQyYNm20iCrDv8zRfc0C02UbWxe5oNeuafYILsNp8gwU2sSwNMBcsKjVLGItViazLV12bCkG x4O5BYPaNNnHm17CvtVwZ6A1Bq1AaWVXL5nZMr3Y5c94dx2Gr0QpH X-Received: by 2002:a05:690c:e094:b0:7bb:c0f:19d4 with SMTP id 00721157ae682-7bdf5d7fff7mr108131597b3.4.1778190103095; Thu, 07 May 2026 14:41:43 -0700 (PDT) X-Received: by 2002:a05:690c:e094:b0:7bb:c0f:19d4 with SMTP id 00721157ae682-7bdf5d7fff7mr108131367b3.4.1778190102565; Thu, 07 May 2026 14:41:42 -0700 (PDT) Received: from li-4c4c4544-0032-4210-804c-c3c04f423534.ibm.com ([2600:1700:6476:1430::29]) by smtp.gmail.com with ESMTPSA id 00721157ae682-7bd66885907sm99820107b3.43.2026.05.07.14.41.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 07 May 2026 14:41:42 -0700 (PDT) Message-ID: Subject: Re: [PATCH] nilfs2: fix backing_dev_info reference leak From: Viacheslav Dubeyko To: Shuangpeng Bai , Ryusuke Konishi Cc: Viacheslav Dubeyko , Christian Brauner , linux-nilfs@vger.kernel.org, linux-kernel@vger.kernel.org Date: Thu, 07 May 2026 14:41:40 -0700 In-Reply-To: <20260507155021.941895-1-shuangpeng.kernel@gmail.com> References: <20260507155021.941895-1-shuangpeng.kernel@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.60.0 (3.60.0-1.fc44app2) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Thu, 2026-05-07 at 11:50 -0400, Shuangpeng Bai wrote: > setup_bdev_super() already initializes sb->s_bdev and takes a > reference on the block device backing_dev_info when assigning sb->s_bdi. >=20 > nilfs_fill_super() takes another reference to the same > backing_dev_info and stores it in sb->s_bdi again. The extra > reference is not paired with a matching bdi_put(), since > generic_shutdown_super() releases sb->s_bdi only once. >=20 > Drop the redundant bdi_get() in nilfs_fill_super(). The single > reference taken by setup_bdev_super() is enough and is released > during superblock shutdown. >=20 > Fixes: c1e012ea9e83 ("nilfs2: use setup_bdev_super to de-duplicate the mo= unt code") > Signed-off-by: Shuangpeng Bai > --- > fs/nilfs2/super.c | 2 -- > 1 file changed, 2 deletions(-) >=20 > diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c > index 7aa5ef8606cd..893a504cb80c 100644 > --- a/fs/nilfs2/super.c > +++ b/fs/nilfs2/super.c > @@ -1070,8 +1070,6 @@ nilfs_fill_super(struct super_block *sb, struct fs_= context *fc) > sb->s_time_gran =3D 1; > sb->s_max_links =3D NILFS_LINK_MAX; > =20 > - sb->s_bdi =3D bdi_get(sb->s_bdev->bd_disk->bdi); > - > err =3D load_nilfs(nilfs, sb); > if (err) > goto failed_nilfs; Makes sense to me. Reviewed-by: Viacheslav Dubeyko Thanks, Slava.