All of lore.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: Roberto Di Cosmo <roberto@dicosmo.org>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>, Pavel Machek <pavel@suse.cz>,
	Roberto Di Cosmo <Roberto.Di-Cosmo@pps.jussieu.fr>,
	linux-kernel@vger.kernel.org, demolinux@demolinux.org
Subject: Re: [isocompr PATCH]: first comparison with HPA's zisofs
Date: Sun, 17 Jun 2001 10:03:10 -0700	[thread overview]
Message-ID: <3B2CE2CE.7020705@zytor.com> (raw)
In-Reply-To: <20010611225944.B959@bug.ucw.cz>	<E159r4y-0001bR-00@the-village.bc.nu> <15148.49603.742951.360288@beryllium.pps.jussieu.fr>

Roberto Di Cosmo wrote:

>
> 
> I have only the following (minor) criticisms 
> 
> - the transparent compression scheme does not rely on a special
>   filename extension (it was .gZ in isocompr): a file foo gets
>   compressed to a file foo, and the only way to see if foo is
>   compressed or not is to read the header. This has pros and cons...
>   and I wonder what the reasons of this choice are.
> 


It caused ALL kinds of nastiness; the chosen solution was vastly simpler 
on a whole bunch of axes.


> - the tools allow to compress/decompress only a whole directory tree,
>   while it should be possible to act on a single file also: in DemoLinux
>   not all files are compressed (some must be readable under (hem...) other
>   less interesting OSs for example ;-)) and the distinction is not on
>   a per-directory basis.
>   [easy to fix, see patch at the end of this message: I did this to
>   be able to try zisofs with DemoLinux]
> 


You can do this by having the compressed and uncompressed files in 
different directory trees and merge them using mkisofs.  I personally 
think that's a cleaner solution, even if your suggestion might make 
sense anyway.  Your patch, though, is too ugly to live.


> - it seems to me that this was written with 2.4.x in mind, and I did not
>   find a version for 2.2.x kernels :-(
> 
> Now I wonder, if zisofs is going to be included into 2.5 (I would strongly
> vote in favour!), would it be worthwhile to include a compatibility mode
> to read the isocompr blocksized format too?
> 


No.  isocompr was misdesigned, and such a compatibility mode would 
needlessly complicate everything.

	-hpa




      reply	other threads:[~2001-06-17 17:03 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-06-11  9:29 [isocompr PATCH]: announcing stable port to kernel 2.2.18 Roberto Di Cosmo
2001-06-11 20:59 ` Pavel Machek
2001-06-12  9:53   ` Ingo Oeser
2001-06-12 16:31   ` Alan Cox
2001-06-13 14:38     ` Roberto Di Cosmo
2001-06-17 14:42     ` [isocompr PATCH]: first comparison with HPA's zisofs Roberto Di Cosmo
2001-06-17 17:03       ` H. Peter Anvin [this message]

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=3B2CE2CE.7020705@zytor.com \
    --to=hpa@zytor.com \
    --cc=Roberto.Di-Cosmo@pps.jussieu.fr \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=demolinux@demolinux.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@suse.cz \
    --cc=roberto@dicosmo.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.