public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Edward Shishkin <edward@namesys.com>
To: Andrew James Wade 
	<ajwade@cpe00095b3131a0-cm0011ae8cd564.cpe.net.cable.rogers.com>
Cc: Hans Reiser <reiser@namesys.com>, Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org, reiserfs-dev@namesys.com
Subject: Re: [PATCH] Fix Reiser4 Dependencies
Date: Tue, 21 Jun 2005 15:27:07 +0400	[thread overview]
Message-ID: <42B7F98B.5050405@namesys.com> (raw)
In-Reply-To: <200506201644.10429.ajwade@cpe00095b3131a0-cm0011ae8cd564.cpe.net.cable.rogers.com>

Andrew James Wade wrote:

>>Andrew James Wade wrote:
>>
>>    
>>
>>>*    ZLIB_INFLATE is not visible in menuconfig. Reiser4 should probably
>>>    just select it like the other filesystems do.
>>>      
>>>
>The issue I had here was that Reiser4 wasn't appearing in menuconfig for
>me as I did not have ZLIB_INFLATE set, and the ZLIB_INFLATE option wasn't
>appearing either (I think it's because it doesn't have a prompt). I just
>followed the pattern for all other filesystems referencing ZLIB_INFLATE,
>and the comment in lib/Kbuild suggests this is the proper approach:
>
>| #
>| # compression support is select'ed if needed
>| #
>
>  
>
>>>*    Reiser4 also depends on ZLIB_DEFLATE.
>>>      
>>>
>I was getting linker errors:
>
>| fs/built-in.o(.text+0x8f465): In function `gzip1_alloc':
>| fs/reiser4/plugin/compress/compress.c:59: undefined reference to `zlib_deflate_workspacesize'
>| fs/built-in.o(.text+0x8f475): In function `gzip1_alloc':
>| include/asm/string.h:361: undefined reference to `zlib_deflate_workspacesize'
>| fs/built-in.o(.text+0x8f64e): In function `gzip1_compress':
>| fs/reiser4/plugin/compress/compress.c:167: undefined reference to `zlib_deflateInit2_'
>| fs/built-in.o(.text+0x8f661):fs/reiser4/plugin/compress/compress.c:174: undefined reference to `zlib_deflateReset'
>| fs/built-in.o(.text+0x8f697):fs/reiser4/plugin/compress/compress.c:184: undefined reference to `zlib_deflate'
>
>select'ing ZLIB_DEFLATE fixed them.
>
>Reiser4 is working for me after the two changes.
>
>HTH,
>Andrew
>  
>

Hello,
ZLIB_INFLATE/DEFLATE  will be selected by special reiser4 related 
configuration
option "Enable reiser4 compression plugins of gzip family" 
(REISER4_ZLIB), but
since this kind of support was discussed, it is in our working 
repository for a while..

Anyway thanks,
Edward.

