*** empty log message ***
[yaffs/.git] / yaffs_fs.c
index b0608f9198953893df6a591077b2753962a5b1d3..d494e83b7f016874bed3af57a1349a865f4baadd 100644 (file)
@@ -20,7 +20,7 @@
  *
  *
  * Acknowledgements:
- * * Luc van OostenRyck for O_APPEND patch.
+ * * Luc van OostenRyck for numerous patches.
  * * Nick Bane for patches marked NCB.
  * * Some code bodily lifted from JFFS2.
  */
 
 #include "yaffs_guts.h"
 
-#ifdef YAFFS_RAM_ENABLED
+#ifdef CONFIG_YAFFS_RAM_ENABLED
 #include "yaffs_nandemul.h" 
 // 2 MB of RAM for emulation
 #define YAFFS_RAM_EMULATION_SIZE  0x200000
-#endif //YAFFS_RAM_ENABLED
+#endif //CONFIG_YAFFS_RAM_ENABLED
 
-#ifdef YAFFS_MTD_ENABLED
+#ifdef CONFIG_YAFFS_MTD_ENABLED
 #include <linux/mtd/mtd.h>
 #include "yaffs_mtdif.h"
-#endif //YAFFS_MTD_ENABLED
+#endif //CONFIG_YAFFS_MTD_ENABLED
 
 #define T(x) printk x
 
@@ -70,6 +70,7 @@ static void yaffs_put_super(struct super_block *sb);
 
 static ssize_t yaffs_file_read(struct file *f, char *buf, size_t n, loff_t *pos);
 static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n, loff_t *pos);
+static int yaffs_file_flush(struct file* file);
 
 static int yaffs_sync_object(struct file * file, struct dentry *dentry, int datasync);
 
@@ -102,7 +103,7 @@ static int yaffs_follow_link(struct dentry *dentry, struct nameidata *nd);
 
 
 static struct address_space_operations yaffs_file_address_operations = {
-       readpage:       yaffs_readpage,
+       readpage:               yaffs_readpage,
        prepare_write:  yaffs_prepare_write,
        commit_write:   yaffs_commit_write
 };
@@ -117,6 +118,7 @@ static struct file_operations yaffs_file_operations = {
        write:          yaffs_file_write,
 #endif
        mmap:           generic_file_mmap,
+       flush:          yaffs_file_flush,
        fsync:          yaffs_sync_object,
 };
 
