From mboxrd@z Thu Jan 1 00:00:00 1970 References: <4972578.pTZnC5dHRY@walnut> <2262429.nQIR6vq4BO@walnut> <5115744.4jzXO94mnC@walnut> From: Zdenek Kabelac Message-ID: <8d90a730-8985-7e7d-fdcd-0db609bb36b2@redhat.com> Date: Tue, 22 Sep 2020 12:59:55 +0200 MIME-Version: 1.0 In-Reply-To: <5115744.4jzXO94mnC@walnut> Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [linux-lvm] Why isn't issue_discards enabled by default? 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: nl6720 , LVM general discussion and development Dne 22. 09. 20 v 12:38 nl6720 napsal(a): > On Tuesday, 22 September 2020 13:15:19 EEST Zdenek Kabelac wrote: >> Dne 22. 09. 20 v 11:14 nl6720 napsal(a): >>> On Monday, 21 September 2020 21:51:39 EEST Zdenek Kabelac wrote: >>>> Dne 21. 09. 20 v 16:14 nl6720 napsal(a): >>>>> Hi, >> thin-pool is using LVs - so this is again about handling the discard >> on a _tdata LV and it is completely unrelated to issue_discards >> setting. > > from lvmthin(7): > "passdown: Process discards in the thin pool (as with nopassdown), and > pass the discards down the the underlying device. This is the default > mode." > > It's the "underlying device" that's confusing me. All it means is - _tdata LV is placed on some PV/device. So the passdown means - TRIM goes through thinLV to thin-pool and then through its _tdata LV it lands on 'underlying device' aka PV used for this LV. If there is better and less confusing way how to describe this case, feel free to submit better wording. Zdenek