linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Btrfs: return -1 when lzo compression makes data bigger
@ 2013-07-01 18:33 Stefan Agner
  0 siblings, 0 replies; only message in thread
From: Stefan Agner @ 2013-07-01 18:33 UTC (permalink / raw)
  To: linux-btrfs

With this fix the lzo code behaves like the zlib code by returning an 
error
code when compression does not help reduce the size of the file.
This is currently not a bug since the compressed size is checked again 
in
the calling method compress_file_range.

Signed-off-by: Stefan Agner <stefan@agner.ch>
---
  fs/btrfs/lzo.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/fs/btrfs/lzo.c b/fs/btrfs/lzo.c
index f93151a..b6a6f07 100644
--- a/fs/btrfs/lzo.c
+++ b/fs/btrfs/lzo.c
@@ -207,8 +207,10 @@ static int lzo_compress_pages(struct list_head *ws,
  		}

  		/* we're making it bigger, give up */
-		if (tot_in > 8192 && tot_in < tot_out)
+		if (tot_in > 8192 && tot_in < tot_out) {
+			ret = -1;
  			goto out;
+		}

  		/* we're all done */
  		if (tot_in >= len)
-- 
1.8.3.1



^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2013-07-01 18:43 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-01 18:33 [PATCH] Btrfs: return -1 when lzo compression makes data bigger Stefan Agner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).