Re: Read before you deploy btrfs + zstd

2017-12-05 Thread Nick Terrell
> On Dec 5, 2017, at 7:54 AM, David Sterba wrote: > > I had a different branch with patches from openSUSE, so the diffs apply with > minimal efforts to the package. The branch btrfs-zstd has been synced up. The > ENOMEM error was not from the file decompression but from the

Re: Read before you deploy btrfs + zstd

2017-12-05 Thread David Sterba
On Wed, Nov 29, 2017 at 12:44:32AM +, Nick Terrell wrote: > >> It looks like XZ had the same issue, and they make the decompression > >> context a static object (grep for GRUB_EMBED_DECOMPRESSOR). We could > >> potentially do the same and statically allocate the workspace: > >> > >> ``` > >>

Re: Read before you deploy btrfs + zstd

2017-11-29 Thread Andrei Borzenkov
29.11.2017 16:24, Austin S. Hemmelgarn пишет: > On 2017-11-28 18:49, David Sterba wrote: >> On Tue, Nov 28, 2017 at 09:31:57PM +, Nick Terrell wrote: >>> On Nov 21, 2017, at 8:22 AM, David Sterba wrote: On Wed, Nov 15, 2017 at 08:09:15PM +, Nick Terrell

Re: Read before you deploy btrfs + zstd

2017-11-29 Thread Austin S. Hemmelgarn
On 2017-11-28 18:49, David Sterba wrote: On Tue, Nov 28, 2017 at 09:31:57PM +, Nick Terrell wrote: On Nov 21, 2017, at 8:22 AM, David Sterba wrote: On Wed, Nov 15, 2017 at 08:09:15PM +, Nick Terrell wrote: On 11/15/17, 6:41 AM, "David Sterba"

Re: Read before you deploy btrfs + zstd

2017-11-28 Thread Nick Terrell
> On Nov 28, 2017, at 3:49 PM, David Sterba wrote: > > On Tue, Nov 28, 2017 at 09:31:57PM +, Nick Terrell wrote: >> >>> On Nov 21, 2017, at 8:22 AM, David Sterba wrote: >>> >>> On Wed, Nov 15, 2017 at 08:09:15PM +, Nick Terrell wrote: On

Re: Read before you deploy btrfs + zstd

2017-11-28 Thread David Sterba
On Tue, Nov 28, 2017 at 09:31:57PM +, Nick Terrell wrote: > > > On Nov 21, 2017, at 8:22 AM, David Sterba wrote: > > > > On Wed, Nov 15, 2017 at 08:09:15PM +, Nick Terrell wrote: > >> On 11/15/17, 6:41 AM, "David Sterba" wrote: > >>> The branch is now

Re: Read before you deploy btrfs + zstd

2017-11-21 Thread David Sterba
On Wed, Nov 15, 2017 at 08:09:15PM +, Nick Terrell wrote: > On 11/15/17, 6:41 AM, "David Sterba" wrote: > > The branch is now in a state that can be tested. Turns out the memory > > requirements are too much for grub, so the boot fails with "not enough > > memory". The

Re: Read before you deploy btrfs + zstd

2017-11-16 Thread Dmitrii Tcvetkov
On Wed, 15 Nov 2017 20:23:44 + (UTC) Duncan <1i5t5.dun...@cox.net> wrote: > Tho from my understanding and last I read, btrfs restore (I believe > it was) hadn't been updated to handle zstd yet, tho btrfs check and > btrfs filesystem defrag had been, and of course btrfs balance if the > kernel

Re: Read before you deploy btrfs + zstd

2017-11-15 Thread Duncan
Austin S. Hemmelgarn posted on Tue, 14 Nov 2017 07:38:03 -0500 as excerpted: > On 2017-11-14 02:34, Martin Steigerwald wrote: >> Hello David. >> >> David Sterba - 13.11.17, 23:50: >>> while 4.14 is still fresh, let me address some concerns I've seen on >>> linux forums already. >>> >>> The newly

Re: Read before you deploy btrfs + zstd

2017-11-15 Thread Nick Terrell
On 11/15/17, 6:41 AM, "David Sterba" wrote: > The branch is now in a state that can be tested. Turns out the memory > requirements are too much for grub, so the boot fails with "not enough > memory". The calculated value > > ZSTD_BTRFS_MAX_INPUT: 131072 >

Re: Read before you deploy btrfs + zstd

2017-11-15 Thread Duncan
Imran Geriskovan posted on Wed, 15 Nov 2017 20:16:56 +0200 as excerpted: > On 11/15/17, Martin Steigerwald wrote: >> Somehow I am happy that I still have a plain Ext4 for /boot. :) > > You may use uncompressed btrfs for /boot. > Both Syslinux (my choice) and Grub supports

Re: Read before you deploy btrfs + zstd

2017-11-15 Thread Imran Geriskovan
On 11/15/17, Martin Steigerwald wrote: > Somehow I am happy that I still have a plain Ext4 for /boot. :) You may use uncompressed btrfs for /boot. Both Syslinux (my choice) and Grub supports it. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the

Re: Read before you deploy btrfs + zstd

2017-11-15 Thread Martin Steigerwald
David Sterba - 15.11.17, 15:39: > On Tue, Nov 14, 2017 at 07:53:31PM +0100, David Sterba wrote: > > On Mon, Nov 13, 2017 at 11:50:46PM +0100, David Sterba wrote: > > > Up to now, there are no bootloaders supporting ZSTD. > > > > I've tried to implement the support to GRUB, still incomplete and

Re: Read before you deploy btrfs + zstd

2017-11-15 Thread David Sterba
On Tue, Nov 14, 2017 at 07:53:31PM +0100, David Sterba wrote: > On Mon, Nov 13, 2017 at 11:50:46PM +0100, David Sterba wrote: > > Up to now, there are no bootloaders supporting ZSTD. > > I've tried to implement the support to GRUB, still incomplete and hacky > but most of the code is there. The

Re: Read before you deploy btrfs + zstd

2017-11-14 Thread Martin Steigerwald
David Sterba - 14.11.17, 19:49: > On Tue, Nov 14, 2017 at 08:34:37AM +0100, Martin Steigerwald wrote: > > Hello David. > > > > David Sterba - 13.11.17, 23:50: > > > while 4.14 is still fresh, let me address some concerns I've seen on > > > linux > > > forums already. > > > > > > The newly added

Re: Read before you deploy btrfs + zstd

2017-11-14 Thread David Sterba
On Mon, Nov 13, 2017 at 11:50:46PM +0100, David Sterba wrote: > Up to now, there are no bootloaders supporting ZSTD. I've tried to implement the support to GRUB, still incomplete and hacky but most of the code is there. The ZSTD implementation is copied from kernel. The allocators need to be

Re: Read before you deploy btrfs + zstd

2017-11-14 Thread David Sterba
On Tue, Nov 14, 2017 at 08:34:37AM +0100, Martin Steigerwald wrote: > Hello David. > > David Sterba - 13.11.17, 23:50: > > while 4.14 is still fresh, let me address some concerns I've seen on linux > > forums already. > > > > The newly added ZSTD support is a feature that has broader impact than

Re: Read before you deploy btrfs + zstd

2017-11-14 Thread Austin S. Hemmelgarn
On 2017-11-14 02:34, Martin Steigerwald wrote: Hello David. David Sterba - 13.11.17, 23:50: while 4.14 is still fresh, let me address some concerns I've seen on linux forums already. The newly added ZSTD support is a feature that has broader impact than just the runtime compression. The

RE: Read before you deploy btrfs + zstd

2017-11-14 Thread Paul Jones
> -Original Message- > From: linux-btrfs-ow...@vger.kernel.org [mailto:linux-btrfs- > ow...@vger.kernel.org] On Behalf Of Martin Steigerwald > Sent: Tuesday, 14 November 2017 6:35 PM > To: dste...@suse.cz; linux-btrfs@vger.kernel.org > Subject: Re: Read before you d

Re: Read before you deploy btrfs + zstd

2017-11-13 Thread Martin Steigerwald
Hello David. David Sterba - 13.11.17, 23:50: > while 4.14 is still fresh, let me address some concerns I've seen on linux > forums already. > > The newly added ZSTD support is a feature that has broader impact than > just the runtime compression. The btrfs-progs understand filesystem with > ZSTD