@@ -165,15 +167,34 @@ static struct super_operations yaffs_super_ops = {
 
 
 
+static void yaffs_GrossLock(yaffs_Device *dev)
+{
+       T((KERN_DEBUG"yaffs locking\n"));
+
+       down(&dev->grossLock);
+}
 
+static void yaffs_GrossUnlock(yaffs_Device *dev)
+{
+       T((KERN_DEBUG"yaffs unlocking\n"));
+       up(&dev->grossLock);
+
+}
 
 static int yaffs_readlink(struct dentry *dentry, char *buffer, int buflen)
 {
        unsigned char *alias;
        int ret;
 
+       yaffs_Device *dev = yaffs_DentryToObject(dentry)->myDev;
+
+
+       yaffs_GrossLock(dev);
+       
        alias = yaffs_GetSymlinkAlias(yaffs_DentryToObject(dentry));
        
+       yaffs_GrossUnlock(dev);
+       
        if(!alias)
                return -ENOMEM;
 
@@ -186,8 +207,15 @@ static int yaffs_follow_link(struct dentry *dentry, struct nameidata *nd)
 {
        unsigned char *alias;
        int ret;
+       yaffs_Device *dev = yaffs_DentryToObject(dentry)->myDev;
+
+
+       yaffs_GrossLock(dev);
+
        alias = yaffs_GetSymlinkAlias(yaffs_DentryToObject(dentry));
        
+       yaffs_GrossUnlock(dev);
+       
        if(!alias)
                return -ENOMEM;
 
@@ -207,6 +235,11 @@ static struct dentry * yaffs_lookup(struct inode *dir, struct dentry *dentry)
        yaffs_Object *obj;
        struct inode *inode;
        
+       yaffs_Device *dev = yaffs_InodeToObject(dir)->myDev;
+
+
+       yaffs_GrossLock(dev);
+
        
        T((KERN_DEBUG"yaffs_lookup for %d:%s\n",yaffs_InodeToObject(dir)->objectId,dentry->d_name.name));
        
@@ -214,6 +247,8 @@ static struct dentry * yaffs_lookup(struct inode *dir, struct dentry *dentry)
        
        obj = yaffs_GetEquivalentObject(obj); // in case it was a hardlink
        
+
+       
        if(obj)
        {
                T((KERN_DEBUG"yaffs_lookup found %d\n",obj->objectId));
@@ -223,8 +258,11 @@ static struct dentry * yaffs_lookup(struct inode *dir, struct dentry *dentry)
                if(inode)
                {
                        T((KERN_DEBUG"yaffs_loookup looks good\n"));
-                       dget(dentry); // try to solve directory bug
+                       // try to fix ln -s prob dget(dentry); // try to solve directory bug
                        d_add(dentry,inode);
+                       
+                       yaffs_GrossUnlock(dev);
+
                        return dentry;
                }
 
@@ -234,6 +272,8 @@ static struct dentry * yaffs_lookup(struct inode *dir, struct dentry *dentry)
                T((KERN_DEBUG"yaffs_lookup not found\n"));
                
        }
+       yaffs_GrossUnlock(dev);
+       
        return NULL;
        
 }
@@ -243,8 +283,27 @@ static void yaffs_put_inode(struct inode *inode)
 {
        T(("yaffs_put_inode: ino %d, count %d\n",(int)inode->i_ino, atomic_read(&inode->i_count)));
        
-       yaffs_FlushFile(yaffs_InodeToObject(inode));
+       // yaffs_FlushFile(yaffs_InodeToObject(inode));
+       
+}
+
+
+static int yaffs_file_flush(struct file* file)
+{
+       yaffs_Object *obj = yaffs_DentryToObject(file->f_dentry);
+       
+       yaffs_Device *dev = obj->myDev;
+       
+       T((KERN_DEBUG"yaffs_file_flush object %d (%s)\n",obj->objectId,
+                               obj->dirty ? "dirty" : "clean"));
+
+       yaffs_GrossLock(dev);
        
+    yaffs_FlushFile(obj);
+
+       yaffs_GrossUnlock(dev);
+
+    return 0;
 }
 
 
@@ -257,12 +316,15 @@ static int yaffs_readpage_nolock(struct file *f, struct page * pg)
        unsigned char *pg_buf;  
        int ret;
        
+       yaffs_Device *dev;
+       
        T((KERN_DEBUG"yaffs_readpage at %08x, size %08x\n", 
                      pg->index << PAGE_CACHE_SHIFT, PAGE_CACHE_SIZE));
 
        obj  = yaffs_DentryToObject(f->f_dentry);
 
-       //down(obj->sem);
+       dev = obj->myDev;
+       
        
        if (!PageLocked(pg))
                 PAGE_BUG(pg);
@@ -270,8 +332,12 @@ static int yaffs_readpage_nolock(struct file *f, struct page * pg)
        pg_buf = kmap(pg);
        /* FIXME: Can kmap fail? */
 
+       yaffs_GrossLock(dev);
+       
        ret = yaffs_ReadDataFromFile(obj, pg_buf, pg->index << PAGE_CACHE_SHIFT, PAGE_CACHE_SIZE);
 
+       yaffs_GrossUnlock(dev);
+       
        if(ret >= 0) ret = 0;
 
        if (ret) {
@@ -286,9 +352,6 @@ static int yaffs_readpage_nolock(struct file *f, struct page * pg)
        kunmap(pg);
 
 
-
-       //up(&obj->sem);
-
        T((KERN_DEBUG"yaffs_readpage done\n"));
        return ret;
 }
@@ -338,6 +401,10 @@ static int yaffs_commit_write(struct file *f, struct page *pg, unsigned offset,
                SetPageError(pg);
                ClearPageUptodate(pg);
        }
+       else
+       {
+               SetPageUptodate(pg);
+       }
 
        T((KERN_DEBUG"yaffs_commit_write returning %d\n",nWritten));
        
@@ -405,8 +472,7 @@ struct inode *yaffs_get_inode(struct super_block *sb, int mode, int dev,yaffs_Ob
 
        inode = iget(sb,obj->objectId);
 
-       // No need to call this since iget calls it via the read_inode callback
-       // yaffs_FillInodeFromObject(inode,obj);
+       // NB Side effect: iget calls back to yaffs_read_inode().
 
        return inode;
 }
@@ -416,10 +482,16 @@ static ssize_t yaffs_file_read(struct file *f, char *buf, size_t n, loff_t *pos)
        yaffs_Object *obj;
        int nRead,ipos;
        struct inode *inode;
+       yaffs_Device *dev;
        
        T((KERN_DEBUG"yaffs_file_read\n"));
 
        obj  = yaffs_DentryToObject(f->f_dentry);
+       
+       dev = obj->myDev;
+       
+       yaffs_GrossLock(dev);
+       
        inode = f->f_dentry->d_inode;
        
        if (*pos < inode->i_size) 
@@ -439,7 +511,11 @@ static ssize_t yaffs_file_read(struct file *f, char *buf, size_t n, loff_t *pos)
        {
                f->f_pos += nRead;
        }
+       
+       yaffs_GrossUnlock(dev);
+       
        ipos = *pos;
+       
        T((KERN_DEBUG"yaffs_file_read read %d bytes, %d read at %d\n",n,nRead,ipos));
        return nRead;
        
@@ -451,8 +527,15 @@ static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n, loff_
        yaffs_Object *obj;
        int nWritten,ipos;
        struct inode *inode;
+       yaffs_Device *dev;
+       
        
        obj  = yaffs_DentryToObject(f->f_dentry);
+       
+       dev = obj->myDev;
+       
+       yaffs_GrossLock(dev);
+       
        inode = f->f_dentry->d_inode;
 
        if(!S_ISBLK(inode->i_mode) && f->f_flags & O_APPEND)
@@ -490,6 +573,8 @@ static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n, loff_
                }
                
        }
+       yaffs_GrossUnlock(dev);
+       
        return nWritten != n ? -ENOSPC : nWritten;      
 }
 
@@ -497,6 +582,7 @@ static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n, loff_
 static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
 {
        yaffs_Object *obj;
+       yaffs_Device *dev;
        struct inode *inode = f->f_dentry->d_inode;
        unsigned long offset, curoffs;
        struct list_head *i;    
@@ -505,6 +591,9 @@ static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
        char name[YAFFS_MAX_NAME_LENGTH +1];
                
        obj =  yaffs_DentryToObject(f->f_dentry);
+       dev = obj->myDev;
+       
+       yaffs_GrossLock(dev);
        
        offset = f->f_pos;
        
@@ -533,9 +622,6 @@ static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
        
        curoffs = 1;
        
-       //down(&obj->sem);
-
-       
        list_for_each(i,&obj->variant.directoryVariant.children)
        {
                curoffs++;
@@ -563,10 +649,10 @@ static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
        }
 
   up_and_out:
-
-       //up(&obj->sem);
-       
   out:
+  
+    yaffs_GrossUnlock(dev);
+    
        return 0;
 }
 
@@ -579,6 +665,8 @@ static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode, int d
        struct inode *inode;
        
        yaffs_Object *obj = NULL;
+       yaffs_Device *dev;
+       
        yaffs_Object *parent = yaffs_InodeToObject(dir);
        
        int error = -ENOSPC;
@@ -595,7 +683,11 @@ static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode, int d
        }
        
        T(("yaffs_mknod: making oject for %s, mode %x dev %x\n",
-                               dentry->d_name.name, mode,dev));
+                                       dentry->d_name.name, mode,dev));
+
+       dev = parent->myDev;
+       
+       yaffs_GrossLock(dev);
 
        switch (mode & S_IFMT) 
        {
@@ -630,7 +722,8 @@ static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode, int d
                T((KERN_DEBUG"yaffs_mknod failed making object\n"));
                error = -ENOMEM;
        }
-       
+
+       yaffs_GrossUnlock(dev);
 
        return error;
 }
