From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx1.redhat.com (ext-mx05.extmail.prod.ext.phx2.redhat.com [10.5.110.29]) by int-mx13.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t7JAcO2Y013917 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for ; Wed, 19 Aug 2015 06:38:24 -0400 Received: from mail-wi0-f169.google.com (mail-wi0-f169.google.com [209.85.212.169]) by mx1.redhat.com (Postfix) with ESMTPS id 91747347A73 for ; Wed, 19 Aug 2015 10:38:23 +0000 (UTC) Received: by wicja10 with SMTP id ja10so116416794wic.1 for ; Wed, 19 Aug 2015 03:38:22 -0700 (PDT) Message-ID: <55D45D08.3060508@siteground.com> Date: Wed, 19 Aug 2015 13:40:08 +0300 From: vaLentin chernoZemski MIME-Version: 1.0 References: <55C86723.4000707@siteground.com> <20150810174900.GC22719@redhat.com> <55C9AD4F.5050900@siteground.com> <20150811143526.GB29369@redhat.com> In-Reply-To: <20150811143526.GB29369@redhat.com> Content-Transfer-Encoding: 7bit Subject: Re: [linux-lvm] lvremove does not pass discards if volume is part of thin pool Reply-To: LVM general discussion and development List-Id: LVM general discussion and development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , List-Id: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Mike Snitzer Cc: Peter Rajnoha , linux-lvm@redhat.com Hello Mike, Peter, I just tested my case with LVM2.2.02.128 with the latest kernel 4.1.6 and it still does not work. To avoid overloading this thread with logs information was added in the following paste: http://pastebin.com/7qxmUCyb Let me know if you need anything else that could help us debug this issue and fix it. Thank you in advance. Sincerely, vaLentin On 08/11/15 17:35, Mike Snitzer wrote: > On Tue, Aug 11 2015 at 4:07am -0400, > vaLentin chernoZemski wrote: > >>> Plese verify your kernel has this commit: >>> 19fa1a6756e ("dm thin: fix discard support to a previously shared block") >>> >>> But it doesn't look like you're using snapshots so this may not matter. >> >> The kernel we are using includes the changes listed in that commit. >> >>> If you do have the patch I referenced above then something else is going >>> on. You should probably run with: lvremove -vvvv to see if lvm is >>> actually issuing a discard. Or you could use blktrace to see if the >>> thin device you're removing is actually receiving a discard. >> >> lsblk -D shows DISC-ZERO as 1 for loop dev, thingroup_tmeta and >> thingroup_tdata >> >> However it shows DISC-ZERO as 0 for thingroup-tpool in both tmeta >> and tdata sections and all its child devices. > > DISC-ZERO is discard_zeroes_data. DM thinp disables that. It doesn't > mean discard aren't enabled. DISC-MAX and DISC-GRAN would need to be 0 > for discards to be disabled. > >> It appears to me that for some reason device mapper or kernel (not >> sure which should do that) are not advertising _tpool_ tmeta and >> tdata devices to support discards (as confirmed by lsblk). That's >> why during lvremove lvm skips issuing discards on those devices. > > Nope, that isn't it. The pool and thin device are advertising > discards. You should verify that the pool is configured to passdown > discards to the underlying loop device. Run: dmsetup table > You should see 'discard_passdown' -- which gets enabled by default if > the thin-pool's underlying data device supports discards. > >> The only references in lvremove -f -vvvv that are stating discards are those >> >> #libdm-deptree.c:2681 Suppressed testgroup-thingroup (253:36) >> identical table reload. >> #ioctl/libdm-iface.c:1795 dm status (253:35) ON [16384] (*1) >> #libdm-deptree.c:1444 Thin pool transaction id: 3 status: 3 >> 32/2560 1679/160928 - rw discard_passdown. >> #ioctl/libdm-iface.c:1795 dm message (253:35) OF delete 1 >> [16384] (*1) >> #ioctl/libdm-iface.c:1795 dm message (253:35) OF >> set_transaction_id 3 4 [16384] (*1) >> #ioctl/libdm-iface.c:1795 dm status (253:35) ON [16384] (*1) >> #libdm-deptree.c:1444 Thin pool transaction id: 4 status: 4 >> 18/2560 0/160928 - rw discard_passdown. >> #activate/dev_manager.c:3127 Creating CLEAN tree for thingroup. >> #activate/dev_manager.c:1789 Getting device info for >> testgroup-thingroup [LVM-qg1G3n02Kkjm0KKnhGhzP7JfoeGiiemlrsYfP0Ti5MCUiiPOWhTxoyRlvclhd3EH-pool] >> #ioctl/libdm-iface.c:1795 dm info LVM-qg1G3n02Kkjm0KKnhGhzP7JfoeGiiemlrsYfP0Ti5MCUiiPOWhTxoyRlvclhd3EH-pool >> OF [16384] (*1) >> #ioctl/libdm-iface.c:1795 dm deps (253:36) OF [16384] (*1) >> #ioctl/libdm-iface.c:1795 dm deps (253:35) OF [16384] (*1) >> #ioctl/libdm-iface.c:1795 dm deps (253:34) OF [16384] (*1) >> #ioctl/libdm-iface.c:1795 dm deps (253:33) OF [16384] (*1) > > Looking above it is clear that discard_passdown _is_ enabled. > > I'll have to defer to the lvm2 developers, I thought we added explicit > logging when lvm2 issues discards (as part of lvremove, etc) -- Peter, > and/or Alasdair? >