>
>  
>
>>>signed-off-by: Andrew Wade <ajwade@alumni.uwaterloo.ca>
>>>
>>>--- 2.6.12-mm1/fs/reiser4/Kconfig	2005-06-20 07:38:22.087653000 -0400
>>>+++ linux/fs/reiser4/Kconfig	2005-06-20 08:01:28.914324250 -0400
>>>@@ -1,6 +1,8 @@
>>>config REISER4_FS
>>>	tristate "Reiser4 (EXPERIMENTAL)"
>>>-	depends on EXPERIMENTAL && !4KSTACKS && ZLIB_INFLATE
>>>+	depends on EXPERIMENTAL && !4KSTACKS
>>>+	select ZLIB_INFLATE
>>>+	select ZLIB_DEFLATE
>>>	help
>>>	  Reiser4 is a filesystem that performs all filesystem operations
>>>	  as atomic transactions, which means that it either performs a
>>>      
>>>
>
>
>  
>


  reply	other threads:[~2005-06-21 11:50 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-20  6:30 2.6.12-mm1 Andrew Morton
2005-06-20  6:57 ` 2.6.12-mm1 Benjamin Herrenschmidt
2005-06-20  8:54   ` 2.6.12-mm1 Ethan Benson
2005-06-20  7:46 ` 2.6.12-mm1 Brice Goglin
2005-06-20  8:14   ` 2.6.12-mm1 Dominik Brodowski
2005-06-20  8:49     ` 2.6.12-mm1 Brice Goglin
     [not found]       ` <20050620085449.GA32330@isilmar.linta.de>
2005-06-20 13:11         ` 2.6.12-mm1 Brice Goglin
2005-06-22 23:34           ` 2.6.12-mm1 Rajesh Shah
2005-06-23  6:25             ` 2.6.12-mm1 Brice Goglin
2005-06-23 17:05               ` 2.6.12-mm1 Rajesh Shah
2005-06-23 18:09                 ` 2.6.12-mm1 Brice Goglin
2005-06-23 20:32                   ` 2.6.12-mm1 Rajesh Shah
2005-06-23 21:07                     ` 2.6.12-mm1 Brice Goglin
2005-06-23 21:33                       ` 2.6.12-mm1 Brice Goglin
2005-06-20  7:59 ` 2.6.12-mm1 Brice Goglin
2005-06-21 12:40   ` 2.6.12-mm1 Brice Goglin
2005-06-21 16:27     ` 2.6.12-mm1 Takashi Iwai
2005-06-21 16:32       ` 2.6.12-mm1 Takashi Iwai
2005-06-21 17:02       ` 2.6.12-mm1 Brice Goglin
2005-06-21 17:16         ` 2.6.12-mm1 Takashi Iwai
2005-06-21 19:39           ` 2.6.12-mm1 Brice Goglin
2005-06-22 10:10             ` 2.6.12-mm1 Takashi Iwai
2005-06-20  9:48 ` [patch] 2.6.12-mm1: saa7134-core.c compile error Adrian Bunk
2005-06-20 22:19   ` Michael Krufky
2005-06-20 11:36 ` [PATCH] bttv fix [was: 2.6.12-mm1] J.A. Magallon
2005-06-20 11:38 ` 2.6.12-mm1 J.A. Magallon
2005-06-20 16:36   ` 2.6.12-mm1 Jeff Garzik
2005-06-20 22:35     ` 2.6.12-mm1 J.A. Magallon
2005-06-20 12:32 ` [PATCH] Fix Reiser4 Dependencies Andrew James Wade
2005-06-20 18:26   ` Hans Reiser
2005-06-20 20:44     ` Andrew James Wade
2005-06-21 11:27       ` Edward Shishkin [this message]
2005-06-21 18:47         ` Hans Reiser
2005-06-21 19:26           ` Andrew James Wade
2005-06-22  8:56             ` Edward Shishkin
2005-06-22  8:39           ` Edward Shishkin
2005-06-20 13:14 ` 2.6.12-mm1 Andy Whitcroft
2005-06-21 22:55   ` 2.6.12-mm1 Greg KH
2005-06-22  9:23     ` 2.6.12-mm1 Andy Whitcroft
2005-06-22 10:04       ` 2.6.12-mm1 Ivan Kokshaysky
2005-06-20 13:21 ` 2.6.12-mm1: Kernel BUG at "fs/open.c":935 Rafael J. Wysocki
2005-06-20 13:54   ` Dipankar Sarma
2005-06-20 14:05     ` Rafael J. Wysocki
2005-06-20 17:38 ` 2.6.12-mm1: drivers/misc/ibmasm/ compile error Adrian Bunk
2005-06-20 20:24   ` Andrew Morton
2005-06-20 18:29 ` 2.6.12-mm1 Jean Delvare
2005-06-20 20:41   ` 2.6.12-mm1 Andrew Morton
2005-06-20 21:11     ` 2.6.12-mm1 Jean Delvare
2005-06-20 21:23       ` 2.6.12-mm1 Andrew Morton
2005-06-20 21:42         ` 2.6.12-mm1 Jean Delvare
2005-06-20 21:53           ` 2.6.12-mm1 Andrew Morton
2005-06-21 20:06           ` 2.6.12-mm1 Mauro Carvalho Chehab
2005-06-21 21:25             ` 2.6.12-mm1 Jean Delvare
2005-06-20 18:39 ` gregkh-usb-usb-isp116x-hcd-add.patch (was 2.6.12-mm1) Alexey Dobriyan
2005-06-20 22:15 ` 2.6.12-mm1 J.A. Magallon
2005-06-20 22:34   ` iptables bug (was: Re: 2.6.12-mm1) Andrew Morton
2005-06-20 23:39     ` iptables bug Patrick McHardy
2005-06-21 19:21       ` Stephen Jones
2005-06-22  1:52         ` Patrick McHardy
2005-06-21 22:09 ` [PATCH] more signed char cleanups in scripts J.A. Magallon
2005-06-21 22:18   ` Andrew Morton
2005-06-21 22:59     ` J.A. Magallon
2005-06-22 11:46       ` Paulo Marques
2005-06-22  0:05     ` Lee Revell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=42B7F98B.5050405@namesys.com \
    --to=edward@namesys.com \
    --cc=ajwade@cpe00095b3131a0-cm0011ae8cd564.cpe.net.cable.rogers.com \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=reiser@namesys.com \
    --cc=reiserfs-dev@namesys.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox