It works: at least it starts w/o errors

fuse/fuse.go

@@ -6,7 +6,6 @@ import (
 	"bytes"
 	"encoding/binary"
 	"fmt"
-	"log"
 	"net"
 	"os"
 	"path"
@@ -18,7 +17,7 @@ const (
 	bufSize = 66000
 )
 
-type FileSystem interface{
+type FileSystem interface {
 	Init(in *InitIn) (out *InitOut, code Error)
 }
 
@@ -29,7 +28,7 @@ type MountPoint struct {
 }
 
 // Mount create a fuse fs on the specified mount point.
-func Mount(mountPoint string, fs FileSystem) (m *MountPoint, err os.Error) {
+func Mount(mountPoint string, fs FileSystem) (m *MountPoint, err os.Error, errors chan os.Error) {
 	local, remote, err := net.Socketpair("unixgram")
 	if err != nil {
 		return
@@ -59,7 +58,8 @@ func Mount(mountPoint string, fs FileSystem) (m *MountPoint, err os.Error) {
 		return
 	}
 	if w.ExitStatus() != 0 {
-		return nil, os.NewError(fmt.Sprintf("fusermount exited with code %d\n", w.ExitStatus()))
+		err = os.NewError(fmt.Sprintf("fusermount exited with code %d\n", w.ExitStatus()))
+		return
 	}
 
 	f, err := getFuseConn(local)
@@ -67,27 +67,29 @@ func Mount(mountPoint string, fs FileSystem) (m *MountPoint, err os.Error) {
 		return
 	}
 	m = &MountPoint{mountPoint, f, fs}
-	go m.loop()
+	errors = make(chan os.Error, 100)
+	go m.loop(errors)
 	return
 }
 
-func (m *MountPoint) loop() {
+func (m *MountPoint) loop(errors chan os.Error) {
 	buf := make([]byte, bufSize)
 	f := m.f
-	errors := make(chan os.Error, 100)
+	defer close(errors)
 	toW := make(chan [][]byte, 100)
-	go m.errorHandler(errors)
+	defer close(toW)
 	go m.writer(f, toW, errors)
 	for {
 		n, err := f.Read(buf)
-		if err != nil {
-			errors <- err
-		}
 		if err == os.EOF {
 			break
 		}
+		if err != nil {
+			errors <- os.NewError(fmt.Sprintf("Failed to read from fuse conn: %v", err))
+			break
+		}
 
-		go handle(m.fs, buf[0:n], toW, errors)
+		handle(m.fs, buf[0:n], toW, errors)
 	}
 }
 
@@ -105,6 +107,7 @@ func handle(fs FileSystem, in_data []byte, toW chan [][]byte, errors chan os.Err
 	}
 	var out interface{}
 	var result Error = OK
+	fmt.Printf("Opcode: %v\n", h.Opcode)
 	switch h.Opcode {
 	case FUSE_INIT:
 		in := new(InitIn)
@@ -118,6 +121,9 @@ func handle(fs FileSystem, in_data []byte, toW chan [][]byte, errors chan os.Err
 		if init_out != nil {
 			out = init_out
 		}
+	case FUSE_FORGET:
+		return
+
 	default:
 		errors <- os.NewError(fmt.Sprintf("Unsupported OpCode: %d", h.Opcode))
 		result = EIO
@@ -130,9 +136,9 @@ func handle(fs FileSystem, in_data []byte, toW chan [][]byte, errors chan os.Err
 	}
 	b := new(bytes.Buffer)
 	out_data := make([]byte, 0)
+	fmt.Printf("OpCode: %v result: %v\n", h.Opcode, result)
 	if out != nil && result == OK {
 		fmt.Printf("out = %v, out == nil: %v\n", out, out == nil)
-		return
 		err = binary.Write(b, binary.LittleEndian, out)
 		if err == nil {
 			out_data = b.Bytes()
@@ -140,6 +146,7 @@ func handle(fs FileSystem, in_data []byte, toW chan [][]byte, errors chan os.Err
 			errors <- os.NewError(fmt.Sprintf("Can serialize out: %v", err))
 		}
 	}
+	fmt.Printf("out_data: %v, len(out_data): %d, SizeOfOutHeader: %d\n", out_data, len(out_data), SizeOfOutHeader)
 	var hout OutHeader
 	hout.Unique = h.Unique
 	hout.Error = int32(result)
@@ -151,26 +158,20 @@ func handle(fs FileSystem, in_data []byte, toW chan [][]byte, errors chan os.Err
 		return
 	}
 	_, _ = b.Write(out_data)
-	toW <- [][]byte { b.Bytes() }
+	fmt.Printf("Sending to writer: %v\n", b.Bytes())
+	toW <- [][]byte{b.Bytes()}
 }
 
 func (m *MountPoint) writer(f *os.File, in chan [][]byte, errors chan os.Error) {
-//	fd := f.Fd()
-	for _ = range in {
-//		_, err := Writev(fd, packet)
-//		if err != nil {
-//			errors <- err
-//			continue
-//		}
-	}
-}
-
-func (m *MountPoint) errorHandler(errors chan os.Error) {
-	for err := range errors {
-		log.Stderr("MountPoint.errorHandler: ", err)
-		if err == os.EOF {
-			break
+	fd := f.Fd()
+	for packet := range in {
+		fmt.Printf("writer, packet: %v\n", packet)
+		_, err := Writev(fd, packet)
+		if err != nil {
+			errors <- os.NewError(fmt.Sprintf("writer: Writev failed, err: %v", err))
+			continue
 		}
+		fmt.Printf("writer: OK\n")
 	}
 }
 

fuse/fuse_test.go

@@ -2,6 +2,7 @@ package fuse
 
 import (
 	"fmt"
+	"log"
 	"os"
 	"testing"
 )
@@ -10,15 +11,15 @@ const (
 	tempMountDir = "./.testMountDir"
 )
 
-type testFuse struct {}
+type testFuse struct{}
 
 func (fs *testFuse) Init(in *InitIn) (out *InitOut, code Error) {
-	if (in.Major != FUSE_KERNEL_VERSION) {
+	if in.Major != FUSE_KERNEL_VERSION {
 		fmt.Printf("Major versions does not match. Given %d, want %d\n", in.Major, FUSE_KERNEL_VERSION)
 		code = EIO
 		return
 	}
-	if (in.Minor < FUSE_KERNEL_MINOR_VERSION) {
+	if in.Minor < FUSE_KERNEL_MINOR_VERSION {
 		fmt.Printf("Minor version is less than we support. Given %d, want at least %d\n", in.Minor, FUSE_KERNEL_MINOR_VERSION)
 		code = EIO
 		return
@@ -26,32 +27,42 @@ func (fs *testFuse) Init(in *InitIn) (out *InitOut, code Error) {
 	out = new(InitOut)
 	out.Major = FUSE_KERNEL_VERSION
 	out.Minor = FUSE_KERNEL_MINOR_VERSION
-	out.MaxReadahead = 65536
+	out.MaxReadAhead = in.MaxReadAhead
 	out.MaxWrite = 65536
 	return
 }
 
+func errorHandler(errors chan os.Error) {
+	for err := range errors {
+		log.Stderr("MountPoint.errorHandler: ", err)
+		if err == os.EOF {
+			break
+		}
+	}
+}
+
+
 func TestMount(t *testing.T) {
 	fs := new(testFuse)
-//	toW := make(chan [][]byte, 100)
-//	errors := make(chan os.Error, 100)
+	//	toW := make(chan [][]byte, 100)
+	//	errors := make(chan os.Error, 100)
 
-//	in_data := []byte { 56, 0, 0, 0, 26, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 7, 0, 0, 0, 13, 0, 0, 0, 0, 0, 2, 0, 123, 0, 0, 0, }
-//	handle(fs, in_data, toW, errors)
-//	return
+	//	in_data := []byte { 56, 0, 0, 0, 26, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 7, 0, 0, 0, 13, 0, 0, 0, 0, 0, 2, 0, 123, 0, 0, 0, }
+	//	handle(fs, in_data, toW, errors)
+	//	return
 
 	err := os.Mkdir(tempMountDir, 0777)
 	if err != nil {
 		t.Fatalf("Can't create temp mount dir at %s, err: %v", tempMountDir, err)
 	}
 	defer os.Remove(tempMountDir)
-	m, err := Mount(tempMountDir, fs)
+	m, err, errors := Mount(tempMountDir, fs)
 	if err != nil {
 		t.Fatalf("Can't mount a dir, err: %v", err)
 	}
+	errorHandler(errors)
 	err = m.Unmount()
 	if err != nil {
 		t.Fatalf("Can't unmount a dir, err: %v", err)
 	}
 }
-

fuse/types.go

@@ -407,14 +407,14 @@ type AccessIn struct {
 type InitIn struct {
 	Major        uint32
 	Minor        uint32
-	Max_readahead uint32
+	MaxReadAhead uint32
 	Flags        uint32
 }
 
 type InitOut struct {
 	Major               uint32
 	Minor               uint32
-	MaxReadahead        uint32
+	MaxReadAhead        uint32
 	Flags               uint32
 	MaxBackground       uint16
 	CongestionThreshold uint16