@@ -659,10 +752,19 @@ static int yaffs_create(struct inode *dir, struct dentry *dentry, int mode)
 
 static int yaffs_unlink(struct inode * dir, struct dentry *dentry)
 {
+       int retVal;
+       yaffs_Device *dev;
+       
        
-       T((KERN_DEBUG"yaffs_unlink\n"));
+       T((KERN_DEBUG"yaffs_unlink %d:%s\n",dir->i_ino,dentry->d_name.name));
        
-       if(yaffs_Unlink(yaffs_InodeToObject(dir),dentry->d_name.name) == YAFFS_OK)
+       dev = yaffs_InodeToObject(dir)->myDev;
+       
+       yaffs_GrossLock(dev);
+       retVal = yaffs_Unlink(yaffs_InodeToObject(dir),dentry->d_name.name);
+       yaffs_GrossUnlock(dev);
+       
+       if( retVal == YAFFS_OK)
        {
                return 0;
        }
@@ -681,13 +783,19 @@ static int yaffs_link(struct dentry *old_dentry, struct inode * dir, struct dent
        struct inode *inode = old_dentry->d_inode;
        yaffs_Object *obj = NULL;
        yaffs_Object *link=NULL;
+       yaffs_Device *dev;
        
        T((KERN_DEBUG"yaffs_link\n"));
        
        obj = yaffs_InodeToObject(inode);
+       dev = obj->myDev;
+       
+       yaffs_GrossLock(dev);
        
        link = yaffs_Link(yaffs_InodeToObject(dir),dentry->d_name.name,obj);
        
+       yaffs_GrossUnlock(dev);
+       
        if(link)
        {
                return 0;
@@ -701,12 +809,16 @@ static int yaffs_link(struct dentry *old_dentry, struct inode * dir, struct dent
 static int yaffs_symlink(struct inode * dir, struct dentry *dentry, const char * symname)
 {
        yaffs_Object *obj;
+       yaffs_Device *dev;
        
        T((KERN_DEBUG"yaffs_symlink\n"));
        
+       dev = yaffs_InodeToObject(dir)->myDev;
+       yaffs_GrossLock(dev);
        obj = yaffs_MknodSymLink(yaffs_InodeToObject(dir), dentry->d_name.name, 
                                                         S_IFLNK | S_IRWXUGO, current->uid, current->gid,
                                                         symname);
+       yaffs_GrossUnlock(dev);
 
        if(obj)
        {
@@ -725,8 +837,17 @@ static int yaffs_symlink(struct inode * dir, struct dentry *dentry, const char *
 static int yaffs_sync_object(struct file * file, struct dentry *dentry, int datasync)
 {
 
+       yaffs_Object *obj;
+       yaffs_Device *dev;
+       
+       obj = yaffs_DentryToObject(dentry);
+
+       dev = obj->myDev;
+       
        T((KERN_DEBUG"yaffs_sync_object\n"));
-       yaffs_FlushFile(yaffs_DentryToObject(dentry));
+       yaffs_GrossLock(dev);
+       yaffs_FlushFile(obj);
+       yaffs_GrossUnlock(dev);
        return 0;
 }
 
@@ -735,10 +856,18 @@ static int yaffs_sync_object(struct file * file, struct dentry *dentry, int data
  */
 static int yaffs_rename(struct inode * old_dir, struct dentry *old_dentry, struct inode * new_dir,struct dentry *new_dentry)
 {
+       yaffs_Device *dev;
+       int retVal;
        
+       dev = yaffs_InodeToObject(old_dir)->myDev;
+
+       yaffs_GrossLock(dev);
+       
+       retVal =  yaffs_RenameObject(yaffs_InodeToObject(old_dir),old_dentry->d_name.name,
+                                        yaffs_InodeToObject(new_dir),new_dentry->d_name.name);
+       yaffs_GrossUnlock(dev);
        
-       if( yaffs_RenameObject(yaffs_InodeToObject(old_dir),old_dentry->d_name.name,
-                              yaffs_InodeToObject(new_dir),new_dentry->d_name.name) == YAFFS_OK)
+       if(retVal == YAFFS_OK)
        {
                return 0;
        }
@@ -754,12 +883,15 @@ static int yaffs_setattr(struct dentry *dentry, struct iattr *attr)
 {
        struct inode *inode = dentry->d_inode;
        int error;
+       yaffs_Device *dev;
        
-       T((KERN_DEBUG"yaffs_setattr\n"));
+       T((KERN_DEBUG"yaffs_setattr of object %d\n",yaffs_InodeToObject(inode)->objectId));
        
        if((error = inode_change_ok(inode,attr)) == 0)
        {
        
+               dev = yaffs_InodeToObject(inode)->myDev;
+               yaffs_GrossLock(dev);
                if(yaffs_SetAttributes(yaffs_InodeToObject(inode),attr) == YAFFS_OK)
                {
                        error = 0;
@@ -768,7 +900,7 @@ static int yaffs_setattr(struct dentry *dentry, struct iattr *attr)
                {
                        error = -EPERM;
                }
-
+               yaffs_GrossUnlock(dev);
                inode_setattr(inode,attr);
        }
        return error;
@@ -776,31 +908,40 @@ static int yaffs_setattr(struct dentry *dentry, struct iattr *attr)
 
 static int yaffs_statfs(struct super_block *sb, struct statfs *buf)
 {
+       yaffs_Device *dev = yaffs_SuperToDevice(sb);
        T((KERN_DEBUG"yaffs_statfs\n"));
 
+       yaffs_GrossLock(dev);
+       
        buf->f_type = YAFFS_MAGIC;
        buf->f_bsize = sb->s_blocksize;
        buf->f_namelen = 255;
-       buf->f_blocks = yaffs_SuperToDevice(sb)->nBlocks * YAFFS_CHUNKS_PER_BLOCK/
+       buf->f_blocks = dev->nBlocks * YAFFS_CHUNKS_PER_BLOCK/
                                                (sb->s_blocksize/YAFFS_BYTES_PER_CHUNK);
        buf->f_files = 0;
        buf->f_ffree = 0;
-       buf->f_bfree = yaffs_GetNumberOfFreeChunks(yaffs_SuperToDevice(sb))/
+       buf->f_bfree = yaffs_GetNumberOfFreeChunks(dev)/
                                                (sb->s_blocksize/YAFFS_BYTES_PER_CHUNK);
        buf->f_bavail =  buf->f_bfree;
+       
+       yaffs_GrossUnlock(dev);
        return 0;
 }
 
 static void yaffs_read_inode (struct inode *inode)
 {
-
+       // NB This is called as a side effect of other functions and
+       // thus gross locking should always be in place already.
+       
        yaffs_Object *obj ; 
+       yaffs_Device *dev = yaffs_SuperToDevice(inode->i_sb);
        
        T((KERN_DEBUG"yaffs_read_inode for %d\n",(int)inode->i_ino));
 
-       obj  = yaffs_FindObjectByNumber(yaffs_SuperToDevice(inode->i_sb),inode->i_ino);
+       obj  = yaffs_FindObjectByNumber(dev,inode->i_ino);
        
        yaffs_FillInodeFromObject(inode,obj);
+
 }
 
 
@@ -808,14 +949,19 @@ static void yaffs_put_super(struct super_block *sb)
 {
        yaffs_Device *dev = yaffs_SuperToDevice(sb);
        
+       yaffs_GrossLock(dev);
        if(dev->putSuperFunc)
        {
                 dev->putSuperFunc(sb);
        }
+       yaffs_Deinitialise(dev);
+       yaffs_GrossUnlock(dev);
+       
+       kfree(dev);
 }
 
 
-#ifdef YAFFS_MTD_ENABLED
+#ifdef CONFIG_YAFFS_MTD_ENABLED
 
 static void  yaffs_MTDPutSuper(struct super_block *sb)
 {
@@ -852,7 +998,7 @@ static struct super_block *yaffs_internal_read_super(int useRam, struct super_bl
 
        
 
-#if CONFIG_YAFFS_USE_CHUNK_SIZE
+#ifdef CONFIG_YAFFS_USE_CHUNK_SIZE
        sb->s_blocksize = YAFFS_BYTES_PER_CHUNK;
        sb->s_blocksize_bits = YAFFS_CHUNK_SIZE_SHIFT;
 #else
@@ -861,11 +1007,16 @@ static struct super_block *yaffs_internal_read_super(int useRam, struct super_bl
 #endif
        T(("yaffs_read_super: %s block size %d\n", useRam ? "RAM" : "MTD",sb->s_blocksize));
 
+#ifdef CONFIG_YAFFS_DISABLE_WRITE_VERIFY
+       T(("yaffs: Write verification disabled. All guarantees null and void\n");
+#endif
+
+
        
        if(useRam)
        {
 
-#ifdef YAFFS_RAM_ENABLED
+#ifdef CONFIG_YAFFS_RAM_ENABLED
                // Set the yaffs_Device up for ram emulation
 
                sb->u.generic_sbp =     dev = kmalloc(sizeof(yaffs_Device),GFP_KERNEL);
@@ -893,7 +1044,7 @@ static struct super_block *yaffs_internal_read_super(int useRam, struct super_bl
        }
        else
        {       
-#ifdef YAFFS_MTD_ENABLED
+#ifdef CONFIG_YAFFS_MTD_ENABLED
                struct mtd_info *mtd;
                
                printk(KERN_DEBUG "yaffs: Attempting MTD mount on %u.%u, \"%s\"\n",
@@ -972,13 +1123,19 @@ static struct super_block *yaffs_internal_read_super(int useRam, struct super_bl
 #endif
        }
 
-
-
+       init_MUTEX(&dev->grossLock);
+       
+       
+       yaffs_GrossLock(dev);
        yaffs_GutsInitialise(yaffs_SuperToDevice(sb));
+
        T(("yaffs_read_super: guts initialised\n"));
 
        // Create root inode
        inode = yaffs_get_inode(sb, S_IFDIR | 0755, 0,yaffs_Root(yaffs_SuperToDevice(sb)));
+
+       yaffs_GrossUnlock(dev);
+
        if (!inode)
                return NULL;
 
@@ -999,7 +1156,7 @@ static struct super_block *yaffs_internal_read_super(int useRam, struct super_bl
        return sb;
 }
 
-#ifdef YAFFS_MTD_ENABLED
+#ifdef CONFIG_YAFFS_MTD_ENABLED
 static struct super_block *yaffs_read_super(struct super_block * sb, void * data, int silent)
 {
        return yaffs_internal_read_super(0,sb,data,silent);
@@ -1008,7 +1165,7 @@ static struct super_block *yaffs_read_super(struct super_block * sb, void * data
 static DECLARE_FSTYPE(yaffs_fs_type, "yaffs", yaffs_read_super, FS_REQUIRES_DEV);
 #endif
 
-#ifdef YAFFS_RAM_ENABLED
+#ifdef CONFIG_YAFFS_RAM_ENABLED
 
 static struct super_block *yaffs_ram_read_super(struct super_block * sb, void * data, int silent)
 {
@@ -1016,7 +1173,7 @@ static struct super_block *yaffs_ram_read_super(struct super_block * sb, void *
 }
 
 static DECLARE_FSTYPE(yaffs_ram_fs_type, "yaffsram", yaffs_ram_read_super, FS_SINGLE);
-#endif // YAFFS_RAM_ENABLED
+#endif // CONFIG_YAFFS_RAM_ENABLED
 
 
 static struct proc_dir_entry *my_proc_entry;
@@ -1051,7 +1208,7 @@ static int __init init_yaffs_fs(void)
        int error = 0;
        
        printk(KERN_DEBUG "yaffs " __DATE__ " " __TIME__ " Initialisation\n");
-#if CONFIG_YAFFS_USE_GENERIC_RW
+#ifdef CONFIG_YAFFS_USE_GENERIC_RW
        printk(KERN_DEBUG "yaffs is using generic read/write (caching)\n");
 #else
        printk(KERN_DEBUG "yaffs is using direct read/write (uncached)\n");
@@ -1069,24 +1226,24 @@ static int __init init_yaffs_fs(void)
        return -ENOMEM;
     }
 
-#ifdef YAFFS_RAM_ENABLED
+#ifdef CONFIG_YAFFS_RAM_ENABLED
 
     error = register_filesystem(&yaffs_ram_fs_type);
     if(error)
     {
        return error;
     }
-#endif //YAFFS_RAM_ENABLED
+#endif //CONFIG_YAFFS_RAM_ENABLED
 
-#ifdef YAFFS_MTD_ENABLED
+#ifdef CONFIG_YAFFS_MTD_ENABLED
        error = register_filesystem(&yaffs_fs_type);
        if(error)
        {
-#ifdef YAFFS_RAM_ENABLED
+#ifdef CONFIG_YAFFS_RAM_ENABLED
                unregister_filesystem(&yaffs_ram_fs_type);
-#endif //YAFFS_RAM_ENABLED
+#endif //CONFIG_YAFFS_RAM_ENABLED
        }
-#endif // YAFFS_MTD_ENABLED
+#endif // CONFIG_YAFFS_MTD_ENABLED
 
        return error;
 }
@@ -1097,10 +1254,10 @@ static void __exit exit_yaffs_fs(void)
 
     remove_proc_entry("yaffs",&proc_root);
     
-#ifdef YAFFS_RAM_ENABLED
+#ifdef CONFIG_YAFFS_RAM_ENABLED
        unregister_filesystem(&yaffs_ram_fs_type);
 #endif
-#ifdef YAFFS_MTD_ENABLED
+#ifdef CONFIG_YAFFS_MTD_ENABLED
        unregister_filesystem(&yaffs_fs_type);
 #endif