[oe-commits] org.oe.dev add yaffs2-utils[-native]

mickeyl commit openembedded-commits at lists.openembedded.org
Tue Sep 11 14:04:02 UTC 2007


add yaffs2-utils[-native]

Author: mickeyl at openembedded.org
Branch: org.openembedded.dev
Revision: f54b6b18bb3a1049fe996ed2d380018b3e8250bf
ViewMTN: http://monotone.openembedded.org/revision/info/f54b6b18bb3a1049fe996ed2d380018b3e8250bf
Files:
1
packages/yaffs2
packages/yaffs2/files
packages/yaffs2/files/mkyaffs2image.patch
packages/yaffs2/files/yaffs2-unioob.patch
packages/yaffs2/yaffs2-utils-native_cvs.bb
packages/yaffs2/yaffs2-utils_cvs.bb
Diffs:

#
# mt diff -rb953369d620df2ede89744d1f9a6a0671cda91b2 -rf54b6b18bb3a1049fe996ed2d380018b3e8250bf
#
# 
# 
# add_dir "packages/yaffs2"
# 
# add_dir "packages/yaffs2/files"
# 
# add_file "packages/yaffs2/files/mkyaffs2image.patch"
#  content [9a9841a1ed6d0f0946853dbd5d7a1b2c530fb308]
# 
# add_file "packages/yaffs2/files/yaffs2-unioob.patch"
#  content [9eafeee087e5fab362a3ca0a94fba7597c96a346]
# 
# add_file "packages/yaffs2/yaffs2-utils-native_cvs.bb"
#  content [46031d55a3c90aaeedc2a3a4abd20ee6df77efd0]
# 
# add_file "packages/yaffs2/yaffs2-utils_cvs.bb"
#  content [92e886482cb0c4b4bbb43777f590da794398331e]
# 
============================================================
--- packages/yaffs2/files/mkyaffs2image.patch	9a9841a1ed6d0f0946853dbd5d7a1b2c530fb308
+++ packages/yaffs2/files/mkyaffs2image.patch	9a9841a1ed6d0f0946853dbd5d7a1b2c530fb308
@@ -0,0 +1,345 @@
+ * 
+ * mkyaffs2image hacks by NCB
+ *
+ * Changes by Sergey Kubushin flagged KSI
+ *
+ */
+ 
+/* KSI:
+ * All this nightmare should be rewritten from ground up. Why save return
+ * values if nobody checks them? The read/write function returns only one
+ * error, -1. Positive return value does NOT mean read/write operation has
+ * been completed successfully. If somebody opens files, he MUST close them
+ * when they are not longer needed. Only those brave enough can write 64
+ * bytes from a yaffs_PackedTags2 structure. The list is too long, there is
+ * enough bugs here to write a couple of thick books on how NOT to write
+ * programs...
+ *
+ * And BTW, what was one supposed to do with that file that this horror
+ * occasionally managed to generate?
+ */
+diff -urN yaffs2.orig/utils/mkyaffs2image.c yaffs2/utils/mkyaffs2image.c
+--- yaffs2.orig/utils/mkyaffs2image.c   2005-12-12 16:34:58.000000000 -0800
++++ yaffs2/utils/mkyaffs2image.c    2006-02-10 16:56:13.000000000 -0800
+@@ -31,10 +47,10 @@
+ #include <dirent.h>
+ #include <string.h>
+ #include <unistd.h>
++#include <mtd/mtd-user.h>
+ #include "yaffs_ecc.h"
+ #include "yaffs_guts.h"
+ 
+-#include "yaffs_tagsvalidity.h"
+ #include "yaffs_packedtags2.h"
+ 
+ unsigned yaffs_traceMask=0;
+@@ -43,9 +59,45 @@
+ 
+ #define chunkSize 2048
+ #define spareSize 64
++#define PT2_BYTES 25
+ 
+ const char * mkyaffsimage_c_version = "$Id: mkyaffs2image.c,v 1.4 2007-02-14 01:09:06 wookey Exp $";
+ 
++static int	layout_no;
++
++static struct nand_oobinfo oob_layout[] = {
++        /* KSI:
++         * Dummy "raw" layout - no ECC, all the bytes are free. Does NOT
++         * really work, only used for compatibility with CVS YAFFS2 that
++         * never ever worked with any stock MTD.
++         */
++        {
++        	.useecc = MTD_NANDECC_AUTOPLACE,
++        	.eccbytes = 0,
++        	.eccpos = {},
++        	.oobfree = { {0, 64} }
++        },
++        /* KSI:
++         * Regular MTD AUTOPLACED ECC for large page NAND devices, the
++         * only one existing in stock MTD so far. It corresponds to layout# 1
++         * in command line arguments. Any other layouts could be added to
++         * the list when they made their way in kernel's MTD. The structure
++         * is simply copied from kernel's drivers/mtd/nand/nand_base.c as-is.
++         */
++        {
++        	.useecc = MTD_NANDECC_AUTOPLACE,
++        	.eccbytes = 24,
++        	.eccpos = {
++	        	40, 41, 42, 43, 44, 45, 46, 47,
++	        	48, 49, 50, 51, 52, 53, 54, 55,
++	        	56, 57, 58, 59, 60, 61, 62, 63},
++        	.oobfree = { {2, 38} }
++        },
++        /* End-of-list marker */
++        {
++                .useecc = -1,
++        }
++};
+ 
+ typedef struct
+ {
+@@ -59,7 +111,7 @@
+ static int n_obj = 0;
+ static int obj_id = YAFFS_NOBJECT_BUCKETS + 1;
+ 
+-static int nObjects, nDirectories, nPages;
++static int nObjects = 0, nDirectories = 0, nPages = 0;
+ 
+ static int outFile;
+ 
+@@ -123,6 +175,11 @@
+ 	return -1;
+ }
+ 
++/* KSI:
++ * No big endian for now. This is left for a later time. The existing code
++ * is FUBAR.
++ */
++#if 0
+ /* This little function converts a little endian tag to a big endian tag.
+  * NOTE: The tag is not usable after this other than calculating the CRC
+  * with.
+@@ -155,11 +212,56 @@
+     tags->asBytes[7] = temp.asBytes[7];
+ #endif
+ }
++#endif
++
++void nandmtd2_pt2buf(unsigned char *buf, yaffs_PackedTags2 *pt)
++{
++	int		i, j = 0, k, n;
++	unsigned char	pt2_byte_buf[PT2_BYTES];
++	
++	*((unsigned int *) &pt2_byte_buf[0]) = pt->t.sequenceNumber;
++	*((unsigned int *) &pt2_byte_buf[4]) = pt->t.objectId;
++	*((unsigned int *) &pt2_byte_buf[8]) = pt->t.chunkId;
++	*((unsigned int *) &pt2_byte_buf[12]) = pt->t.byteCount;
++	pt2_byte_buf[16] = pt->ecc.colParity;
++	pt2_byte_buf[17] = pt->ecc.lineParity & 0xff;
++	pt2_byte_buf[18] = (pt->ecc.lineParity >> 8) & 0xff;
++	pt2_byte_buf[19] = (pt->ecc.lineParity >> 16) & 0xff;
++	pt2_byte_buf[20] = (pt->ecc.lineParity >> 24) & 0xff;
++	pt2_byte_buf[21] = pt->ecc.lineParityPrime & 0xff;
++	pt2_byte_buf[22] = (pt->ecc.lineParityPrime >> 8) & 0xff;
++	pt2_byte_buf[23] = (pt->ecc.lineParityPrime >> 16) & 0xff;
++	pt2_byte_buf[24] = (pt->ecc.lineParityPrime >> 24) & 0xff;
++
++	k = oob_layout[layout_no].oobfree[j][0];
++	n = oob_layout[layout_no].oobfree[j][1];
++		
++	if (n == 0) {
++		fprintf(stderr, "No OOB space for tags");
++		exit(-1);
++	}
++                                
++	for (i = 0; i < PT2_BYTES; i++) {
++		if (n == 0) {
++			j++;
++			k = oob_layout[layout_no].oobfree[j][0];
++			n = oob_layout[layout_no].oobfree[j][1];
++			if (n == 0) {
++				fprintf(stderr, "No OOB space for tags");
++				exit(-1);
++			}
++		}
++		buf[k++] = pt2_byte_buf[i];
++		n--;
++	}
++}
+ 
+ static int write_chunk(__u8 *data, __u32 objId, __u32 chunkId, __u32 nBytes)
+ {
+ 	yaffs_ExtendedTags t;
+ 	yaffs_PackedTags2 pt;
++	unsigned char	spare_buf[spareSize];
++
+ 
+ 	error = write(outFile,data,chunkSize);
+ 	if(error < 0) return error;
+@@ -177,18 +279,27 @@
+ // added NCB **CHECK**
+ 	t.chunkUsed = 1;
+ 
++/* KSI: Broken anyway -- e.g. &t is pointer to a wrong type... */
++#if 0
+ 	if (convert_endian)
+ 	{
+     	    little_to_big_endian(&t);
+ 	}
++#endif
+ 
+ 	nPages++;
+ 
+ 	yaffs_PackTags2(&pt,&t);
+-	
+-//	return write(outFile,&pt,sizeof(yaffs_PackedTags2));
+-	return write(outFile,&pt,spareSize);
+-	
++
++        memset(spare_buf, 0xff, sizeof(spare_buf));
++        
++        if (layout_no == 0) {
++                memcpy(spare_buf, &pt, sizeof(yaffs_PackedTags2));
++        } else {
++                nandmtd2_pt2buf(spare_buf, &pt);
++        }	
++
++	return write(outFile,spare_buf,spareSize);
+ }
+ 
+ #define SWAP32(x)   ((((x) & 0x000000FF) << 24) | \
+@@ -199,6 +310,8 @@
+ #define SWAP16(x)   ((((x) & 0x00FF) << 8) | \
+                      (((x) & 0xFF00) >> 8))
+         
++/* KSI: Removed for now. TBD later when the proper util (from scratch) is written */
++#if 0
+ // This one is easier, since the types are more standard. No funky shifts here.
+ static void object_header_little_to_big_endian(yaffs_ObjectHeader* oh)
+ {
+@@ -256,6 +369,7 @@
+     oh->roomToGrow[11] = SWAP32(oh->roomToGrow[11]);
+ #endif
+ }
++#endif
+ 
+ static int write_object_header(int objId, yaffs_ObjectType t, struct stat *s, int parent, const char *name, int equivalentObj, const char * alias)
+ {
+@@ -300,10 +414,13 @@
+ 		strncpy(oh->alias,alias,YAFFS_MAX_ALIAS_LENGTH);
+ 	}
+ 
++/* KSI: FUBAR. Left for a leter time. */
++#if 0
+ 	if (convert_endian)
+ 	{
+     		object_header_little_to_big_endian(oh);
+ 	}
++#endif
+ 	
+ 	return write_chunk(bytes,objId,0,0xffff);
+ 	
+@@ -319,7 +436,7 @@
+ 	nDirectories++;
+ 	
+ 	dir = opendir(path);
+-	
++
+ 	if(dir)
+ 	{
+ 		while((entry = readdir(dir)) != NULL)
+@@ -403,12 +520,12 @@
+ 									   error = nBytes;
+ 									   
+ 									printf("%d data chunks written\n",chunk);
++        								close(h);
+ 								}
+ 								else
+ 								{
+ 									perror("Error opening file");
+ 								}
+-								close(h);
+ 								
+ 							}							
+ 														
+@@ -448,58 +565,82 @@
+ 				}
+ 			}
+ 		}
++		/* KSI:
++		 * Who is supposed to close those open directories in this
++		 * recursive function, lord Byron? Stock "ulimit -n" is 1024
++		 * and e.g. stock Fedora /etc directory has more that 1024
++		 * directories...
++		 */
++		closedir(dir);
+ 	}
+ 	
+ 	return 0;
+ 
+ }
+ 
++void usage(void)
++{
++	printf("usage: mkyaffs2image layout# dir image_file [convert]\n");
++	printf("	layout#    NAND OOB layout # (0 - raw, 1 - nand_oob_64)\n");
++	printf("	dir        the directory tree to be converted\n");
++	printf("	image_file the output file to hold the image\n");
++	printf("	'convert'  make a big-endian img on a little-endian machine. BROKEN !\n");
++	exit(1);
++}
+ 
+ int main(int argc, char *argv[])
+ {
+ 	struct stat stats;
++	int	i;
+ 	
+ 	printf("mkyaffs2image: image building tool for YAFFS2 built "__DATE__"\n");
+ 	
+-	if(argc < 3)
++	if ((argc < 4) || (sscanf(argv[1], "%u", &layout_no) != 1))
+ 	{
+-		printf("usage: mkyaffs2image dir image_file [convert]\n");
+-		printf("           dir        the directory tree to be converted\n");
+-		printf("           image_file the output file to hold the image\n");
+-        printf("           'convert'  produce a big-endian image from a little-endian machine\n");
+-		exit(1);
++	        usage();
+ 	}
+ 
+-    if ((argc == 4) && (!strncmp(argv[3], "convert", strlen("convert"))))
+-    {
+-        convert_endian = 1;
+-    }
++	i = 0;
++	
++	while (oob_layout[i].useecc != -1)
++	        i++;
++	        
++        if (layout_no >= i)
++                usage();
++
++	if ((argc == 5) && (!strncmp(argv[4], "convert", strlen("convert"))))
++	{
++	        /* KSI: Broken as of now. TBD. Fail. */
++	        usage();
++                convert_endian = 1;
++        }
+     
+-	if(stat(argv[1],&stats) < 0)
++	if(stat(argv[2],&stats) < 0)
+ 	{
+-		printf("Could not stat %s\n",argv[1]);
++		printf("Could not stat %s\n",argv[2]);
+ 		exit(1);
+ 	}
+ 	
+ 	if(!S_ISDIR(stats.st_mode))
+ 	{
+-		printf(" %s is not a directory\n",argv[1]);
++		printf(" %s is not a directory\n",argv[2]);
+ 		exit(1);
+ 	}
+ 	
+-	outFile = open(argv[2],O_CREAT | O_TRUNC | O_WRONLY, S_IREAD | S_IWRITE);
++	outFile = open(argv[3],O_CREAT | O_TRUNC | O_WRONLY, S_IREAD | S_IWRITE);
+ 	
+ 	
+ 	if(outFile < 0)
+ 	{
+-		printf("Could not open output file %s\n",argv[2]);
++		printf("Could not open output file %s\n",argv[3]);
+ 		exit(1);
+ 	}
+ 	
+-	printf("Processing directory %s into image file %s\n",argv[1],argv[2]);
++	printf("Processing directory %s into image file %s\n",argv[2],argv[3]);
+ 	error =  write_object_header(1, YAFFS_OBJECT_TYPE_DIRECTORY, &stats, 1,"", -1, NULL);
++
+ 	if(error)
+-	error = process_directory(YAFFS_OBJECTID_ROOT,argv[1]);
++	error = process_directory(YAFFS_OBJECTID_ROOT,argv[2]);
+ 	
+ 	close(outFile);
+ 	
============================================================
--- packages/yaffs2/files/yaffs2-unioob.patch	9eafeee087e5fab362a3ca0a94fba7597c96a346
+++ packages/yaffs2/files/yaffs2-unioob.patch	9eafeee087e5fab362a3ca0a94fba7597c96a346
@@ -0,0 +1,216 @@
+diff -urN yaffs2.orig/yaffs_mtdif2.c yaffs2/yaffs_mtdif2.c
+--- yaffs2.orig/yaffs_mtdif2.c	2005-12-07 14:00:38.000000000 -0800
++++ yaffs2/yaffs_mtdif2.c	2006-02-10 17:13:58.000000000 -0800
+@@ -29,6 +29,130 @@
+ 
+ #include "yaffs_packedtags2.h"
+ 
++#define PT2_BYTES		25
++
++void nandmtd2_pt2buf(yaffs_Device *dev, yaffs_PackedTags2 *pt, int is_raw)
++{
++	struct mtd_info *mtd = (struct mtd_info *)(dev->genericDevice);
++	int	i, j = 0, k, n;
++	__u8	pt2_byte_buf[PT2_BYTES];
++	
++	/* Pack buffer with 0xff */
++	for (i = 0; i < mtd->oobsize; i++)
++		dev->spareBuffer[i] = 0xff;
++		
++	if (!is_raw) {
++		*((unsigned int *) &dev->spareBuffer[0]) = pt->t.sequenceNumber;
++		*((unsigned int *) &dev->spareBuffer[4]) = pt->t.objectId;
++		*((unsigned int *) &dev->spareBuffer[8]) = pt->t.chunkId;
++		*((unsigned int *) &dev->spareBuffer[12]) = pt->t.byteCount;
++		dev->spareBuffer[16] = pt->ecc.colParity;
++		dev->spareBuffer[17] = pt->ecc.lineParity & 0xff;
++		dev->spareBuffer[18] = (pt->ecc.lineParity >> 8) & 0xff;
++		dev->spareBuffer[19] = (pt->ecc.lineParity >> 16) & 0xff;
++		dev->spareBuffer[20] = (pt->ecc.lineParity >> 24) & 0xff;
++		dev->spareBuffer[21] = pt->ecc.lineParityPrime & 0xff;
++		dev->spareBuffer[22] = (pt->ecc.lineParityPrime >> 8) & 0xff;
++		dev->spareBuffer[23] = (pt->ecc.lineParityPrime >> 16) & 0xff;
++		dev->spareBuffer[24] = (pt->ecc.lineParityPrime >> 24) & 0xff;
++	} else {
++		*((unsigned int *) &pt2_byte_buf[0]) = pt->t.sequenceNumber;
++		*((unsigned int *) &pt2_byte_buf[4]) = pt->t.objectId;
++		*((unsigned int *) &pt2_byte_buf[8]) = pt->t.chunkId;
++		*((unsigned int *) &pt2_byte_buf[12]) = pt->t.byteCount;
++		pt2_byte_buf[16] = pt->ecc.colParity;
++		pt2_byte_buf[17] = pt->ecc.lineParity & 0xff;
++		pt2_byte_buf[18] = (pt->ecc.lineParity >> 8) & 0xff;
++		pt2_byte_buf[19] = (pt->ecc.lineParity >> 16) & 0xff;
++		pt2_byte_buf[20] = (pt->ecc.lineParity >> 24) & 0xff;
++		pt2_byte_buf[21] = pt->ecc.lineParityPrime & 0xff;
++		pt2_byte_buf[22] = (pt->ecc.lineParityPrime >> 8) & 0xff;
++		pt2_byte_buf[23] = (pt->ecc.lineParityPrime >> 16) & 0xff;
++		pt2_byte_buf[24] = (pt->ecc.lineParityPrime >> 24) & 0xff;
++
++		k = mtd->oobinfo.oobfree[j][0];
++		n = mtd->oobinfo.oobfree[j][1];
++		
++		if (n == 0) {
++			T(YAFFS_TRACE_ERROR, (TSTR("No OOB space for tags" TENDSTR)));
++			YBUG();
++		}
++                                
++		for (i = 0; i < PT2_BYTES; i++) {
++			if (n == 0) {
++				j++;
++				k = mtd->oobinfo.oobfree[j][0];
++				n = mtd->oobinfo.oobfree[j][1];
++				if (n == 0) {
++					T(YAFFS_TRACE_ERROR, (TSTR("No OOB space for tags" TENDSTR)));
++					YBUG();
++				}
++			}
++			dev->spareBuffer[k++] = pt2_byte_buf[i];
++			n--;
++		}
++	}
++}
++
++void nandmtd2_buf2pt(yaffs_Device *dev, yaffs_PackedTags2 *pt, int is_raw)
++{
++	struct mtd_info *mtd = (struct mtd_info *)(dev->genericDevice);
++	int	i, j = 0, k, n;
++	__u8	pt2_byte_buf[PT2_BYTES];
++	
++
++	if (!is_raw) {
++		pt->t.sequenceNumber = *((unsigned int *) &dev->spareBuffer[0]);
++		pt->t.objectId = *((unsigned int *) &dev->spareBuffer[4]);
++		pt->t.chunkId = *((unsigned int *) &dev->spareBuffer[8]);
++		pt->t.byteCount = *((unsigned int *) &dev->spareBuffer[12]);
++		pt->ecc.colParity = dev->spareBuffer[16];
++		pt->ecc.lineParity = (dev->spareBuffer[17] & 0x000000ff) |
++	 		((dev->spareBuffer[18] << 8) & 0x0000ff00) |
++	 		((dev->spareBuffer[19] << 16) & 0x00ff0000) |
++	 		((dev->spareBuffer[20] << 24) & 0xff000000);
++		pt->ecc.lineParityPrime = (dev->spareBuffer[21] & 0x000000ff) |
++	 		((dev->spareBuffer[22] << 8) & 0x0000ff00) |
++	 		((dev->spareBuffer[23] << 16) & 0x00ff0000) |
++	 		((dev->spareBuffer[24] << 24) & 0xff000000);
++	} else {
++		k = mtd->oobinfo.oobfree[j][0];
++		n = mtd->oobinfo.oobfree[j][1];
++		
++		if (n == 0) {
++			T(YAFFS_TRACE_ERROR, (TSTR("No space in OOB for tags" TENDSTR)));
++			YBUG();
++		}
++                                
++		for (i = 0; i < PT2_BYTES; i++) {
++			if (n == 0) {
++				j++;
++				k = mtd->oobinfo.oobfree[j][0];
++				n = mtd->oobinfo.oobfree[j][1];
++				if (n == 0) {
++					T(YAFFS_TRACE_ERROR, (TSTR("No space in OOB for tags" TENDSTR)));
++					YBUG();
++				}
++			}
++			pt2_byte_buf[i] = dev->spareBuffer[k++];
++			n--;
++		}
++		pt->t.sequenceNumber = *((unsigned int *) &pt2_byte_buf[0]);
++		pt->t.objectId = *((unsigned int *) &pt2_byte_buf[4]);
++		pt->t.chunkId = *((unsigned int *) &pt2_byte_buf[8]);
++		pt->t.byteCount = *((unsigned int *) &pt2_byte_buf[12]);
++		pt->ecc.colParity = pt2_byte_buf[16];
++		pt->ecc.lineParity = (pt2_byte_buf[17] & 0x000000ff) |
++	 		((pt2_byte_buf[18] << 8) & 0x0000ff00) |
++	 		((pt2_byte_buf[19] << 16) & 0x00ff0000) |
++	 		((pt2_byte_buf[20] << 24) & 0xff000000);
++		pt->ecc.lineParityPrime = (pt2_byte_buf[21] & 0x000000ff) |
++	 		((pt2_byte_buf[22] << 8) & 0x0000ff00) |
++	 		((pt2_byte_buf[23] << 16) & 0x00ff0000) |
++	 		((pt2_byte_buf[24] << 24) & 0xff000000);
++	}
++}
++
+ int nandmtd2_WriteChunkWithTagsToNAND(yaffs_Device * dev, int chunkInNAND,
+ 				      const __u8 * data,
+ 				      const yaffs_ExtendedTags * tags)
+@@ -51,24 +175,22 @@
+ 	}
+ 
+ 	if (data && tags) {
+-		if (dev->useNANDECC)
+-			retval =
+-			    mtd->write_ecc(mtd, addr, dev->nBytesPerChunk,
+-					   &dummy, data, (__u8 *) & pt, NULL);
+-		else
++			nandmtd2_pt2buf(dev, &pt, 0);
+ 			retval =
+ 			    mtd->write_%s
>>> DIFF TRUNCATED @ 16K






More information about the Openembedded-commits mailing list