On 03/31/2017 10:43 PM, Adam Borowski wrote: > On Fri, Mar 31, 2017 at 10:24:57PM +0200, Hans van Kranenburg wrote: >> >> Yes, but we're not doing the same thing here. >> >> You have a file via a loop mount. If I do that, I get the same output as >> you show, the right messages when I remount ssd and nossd. >> >> My test was lvm based on an ssd. When I mount that, I get the "detected >> SSD devices, enabling SSD mode", and everytime I remount, being it ssd >> or nossd, it *always* says "use ssd allocation scheme". >> >> So, this needs some more research I guess. It doesn't feel right. > > I can't reproduce: > > [~]# cat /proc/swaps > Filename Type Size Used Priority > /dev/sda2 partition 8822780 0 -1 > [~]# swapoff /dev/sda2 > [~]# mkfs.btrfs -f /dev/sda2 > ... > [ 2459.856819] BTRFS info (device sda2): detected SSD devices, enabling SSD > mode > [ 2459.857699] BTRFS info (device sda2): creating UUID tree > [ 2477.234868] BTRFS info (device sda2): not using ssd allocation scheme > [ 2477.234873] BTRFS info (device sda2): disk space caching is enabled > [ 2482.306649] BTRFS info (device sda2): use ssd allocation scheme > [ 2482.306654] BTRFS info (device sda2): disk space caching is enabled > [ 2483.618578] BTRFS info (device sda2): not using ssd allocation scheme > [ 2483.618583] BTRFS info (device sda2): disk space caching is enabled > > Same partition on lvm: > [ 2813.259749] BTRFS info (device dm-0): detected SSD devices, enabling SSD > mode > [ 2813.260586] BTRFS info (device dm-0): creating UUID tree > [ 2827.131076] BTRFS info (device dm-0): not using ssd allocation scheme > [ 2827.131081] BTRFS info (device dm-0): disk space caching is enabled > [ 2828.618841] BTRFS info (device dm-0): use ssd allocation scheme > [ 2828.618845] BTRFS info (device dm-0): disk space caching is enabled > [ 2829.546796] BTRFS info (device dm-0): not using ssd allocation scheme > [ 2829.546801] BTRFS info (device dm-0): disk space caching is enabled > [ 2833.770787] BTRFS info (device dm-0): use ssd allocation scheme > [ 2833.770792] BTRFS info (device dm-0): disk space caching is enabled > > Seems to flip back and forth correctly for me. > > Are you sure you have this patch applied?
Oh ok, that's with the patch. The output I show is without the patch. If it does my output without the patch instead and the right output with it applied, then the puzzle pieces are in the right place again. -- Hans van Kranenburg -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
