AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:
CFG:
PT:0
S:andy.sharp@lsi.com
RQ:
SSV:mhbs.lsil.com
NSV:
SSH:
R:<Brian.Stark@lsi.com>,<Jobi.Ariyamannil@lsi.com>
MAID:2
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#mh/Mailbox/perfarce	0	WEBMAILe7zpMfntA5ZR000693f5@mail.onstor.com
X-Sylpheed-End-Special-Headers: 1
Date: Wed, 7 Oct 2009 14:21:16 -0700
From: Andrew Sharp <andy.sharp@lsi.com>
To: "Stark, Brian" <Brian.Stark@lsi.com>
Cc: Jobi Ariyamannil <Jobi.Ariyamannil@lsi.com>
Subject: Re: PERFORCE change 33483 for review
Message-ID: <20091007142116.0be0cece@ripper.onstor.net>
In-Reply-To: <WEBMAILe7zpMfntA5ZR000693f5@mail.onstor.com>
References: <WEBMAILe7zpMfntA5ZR000693f5@mail.onstor.com>
Organization: LSI
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

OK,

All right, they checked in this stupid crap that we will never use and
is the wrong approach anyway*.  Can we now re-task these resources to
some of our important near term StorFS items like snapshot
implementation and mirroring and whatnot?  That we badly need ASAP?
This is a chance to show that we can make the hard decisions as opposed
to waffling around coddling certain dysfunctional egos.  Which I
normally wouldn't care about except that it's costing us too much in
this case.

* Talk about not having any design documents, schedules, design
reviews, requirements reviews or anything else....Some userland server
complying with fs-api?  What-the-deuce?  Tuxstor is the beginning of
the end for fs-api.  The next logical step is to port StorFS and
NFS/CIFS to the Linux FS framework, and toss fs-api on the scrap heap of
history.

* We will never use this.  Btrfs will be shippable by the time this is
useable.  We'll be stuck with a fork of an open source project (ZFS)
that we alone maintain the synchronization with both upstream (ZFS and
Linux kernel) projects.  And it will be one big loser compared to btrfs.


On Wed, 7 Oct 2009 11:50:00 -0600 amitra <Anand.Mitra@lsi.com> wrote:

> Change 33483 by amitra@amitra-2 on 2009/10/07 10:47:27
> 
> 	A zfs based userland server to serve fs requests as defined
> in fs-api.h Reviewed by: maximk
> 
> Affected files ...
> 
> ... //depot/dev/opensol/docs/zfs-onstor.tex#2 edit
> ... //depot/dev/opensol/usr/src/cmd/zsrv/Makefile#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/Makefile.com#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-api.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-compat.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-context.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-context.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-dir.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-dir.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-hdrs.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-nfs.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-nfs.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-port.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-security.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-security.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-status.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-zil.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-znode.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/fs-znode.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/i386/Makefile#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/inc.flg#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/nfx-defs.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/req.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/utils-fs-api.h#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/zcli.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/zsrv.c#1 add
> ... //depot/dev/opensol/usr/src/cmd/zsrv/zsrv.h#1 add
> ... //depot/dev/opensol/usr/src/lib/libzpool/Makefile.com#2 edit
> ... //depot/dev/opensol/usr/src/uts/common/fs/zfs/dmu.c#2 edit
> ... //depot/dev/opensol/usr/src/uts/common/fs/zfs/dsl_dataset.c#2 edit
> ... //depot/dev/opensol/usr/src/uts/common/fs/zfs/sys/dmu.h#2 edit
> ... //depot/dev/opensol/usr/src/uts/common/fs/zfs/sys/dsl_dataset.h#2
> edit ... //depot/dev/opensol/usr/src/uts/common/fs/zfs/sys/zap.h#2
> edit ... //depot/dev/opensol/zfs-port/Includes/bqueue.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/linux/ipm-if.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/linux/nfxdns-inlines.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/nfx-defs.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/nfx-error.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/nfx-incl.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/nfx-limits.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/nfx-ports.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/nfx-target.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/nfx-types.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/openbsd/ipm-if.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/openbsd/nfxdns-inlines.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/opt-msgs.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/alloca.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/assert.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/cop1.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/ctype.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/errno.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/fcntl.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/lr33000.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/lr33020.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/malloc.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/math.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/mips_opcode.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/netio.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/qtc_ep.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/queue.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/r3041.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/r3081.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/regdef.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/setjmp.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/signal.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/stdarg.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/stddef.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/stdio.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/string.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/termio.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/types.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/pmon/unistd.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/smp-api.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/amd29f040.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/bmfpga.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/cache.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/exec_elf.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/gt64240r.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/gt64440r.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/i386.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/led-api.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/machine-dep.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/mips.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/ns16550.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_draminit.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_hr.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_hsp.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_ht.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_int.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_l2c.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_mc.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_pci.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_pm.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_regs.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_rld.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_scd.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/bcm1480_uart.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_defs.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_dma.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_draminit.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_genbus.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_int.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_jtag.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_l2c.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_ldt.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_mac.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_mc.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_pci.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_regs.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_scd.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_smbus.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_syncser.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_uart.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sb1250_wid.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/sb1250/sbmips.h#1
> add ... //depot/dev/opensol/zfs-port/Includes/sys/st16552.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/sys/tx3927.h#1 add
> ... //depot/dev/opensol/zfs-port/Includes/test/neteee/eee.h#1 add
> ... //depot/dev/opensol/zfs-port/req.h#1 add
> ... //depot/dev/opensol/zfs-port/sm-anpssc/anpssc-api.h#1 add
> ... //depot/dev/opensol/zfs-port/sm-debug/Makefile#1 add
> ... //depot/dev/opensol/zfs-port/sm-debug/debug.c#1 add
> ... //depot/dev/opensol/zfs-port/sm-debug/debug.h#1 add
> ... //depot/dev/opensol/zfs-port/sm-eee/eee-error.h#1 add
> ... //depot/dev/opensol/zfs-port/sm-rmc/rmc_common.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-cluster/cluster-api.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-nfxnis/nfxdns-api.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/1#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/Makefile#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/Makefile.1#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/Makefile.client#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/Makefile.lib#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/Makefile.org#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/Makefile.server#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/app_ids.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/R1_1_changes.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/close.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/errors.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/examples.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/large_xter.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/mixedop.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/rmc.doc#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/session.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/design/ssc-conv.txt#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/echo-client#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/echo-client.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/echo-server#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/echo-server.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/onstor.services#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/out#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_api.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_apps.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_debug.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_internal.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_mp.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_mp.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_pm.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_rpc_api.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_rpc_api.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_rpc_internal.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_rpc_ssc.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_rpc_ssc.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_ssc.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_ssc_api.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_ssc_util.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_test.h#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/rmc_util.c#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/services#1 add
> ... //depot/dev/opensol/zfs-port/ssc-rmc/test.sh#1 add
> ... //depot/dev/opensol/zfs-port/test.sh#1 add
> ... //depot/dev/opensol/zfs-port/zcli.c#1 add
> ... //depot/dev/opensol/zfs-port/zcli/Makefile#1 add
> ... //depot/dev/opensol/zfs-port/zcli/req.h#1 add
> ... //depot/dev/opensol/zfs-port/zcli/zcli.c#1 add
> ... //depot/dev/opensol/zfs-port/zcli/zserv.c#1 add
> ... //depot/dev/opensol/zfs-port/ztest.c#1 add
> 
> 
> http://liszt.onstor.net:1818/@md=d&cd=//depot/$c=G35@/33483?ac=10
