X-Git-Url: http://www.aleph1.co.uk/gitweb/?p=yaffs2.git;a=blobdiff_plain;f=direct%2Fyaffsfs.c;h=ddbdcdb65354743c00bc02efc70e8111044f6119;hp=cc7eb643e19779f6493b902b71e9b345f501346a;hb=b0173df3b2c8cc1f68d877a5a64e0f0ca93c0bd6;hpb=7f0f2c68474e11183cfbaebfd7c21ba48228f05e diff --git a/direct/yaffsfs.c b/direct/yaffsfs.c index cc7eb64..ddbdcdb 100644 --- a/direct/yaffsfs.c +++ b/direct/yaffsfs.c @@ -1,7 +1,7 @@ /* * YAFFS: Yet Another Flash File System. A NAND-flash specific file system. * - * Copyright (C) 2002-2007 Aleph One Ltd. + * Copyright (C) 2002-2010 Aleph One Ltd. * for Toby Churchill Ltd and Brightstar Engineering * * Created by Charles Manning @@ -17,7 +17,7 @@ #include "yportenv.h" #include "yaffs_trace.h" -#include // for memset +#include /* for memset */ #define YAFFSFS_MAX_SYMLINK_DEREFERENCES 5 @@ -26,9 +26,14 @@ #endif -const char *yaffsfs_c_version="$Id: yaffsfs.c,v 1.31 2010-01-11 04:06:47 charles Exp $"; +/* YAFFSFS_RW_SIZE must be a power of 2 */ +#define YAFFSFS_RW_SHIFT (13) +#define YAFFSFS_RW_SIZE (1<inUse && h->inodeId >= 0 && h->inodeId < YAFFSFS_N_HANDLES) + if(h && h->useCount > 0 && h->inodeId >= 0 && h->inodeId < YAFFSFS_N_HANDLES) return &yaffsfs_inode[h->inodeId]; return NULL; @@ -102,9 +114,10 @@ yaffs_Object *yaffsfs_GetHandleObject(int handle) return NULL; } -//yaffsfs_GetInodeIdForObject -// Grab an inode entry when opening a new inode. -// +/* + * yaffsfs_GetInodeIdForObject + * Grab an inode entry when opening a new inode. + */ static int yaffsfs_GetInodeIdForObject(yaffs_Object *obj) { @@ -164,11 +177,12 @@ static void yaffsfs_PutInode(int inodeId) } -//yaffsfs_GetHandle -// Grab a handle (when opening a file) -// +/* + * yaffsfs_GetHandle + * Grab a handle (when opening a file) + */ -static int yaffsfs_GetHandle(void) +static int yaffsfs_GetNewHandle(void) { int i; yaffsfs_Handle *h; @@ -176,30 +190,49 @@ static int yaffsfs_GetHandle(void) for(i = 0; i < YAFFSFS_N_HANDLES; i++){ h = yaffsfs_GetHandlePointer(i); if(!h){ - // todo bug: should never happen + /* todo bug: should never happen */ } - if(!h->inUse){ + if(h->useCount < 1){ memset(h,0,sizeof(yaffsfs_Handle)); h->inodeId=-1; - h->inUse=1; + h->useCount=1; return i; } } return -1; } -// yaffs_PutHandle -// Let go of a handle when closing a file or aborting an open. -// +/* + * yaffs_GetHandle + * Increase use of handle when reading/writing a file + */ +static int yaffsfs_GetHandle(int handle) +{ + yaffsfs_Handle *h = yaffsfs_GetHandlePointer(handle); + + if(h && h->useCount > 0){ + h->useCount++; + } + return 0; +} + +/* + * yaffs_PutHandle + * Let go of a handle when closing a file or aborting an open or + * ending a read or write. + */ static int yaffsfs_PutHandle(int handle) { yaffsfs_Handle *h = yaffsfs_GetHandlePointer(handle); - if(h){ - if(h->inodeId >= 0) - yaffsfs_PutInode(h->inodeId); - h->inUse = 0; - h->inodeId = -1; + if(h && h->useCount > 0){ + h->useCount--; + if(h->useCount < 1){ + if(h->inodeId >= 0){ + yaffsfs_PutInode(h->inodeId); + h->inodeId = -1; + } + } } return 0; } @@ -207,12 +240,14 @@ static int yaffsfs_PutHandle(int handle) -// Stuff to search for a directory from a path +/* + * Stuff to search for a directory from a path + */ int yaffsfs_Match(YCHAR a, YCHAR b) { - // case sensitive + /* case sensitive */ return (a == b); } @@ -229,11 +264,13 @@ int yaffsfs_IsPathDivider(YCHAR ch) return 0; } -// yaffsfs_FindDevice -// yaffsfs_FindRoot -// Scan the configuration list to find the root. -// Curveballs: Should match paths that end in '/' too -// Curveball2 Might have "/x/ and "/x/y". Need to return the longest match +/* + * yaffsfs_FindDevice + * yaffsfs_FindRoot + * Scan the configuration list to find the root. + * Curveballs: Should match paths that end in '/' too + * Curveball2 Might have "/x/ and "/x/y". Need to return the longest match + */ static yaffs_Device *yaffsfs_FindDevice(const YCHAR *path, YCHAR **restOfPath) { yaffsfs_DeviceConfiguration *cfg = yaffsfs_configurationList; @@ -244,9 +281,11 @@ static yaffs_Device *yaffsfs_FindDevice(const YCHAR *path, YCHAR **restOfPath) int longestMatch = -1; int matching; - // Check all configs, choose the one that: - // 1) Actually matches a prefix (ie /a amd /abc will not match - // 2) Matches the longest. + /* + * Check all configs, choose the one that: + * 1) Actually matches a prefix (ie /a amd /abc will not match + * 2) Matches the longest. + */ while(cfg && cfg->prefix && cfg->dev){ leftOver = path; p = cfg->prefix; @@ -255,15 +294,15 @@ static yaffs_Device *yaffsfs_FindDevice(const YCHAR *path, YCHAR **restOfPath) while(matching && *p && *leftOver){ - // Skip over any /s + /* Skip over any /s */ while(yaffsfs_IsPathDivider(*p)) p++; - // Skip over any /s + /* Skip over any /s */ while(yaffsfs_IsPathDivider(*leftOver)) leftOver++; - // Now match the text part + /* Now match the text part */ while(matching && *p && !yaffsfs_IsPathDivider(*p) && *leftOver && !yaffsfs_IsPathDivider(*leftOver)){ @@ -277,13 +316,13 @@ static yaffs_Device *yaffsfs_FindDevice(const YCHAR *path, YCHAR **restOfPath) } } - // Skip over any /s in leftOver + /* Skip over any /s in leftOver */ while(yaffsfs_IsPathDivider(*leftOver)) leftOver++; if( matching && (thisMatchLength > longestMatch)){ - // Matched prefix + /* Matched prefix */ *restOfPath = (YCHAR *)leftOver; retval = cfg->dev; longestMatch = thisMatchLength; @@ -304,28 +343,29 @@ static yaffs_Device *yaffsfs_FindDevice(const YCHAR *path, YCHAR **restOfPath) int thisMatchLength; int longestMatch = -1; - // Check all configs, choose the one that: - // 1) Actually matches a prefix (ie /a amd /abc will not match - // 2) Matches the longest. + /* + * Check all configs, choose the one that: + * 1) Actually matches a prefix (ie /a amd /abc will not match + * 2) Matches the longest. + */ while(cfg && cfg->prefix && cfg->dev){ leftOver = path; p = cfg->prefix; thisMatchLength = 0; - while(*p && //unmatched part of prefix - !(yaffsfs_IsPathDivider(*p) && (p[1] == 0)) && // the rest of the prefix is not / (to catch / at end) - *leftOver && - yaffsfs_Match(*p,*leftOver)){ + while(*p && /* unmatched part of prefix */ + !(yaffsfs_IsPathDivider(*p) && (p[1] == 0)) && + *leftOver && yaffsfs_Match(*p,*leftOver)){ p++; leftOver++; thisMatchLength++; } - if((!*p || (yaffsfs_IsPathDivider(*p) && (p[1] == 0))) && // end of prefix - (!*leftOver || yaffsfs_IsPathDivider(*leftOver)) && // no more in this path name part + if((!*p || (yaffsfs_IsPathDivider(*p) && (p[1] == 0))) && /* end of prefix */ + (!*leftOver || yaffsfs_IsPathDivider(*leftOver)) && /* no more in this path name part */ (thisMatchLength > longestMatch)){ - // Matched prefix + /* Matched prefix */ *restOfPath = (YCHAR *)leftOver; retval = cfg->dev; longestMatch = thisMatchLength; @@ -355,21 +395,24 @@ static yaffs_Object *yaffsfs_FollowLink(yaffs_Object *obj,int symDepth) YCHAR *alias = obj->variant.symLinkVariant.alias; if(yaffsfs_IsPathDivider(*alias)) - // Starts with a /, need to scan from root up + /* Starts with a /, need to scan from root up */ obj = yaffsfs_FindObject(NULL,alias,symDepth++); else - // Relative to here, so use the parent of the symlink as a start + /* Relative to here, so use the parent of the symlink as a start */ obj = yaffsfs_FindObject(obj->parent,alias,symDepth++); } return obj; } -// yaffsfs_FindDirectory -// Parse a path to determine the directory and the name within the directory. -// -// eg. "/data/xx/ff" --> puts name="ff" and returns the directory "/data/xx" -static yaffs_Object *yaffsfs_DoFindDirectory(yaffs_Object *startDir,const YCHAR *path,YCHAR **name,int symDepth) +/* + * yaffsfs_FindDirectory + * Parse a path to determine the directory and the name within the directory. + * + * eg. "/data/xx/ff" --> puts name="ff" and returns the directory "/data/xx" + */ +static yaffs_Object *yaffsfs_DoFindDirectory(yaffs_Object *startDir, + const YCHAR *path, YCHAR **name, int symDepth) { yaffs_Object *dir; YCHAR *restOfPath; @@ -387,11 +430,13 @@ static yaffs_Object *yaffsfs_DoFindDirectory(yaffs_Object *startDir,const YCHAR dir = yaffsfs_FindRoot(path,&restOfPath); while(dir){ - // parse off /. - // curve ball: also throw away surplus '/' - // eg. "/ram/x////ff" gets treated the same as "/ram/x/ff" + /* + * parse off /. + * curve ball: also throw away surplus '/' + * eg. "/ram/x////ff" gets treated the same as "/ram/x/ff" + */ while(yaffsfs_IsPathDivider(*restOfPath)) - restOfPath++; // get rid of '/' + restOfPath++; /* get rid of '/' */ *name = restOfPath; i = 0; @@ -406,12 +451,12 @@ static yaffs_Object *yaffsfs_DoFindDirectory(yaffs_Object *startDir,const YCHAR } if(!*restOfPath) - // got to the end of the string + /* got to the end of the string */ return dir; else{ if(yaffs_strcmp(str,_Y(".")) == 0) { - // Do nothing + /* Do nothing */ } else if(yaffs_strcmp(str,_Y("..")) == 0) dir = dir->parent; @@ -427,17 +472,19 @@ static yaffs_Object *yaffsfs_DoFindDirectory(yaffs_Object *startDir,const YCHAR } } } - // directory did not exist. + /* directory did not exist. */ return NULL; } -static yaffs_Object *yaffsfs_FindDirectory(yaffs_Object *relativeDirectory,const YCHAR *path,YCHAR **name,int symDepth) +static yaffs_Object *yaffsfs_FindDirectory(yaffs_Object *relativeDirectory, + const YCHAR *path,YCHAR **name,int symDepth) { return yaffsfs_DoFindDirectory(relativeDirectory,path,name,symDepth); } -// yaffsfs_FindObject turns a path for an existing object into the object -// +/* + * yaffsfs_FindObject turns a path for an existing object into the object + */ static yaffs_Object *yaffsfs_FindObject(yaffs_Object *relativeDirectory, const YCHAR *path,int symDepth) { yaffs_Object *dir; @@ -461,8 +508,8 @@ int yaffs_dup(int fd) yaffsfs_Lock(); oldPtr = yaffsfs_GetHandlePointer(fd); - if(oldPtr && oldPtr->inUse) - newHandle = yaffsfs_GetHandle(); + if(oldPtr && oldPtr->useCount > 0) + newHandle = yaffsfs_GetNewHandle(); if(newHandle >= 0) newPtr = yaffsfs_GetHandlePointer(newHandle); @@ -487,28 +534,31 @@ int yaffs_open(const YCHAR *path, int oflag, int mode) YCHAR *name; int handle = -1; yaffsfs_Handle *h = NULL; - int alreadyOpen = 0; - int alreadyExclusive = 0; int openDenied = 0; int symDepth = 0; int errorReported = 0; - int i; + /* O_EXCL only has meaning if O_CREAT is specified */ + if(!(oflag & O_CREAT)) + oflag &= ~(O_EXCL); + /* O_TRUNC has no meaning if (O_CREAT | O_EXCL) is specified */ + if( (oflag & O_CREAT) & (oflag & O_EXCL)) + oflag &= ~(O_TRUNC); - // todo sanity check oflag (eg. can't have O_TRUNC without WRONLY or RDWR + /* Todo: Are there any more flag combos to sanitise ? */ yaffsfs_Lock(); - handle = yaffsfs_GetHandle(); + handle = yaffsfs_GetNewHandle(); if(handle >= 0){ h = yaffsfs_GetHandlePointer(handle); - // try to find the exisiting object + /* try to find the exisiting object */ obj = yaffsfs_FindObject(NULL,path,0); if(obj && obj->variantType == YAFFS_OBJECT_TYPE_SYMLINK) @@ -521,31 +571,20 @@ int yaffs_open(const YCHAR *path, int oflag, int mode) obj = NULL; if(obj){ - // Check if the object is already in use - alreadyOpen = alreadyExclusive = 0; - - for(i = 0; i < YAFFSFS_N_HANDLES; i++){ - if(i != handle && - yaffsfs_handle[i].inUse && - obj == yaffsfs_inode[yaffsfs_handle[i].inodeId].iObj){ - alreadyOpen = 1; - if(yaffsfs_handle[i].exclusive) - alreadyExclusive = 1; - } - } - if(((oflag & O_EXCL) && alreadyOpen) || alreadyExclusive) - openDenied = 1; + /* The file already exists */ - // Open should fail if O_CREAT and O_EXCL are specified + /* Open should fail if O_CREAT and O_EXCL are specified since + * the file exists + */ if((oflag & O_EXCL) && (oflag & O_CREAT)){ openDenied = 1; yaffsfs_SetError(-EEXIST); errorReported = 1; } - // Check file permissions - if( (oflag & (O_RDWR | O_WRONLY)) == 0 && // ie O_RDONLY + /* Check file permissions */ + if( (oflag & (O_RDWR | O_WRONLY)) == 0 && /* ie O_RDONLY */ !(obj->yst_mode & S_IREAD)) openDenied = 1; @@ -558,9 +597,12 @@ int yaffs_open(const YCHAR *path, int oflag, int mode) openDenied = 1; } else if((oflag & O_CREAT)) { - // Let's see if we can create this file + /* Let's see if we can create this file */ dir = yaffsfs_FindDirectory(NULL,path,&name,0); - if(dir) + if(dir && dir->myDev->readOnly){ + yaffsfs_SetError(-EINVAL); + errorReported = 1; + } else if(dir) obj = yaffs_MknodFile(dir,name,mode,0,0); else { yaffsfs_SetError(-ENOTDIR); @@ -579,10 +621,8 @@ int yaffs_open(const YCHAR *path, int oflag, int mode) } h->inodeId = inodeId; - h->inUse = 1; h->readOnly = (oflag & (O_WRONLY | O_RDWR)) ? 0 : 1; h->append = (oflag & O_APPEND) ? 1 : 0; - h->exclusive = (oflag & O_EXCL) ? 1 : 0; h->position = 0; /* Hook inode to object */ @@ -614,11 +654,11 @@ int yaffs_Dofsync(int fd,int datasync) h = yaffsfs_GetHandlePointer(fd); - if(h && h->inUse) - // flush the file + if(h && h->useCount > 0) + /* flush the file */ yaffs_FlushFile(yaffsfs_inode[h->inodeId].iObj,1,datasync); else { - // bad handle + /* bad handle */ yaffsfs_SetError(-EBADF); retVal = -1; } @@ -652,13 +692,13 @@ int yaffs_close(int fd) h = yaffsfs_GetHandlePointer(fd); - if(h && h->inUse) { - // clean up + if(h && h->useCount > 0) { + /* clean up */ yaffs_FlushFile(yaffsfs_inode[h->inodeId].iObj,1,0); yaffsfs_PutHandle(fd); retVal = 0; } else { - // bad handle + /* bad handle */ yaffsfs_SetError(-EBADF); retVal = -1; } @@ -668,24 +708,36 @@ int yaffs_close(int fd) return retVal; } -int yaffs_read(int fd, void *buf, unsigned int nbyte) + + +int yaffsfs_do_read(int fd, void *vbuf, unsigned int nbyte, int isPread, int offset) { yaffsfs_Handle *h = NULL; yaffs_Object *obj = NULL; int pos = 0; - int nRead = -1; + int startPos = 0; + int nRead = 0; + int nToRead = 0; + int totalRead = 0; unsigned int maxRead; + __u8 *buf = (__u8 *)vbuf; yaffsfs_Lock(); h = yaffsfs_GetHandlePointer(fd); obj = yaffsfs_GetHandleObject(fd); - if(!h || !obj) - // bad handle + if(!h || !obj){ + /* bad handle */ yaffsfs_SetError(-EBADF); + totalRead = -1; + } else if( h && obj){ + if(isPread) + startPos = offset; + else + startPos = h->position; - else if( h && obj){ - pos= h->position; + pos = startPos; + if(yaffs_GetObjectFileLength(obj) > pos) maxRead = yaffs_GetObjectFileLength(obj) - pos; else @@ -695,137 +747,146 @@ int yaffs_read(int fd, void *buf, unsigned int nbyte) nbyte = maxRead; - if(nbyte > 0) { - nRead = yaffs_ReadDataFromFile(obj,buf,pos,nbyte); - if(nRead >= 0) - h->position = pos + nRead; - else { - //todo error - } - } else - nRead = 0; + yaffsfs_GetHandle(fd); - } + while(nbyte > 0) { + nToRead = YAFFSFS_RW_SIZE - (pos & (YAFFSFS_RW_SIZE -1)); + if(nToRead > nbyte) + nToRead = nbyte; + nRead = yaffs_ReadDataFromFile(obj,buf,pos,nToRead); - yaffsfs_Unlock(); - - - return (nRead >= 0) ? nRead : -1; - -} - -int yaffs_pread(int fd, void *buf, unsigned int nbyte, unsigned int offset) -{ - yaffsfs_Handle *h = NULL; - yaffs_Object *obj = NULL; - int pos = 0; - int nRead = -1; - unsigned int maxRead; + if(nRead > 0){ + totalRead += nRead; + pos += nRead; + buf += nRead; + } - yaffsfs_Lock(); - h = yaffsfs_GetHandlePointer(fd); - obj = yaffsfs_GetHandleObject(fd); + if(nRead == nToRead) + nbyte-=nRead; + else + nbyte = 0; /* no more to read */ + + + if(nbyte > 0){ + yaffsfs_Unlock(); + yaffsfs_Lock(); + } - if(!h || !obj) - // bad handle - yaffsfs_SetError(-EBADF); - else if( h && obj) { - pos= offset; - if(yaffs_GetObjectFileLength(obj) > pos) - maxRead = yaffs_GetObjectFileLength(obj) - pos; - else - maxRead = 0; + } - if(nbyte > maxRead) - nbyte = maxRead; + yaffsfs_PutHandle(fd); + if(!isPread) { + if(totalRead >= 0) + h->position = startPos + totalRead; + else { + /* todo error */ + } + } - if(nbyte > 0) - nRead = yaffs_ReadDataFromFile(obj,buf,pos,nbyte); - else - nRead = 0; } yaffsfs_Unlock(); + return (totalRead >= 0) ? totalRead : -1; - return (nRead >= 0) ? nRead : -1; +} +int yaffs_read(int fd, void *buf, unsigned int nbyte) +{ + return yaffsfs_do_read(fd, buf, nbyte, 0, 0); } -int yaffs_write(int fd, const void *buf, unsigned int nbyte) +int yaffs_pread(int fd, void *buf, unsigned int nbyte, unsigned int offset) +{ + return yaffsfs_do_read(fd, buf, nbyte, 1, offset); +} + +int yaffsfs_do_write(int fd, const void *vbuf, unsigned int nbyte, int isPwrite, int offset) { yaffsfs_Handle *h = NULL; yaffs_Object *obj = NULL; int pos = 0; - int nWritten = -1; + int startPos = 0; + int nWritten = 0; + int totalWritten = 0; int writeThrough = 0; + int nToWrite = 0; + const __u8 *buf = (const __u8 *)vbuf; yaffsfs_Lock(); h = yaffsfs_GetHandlePointer(fd); obj = yaffsfs_GetHandleObject(fd); - if(!h || !obj) - // bad handle + if(!h || !obj){ + /* bad handle */ yaffsfs_SetError(-EBADF); - else if( h && obj && h->readOnly){ - // todo error + totalWritten = -1; + } else if( h && obj && (h->readOnly || obj->myDev->readOnly)){ + yaffsfs_SetError(-EINVAL); + totalWritten=-1; } else if( h && obj){ + if(isPwrite) + startPos = offset; if(h->append) - pos = yaffs_GetObjectFileLength(obj); + startPos = yaffs_GetObjectFileLength(obj); else - pos = h->position; + startPos = h->position; + + yaffsfs_GetHandle(fd); + pos = startPos; + while(nbyte > 0) { + nToWrite = YAFFSFS_RW_SIZE - (pos & (YAFFSFS_RW_SIZE -1)); + if(nToWrite > nbyte) + nToWrite = nbyte; + + nWritten = yaffs_WriteDataToFile(obj,buf,pos,nToWrite,writeThrough); + if(nWritten > 0){ + totalWritten += nWritten; + pos += nWritten; + buf += nWritten; + } - nWritten = yaffs_WriteDataToFile(obj,buf,pos,nbyte,writeThrough); + if(nWritten == nToWrite) + nbyte -= nToWrite; + else + nbyte = 0; - if(nWritten >= 0) - h->position = pos + nWritten; - else { - //todo error + if(nWritten < 1 && totalWritten < 1){ + yaffsfs_SetError(-ENOSPC); + totalWritten = -1; + } + + if(nbyte > 0){ + yaffsfs_Unlock(); + yaffsfs_Lock(); + } } + yaffsfs_PutHandle(fd); + + if(!isPwrite){ + if(totalWritten > 0) + h->position = startPos + totalWritten; + else { + /* todo error */ + } + } } yaffsfs_Unlock(); + return (totalWritten >= 0) ? totalWritten : -1; +} - return (nWritten >= 0) ? nWritten : -1; - +int yaffs_write(int fd, const void *buf, unsigned int nbyte) +{ + return yaffsfs_do_write(fd, buf, nbyte, 0, 0); } int yaffs_pwrite(int fd, const void *buf, unsigned int nbyte, unsigned int offset) { - yaffsfs_Handle *h = NULL; - yaffs_Object *obj = NULL; - int pos = 0; - int nWritten = -1; - int writeThrough = 0; - - yaffsfs_Lock(); - h = yaffsfs_GetHandlePointer(fd); - obj = yaffsfs_GetHandleObject(fd); - - if(!h || !obj) - // bad handle - yaffsfs_SetError(-EBADF); - else if( h && obj && h->readOnly){ - // todo error - } - else if( h && obj){ - pos = offset; - - nWritten = yaffs_WriteDataToFile(obj,buf,pos,nbyte,writeThrough); - - if(nWritten < 0 || ((unsigned int)nWritten) < nbyte) - yaffsfs_SetError(-ENOSPC); - - } - - yaffsfs_Unlock(); - - - return (nWritten >= 0) ? nWritten : -1; - + return yaffsfs_do_write(fd, buf, nbyte, 1, offset); } @@ -844,6 +905,8 @@ int yaffs_truncate(const YCHAR *path,off_t newSize) yaffsfs_SetError(-ENOENT); else if(obj->variantType != YAFFS_OBJECT_TYPE_FILE) yaffsfs_SetError(-EISDIR); + else if(obj->myDev->readOnly) + yaffsfs_SetError(-EINVAL); else result = yaffs_ResizeFile(obj,newSize); @@ -864,10 +927,12 @@ int yaffs_ftruncate(int fd, off_t newSize) obj = yaffsfs_GetHandleObject(fd); if(!h || !obj) - // bad handle + /* bad handle */ yaffsfs_SetError(-EBADF); + else if(obj->myDev->readOnly) + yaffsfs_SetError(-EINVAL); else - // resize the file + /* resize the file */ result = yaffs_ResizeFile(obj,newSize); yaffsfs_Unlock(); @@ -888,7 +953,7 @@ off_t yaffs_lseek(int fd, off_t offset, int whence) obj = yaffsfs_GetHandleObject(fd); if(!h || !obj) - // bad handle + /* bad handle */ yaffsfs_SetError(-EBADF); else if(whence == SEEK_SET){ if(offset >= 0) @@ -907,7 +972,7 @@ off_t yaffs_lseek(int fd, off_t offset, int whence) if(pos >= 0) h->position = pos; else { - // todo error + /* todo error */ } @@ -932,6 +997,8 @@ int yaffsfs_DoUnlink(const YCHAR *path,int isDirectory) yaffsfs_SetError(-ENOTDIR); else if(!obj) yaffsfs_SetError(-ENOENT); + else if(obj->myDev->readOnly) + yaffsfs_SetError(-EINVAL); else if(!isDirectory && obj->variantType == YAFFS_OBJECT_TYPE_DIRECTORY) yaffsfs_SetError(-EISDIR); else if(isDirectory && obj->variantType != YAFFS_OBJECT_TYPE_DIRECTORY) @@ -945,7 +1012,7 @@ int yaffsfs_DoUnlink(const YCHAR *path,int isDirectory) yaffsfs_Unlock(); - // todo error + /* todo error */ return (result == YAFFS_FAIL) ? -1 : 0; } @@ -978,18 +1045,23 @@ int yaffs_rename(const YCHAR *oldPath, const YCHAR *newPath) obj = yaffsfs_FindObject(NULL,oldPath,0); if(!olddir || !newdir || !obj) { - // bad file + /* bad file */ yaffsfs_SetError(-EBADF); renameAllowed = 0; + } else if(obj->myDev->readOnly){ + yaffsfs_SetError(-EINVAL); + renameAllowed = 0; } else if(olddir->myDev != newdir->myDev) { - // oops must be on same device - // todo error + /* oops must be on same device */ + /* todo error */ yaffsfs_SetError(-EXDEV); renameAllowed = 0; } else if(obj && obj->variantType == YAFFS_OBJECT_TYPE_DIRECTORY) { - // It is a directory, check that it is not being renamed to - // being its own decendent. - // Do this by tracing from the new directory back to the root, checking for obj + /* + * It is a directory, check that it is not being renamed to + * being its own decendent. + * Do this by tracing from the new directory back to the root, checking for obj + */ yaffs_Object *xx = newdir; @@ -1019,9 +1091,9 @@ static int yaffsfs_DoStat(yaffs_Object *obj,struct yaffs_stat *buf) obj = yaffs_GetEquivalentObject(obj); if(obj && buf){ - buf->st_dev = (int)obj->myDev->genericDevice; + buf->st_dev = (int)obj->myDev->context; buf->st_ino = obj->objectId; - buf->st_mode = obj->yst_mode & ~S_IFMT; // clear out file type bits + buf->st_mode = obj->yst_mode & ~S_IFMT; /* clear out file type bits */ if(obj->variantType == YAFFS_OBJECT_TYPE_DIRECTORY) buf->st_mode |= S_IFDIR; @@ -1069,7 +1141,7 @@ static int yaffsfs_DoStatOrLStat(const YCHAR *path, struct yaffs_stat *buf,int d if(obj) retVal = yaffsfs_DoStat(obj,buf); else - // todo error not found + /* todo error not found */ yaffsfs_SetError(-ENOENT); yaffsfs_Unlock(); @@ -1100,7 +1172,7 @@ int yaffs_fstat(int fd, struct yaffs_stat *buf) if(obj) retVal = yaffsfs_DoStat(obj,buf); else - // bad handle + /* bad handle */ yaffsfs_SetError(-EBADF); yaffsfs_Unlock(); @@ -1136,7 +1208,7 @@ int yaffs_get_wince_times(int fd, unsigned *wctime, unsigned *watime, unsigned * retVal = 0; } else - // bad handle + /* bad handle */ yaffsfs_SetError(-EBADF); yaffsfs_Unlock(); @@ -1176,7 +1248,7 @@ int yaffs_set_wince_times(int fd, result = yaffs_FlushFile(obj,0,0); retVal = 0; } else - // bad handle + /* bad handle */ yaffsfs_SetError(-EBADF); yaffsfs_Unlock(); @@ -1228,7 +1300,7 @@ int yaffs_access(const YCHAR *path, int amode) retval = -1; } } else { - // todo error not found + /* todo error not found */ yaffsfs_SetError(-ENOENT); retval = -1; } @@ -1249,11 +1321,12 @@ int yaffs_chmod(const YCHAR *path, mode_t mode) yaffsfs_Lock(); obj = yaffsfs_FindObject(NULL,path,0); - if(obj) - retVal = yaffsfs_DoChMod(obj,mode); - else - // todo error not found + if(!obj) yaffsfs_SetError(-ENOENT); + else if(obj->myDev->readOnly) + yaffsfs_SetError(-EINVAL); + else + retVal = yaffsfs_DoChMod(obj,mode); yaffsfs_Unlock(); @@ -1271,11 +1344,12 @@ int yaffs_fchmod(int fd, mode_t mode) yaffsfs_Lock(); obj = yaffsfs_GetHandleObject(fd); - if(obj) - retVal = yaffsfs_DoChMod(obj,mode); + if(!obj) + yaffsfs_SetError(-ENOENT); + else if(obj->myDev->readOnly) + yaffsfs_SetError(-EINVAL); else - // bad handle - yaffsfs_SetError(-EBADF); + retVal = yaffsfs_DoChMod(obj,mode); yaffsfs_Unlock(); @@ -1292,18 +1366,22 @@ int yaffs_mkdir(const YCHAR *path, mode_t mode) yaffsfs_Lock(); parent = yaffsfs_FindDirectory(NULL,path,&name,0); - if(parent) - dir = yaffs_MknodDirectory(parent,name,mode,0,0); - if(dir) - retVal = 0; - else { - if(!parent) - yaffsfs_SetError(-ENOENT); // missing path - else if (yaffs_FindObjectByName(parent,name)) - yaffsfs_SetError(-EEXIST); // the name already exists - else - yaffsfs_SetError(-ENOSPC); // just assume no space - retVal = -1; + if(parent && parent->myDev->readOnly){ + yaffsfs_SetError(-EINVAL); + } else { + if(parent) + dir = yaffs_MknodDirectory(parent,name,mode,0,0); + if(dir) + retVal = 0; + else { + if(!parent) + yaffsfs_SetError(-ENOENT); /* missing path */ + else if (yaffs_FindObjectByName(parent,name)) + yaffsfs_SetError(-EEXIST); /* the name already exists */ + else + yaffsfs_SetError(-ENOSPC); /* just assume no space */ + retVal = -1; + } } yaffsfs_Unlock(); @@ -1311,7 +1389,7 @@ int yaffs_mkdir(const YCHAR *path, mode_t mode) return retVal; } -int yaffs_mount(const YCHAR *path) +int yaffs_mount2(const YCHAR *path,int readOnly) { int retVal=-1; int result=YAFFS_FAIL; @@ -1324,18 +1402,19 @@ int yaffs_mount(const YCHAR *path) dev = yaffsfs_FindDevice(path,&dummy); if(dev){ if(!dev->isMounted){ + dev->readOnly = readOnly ? 1 : 0; result = yaffs_GutsInitialise(dev); if(result == YAFFS_FAIL) - // todo error - mount failed + /* todo error - mount failed */ yaffsfs_SetError(-ENOMEM); retVal = result ? 0 : -1; } else - //todo error - already mounted. + /* todo error - already mounted. */ yaffsfs_SetError(-EBUSY); } else - // todo error - no device + /* todo error - no device */ yaffsfs_SetError(-ENODEV); yaffsfs_Unlock(); @@ -1343,6 +1422,11 @@ int yaffs_mount(const YCHAR *path) } +int yaffs_mount(const YCHAR *path) +{ + return yaffs_mount2(path,0); +} + int yaffs_sync(const YCHAR *path) { int retVal=-1; @@ -1359,11 +1443,11 @@ int yaffs_sync(const YCHAR *path) } else - //todo error - not mounted. + /* todo error - not mounted. */ yaffsfs_SetError(-EINVAL); }else - // todo error - no device + /* todo error - no device */ yaffsfs_SetError(-ENODEV); yaffsfs_Unlock(); @@ -1371,7 +1455,47 @@ int yaffs_sync(const YCHAR *path) } -int yaffs_unmount(const YCHAR *path) +int yaffs_remount(const YCHAR *path, int force, int readOnly) +{ + int retVal=-1; + yaffs_Device *dev=NULL; + YCHAR *dummy; + + yaffsfs_Lock(); + dev = yaffsfs_FindDevice(path,&dummy); + if(dev){ + if(dev->isMounted){ + int i; + int inUse; + + yaffs_FlushEntireDeviceCache(dev); + + for(i = inUse = 0; i < YAFFSFS_N_HANDLES && !inUse && !force; i++){ + if(yaffsfs_handle[i].useCount>0 && yaffsfs_inode[yaffsfs_handle[i].inodeId].iObj->myDev == dev) + inUse = 1; /* the device is in use, can't unmount */ + } + + if(!inUse || force){ + if(readOnly) + yaffs_CheckpointSave(dev); + dev->readOnly = readOnly ? 1 : 0; + retVal = 0; + } else + yaffsfs_SetError(-EBUSY); + + } else + yaffsfs_SetError(-EINVAL); + + } + else + yaffsfs_SetError(-ENODEV); + + yaffsfs_Unlock(); + return retVal; + +} + +int yaffs_unmount2(const YCHAR *path, int force) { int retVal=-1; yaffs_Device *dev=NULL; @@ -1388,25 +1512,25 @@ int yaffs_unmount(const YCHAR *path) yaffs_CheckpointSave(dev); for(i = inUse = 0; i < YAFFSFS_N_HANDLES && !inUse; i++){ - if(yaffsfs_handle[i].inUse && yaffsfs_inode[yaffsfs_handle[i].inodeId].iObj->myDev == dev) - inUse = 1; // the device is in use, can't unmount + if(yaffsfs_handle[i].useCount > 0 && yaffsfs_inode[yaffsfs_handle[i].inodeId].iObj->myDev == dev) + inUse = 1; /* the device is in use, can't unmount */ } - if(!inUse){ + if(!inUse || force){ yaffs_Deinitialise(dev); retVal = 0; } else - // todo error can't unmount as files are open + /* todo error can't unmount as files are open */ yaffsfs_SetError(-EBUSY); } else - //todo error - not mounted. + /* todo error - not mounted. */ yaffsfs_SetError(-EINVAL); } else - // todo error - no device + /* todo error - no device */ yaffsfs_SetError(-ENODEV); yaffsfs_Unlock(); @@ -1414,6 +1538,11 @@ int yaffs_unmount(const YCHAR *path) } +int yaffs_unmount(const YCHAR *path) +{ + return yaffs_unmount2(path,0); +} + loff_t yaffs_freespace(const YCHAR *path) { loff_t retVal=-1; @@ -1442,8 +1571,8 @@ loff_t yaffs_totalspace(const YCHAR *path) yaffsfs_Lock(); dev = yaffsfs_FindDevice(path,&dummy); if(dev && dev->isMounted){ - retVal = (dev->endBlock - dev->startBlock + 1) - dev->nReservedBlocks; - retVal *= dev->nChunksPerBlock; + retVal = (dev->param.endBlock - dev->param.startBlock + 1) - dev->param.nReservedBlocks; + retVal *= dev->param.nChunksPerBlock; retVal *= dev->nDataBytesPerChunk; } else @@ -1489,7 +1618,7 @@ void yaffs_initialise(yaffsfs_DeviceConfiguration *cfgList) while(cfg && cfg->prefix && cfg->dev){ cfg->dev->isMounted = 0; - cfg->dev->removeObjectCallback = yaffsfs_RemoveObjectCallback; + cfg->dev->param.removeObjectCallback = yaffsfs_RemoveObjectCallback; cfg++; } @@ -1497,13 +1626,13 @@ void yaffs_initialise(yaffsfs_DeviceConfiguration *cfgList) } -// -// Directory search stuff. +/* Directory search stuff. */ -// -// Directory search context -// -// NB this is an opaque structure. +/* + * Directory search context + * + * NB this is an opaque structure. + */ typedef struct @@ -1638,7 +1767,7 @@ struct yaffs_dirent *yaffs_readdir(yaffs_DIR *dirp) yaffs_GetObjectName(dsc->nextReturn,dsc->de.d_name,NAME_MAX); if(yaffs_strnlen(dsc->de.d_name,NAME_MAX+1) == 0) { - // this should not happen! + /* this should not happen! */ yaffs_strcpy(dsc->de.d_name,_Y("zz")); } dsc->de.d_reclen = sizeof(struct yaffs_dirent); @@ -1680,7 +1809,7 @@ int yaffs_closedir(yaffs_DIR *dirp) return 0; } -// end of directory stuff +/* End of directory stuff */ int yaffs_symlink(const YCHAR *oldpath, const YCHAR *newpath) @@ -1689,16 +1818,18 @@ int yaffs_symlink(const YCHAR *oldpath, const YCHAR *newpath) yaffs_Object *obj; YCHAR *name; int retVal= -1; - int mode = 0; // ignore for now + int mode = 0; /* ignore for now */ yaffsfs_Lock(); parent = yaffsfs_FindDirectory(NULL,newpath,&name,0); - if(parent){ + if(parent && parent->myDev->readOnly) + yaffsfs_SetError(-EINVAL); + else if(parent){ obj = yaffs_MknodSymLink(parent,name,mode,0,0,oldpath); if(obj) retVal = 0; else{ - yaffsfs_SetError(-ENOSPC); // just assume no space for now + yaffsfs_SetError(-ENOSPC); /* just assume no space for now */ retVal = -1; } } else { @@ -1740,7 +1871,7 @@ int yaffs_readlink(const YCHAR *path, YCHAR *buf, int bufsiz) int yaffs_link(const YCHAR *oldpath, const YCHAR *newpath) { - // Creates a link called newpath to existing oldpath + /* Creates a link called newpath to existing oldpath */ yaffs_Object *obj = NULL; yaffs_Object *target = NULL; int retVal = 0; @@ -1755,6 +1886,9 @@ int yaffs_link(const YCHAR *oldpath, const YCHAR *newpath) if(!obj) { yaffsfs_SetError(-ENOENT); retVal = -1; + } else if(obj->myDev->readOnly){ + yaffsfs_SetError(-EINVAL); + retVal= -1; } else if(target) { yaffsfs_SetError(-EEXIST); retVal = -1;