)]}' { "log": [ { "commit": "29a6ccca3869bbe33879dae0cd7df2a1559eff54", "tree": "2d9d355d8662ede95af7bc812d686dc4d5f37ff3", "parents": [ "426048313dfa7d65dbd2379b1665755511f9544f", "6a8a98b22b10f1560d5f90aded4a54234b9b2724" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 27 20:06:53 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 27 20:06:53 2011 -0700" }, "message": "Merge git://git.infradead.org/mtd-2.6\n\n* git://git.infradead.org/mtd-2.6: (97 commits)\n mtd: kill CONFIG_MTD_PARTITIONS\n mtd: remove add_mtd_partitions, add_mtd_device and friends\n mtd: convert remaining users to mtd_device_register()\n mtd: samsung onenand: convert to mtd_device_register()\n mtd: omap2 onenand: convert to mtd_device_register()\n mtd: txx9ndfmc: convert to mtd_device_register()\n mtd: tmio_nand: convert to mtd_device_register()\n mtd: socrates_nand: convert to mtd_device_register()\n mtd: sharpsl: convert to mtd_device_register()\n mtd: s3c2410 nand: convert to mtd_device_register()\n mtd: ppchameleonevb: convert to mtd_device_register()\n mtd: orion_nand: convert to mtd_device_register()\n mtd: omap2: convert to mtd_device_register()\n mtd: nomadik_nand: convert to mtd_device_register()\n mtd: ndfc: convert to mtd_device_register()\n mtd: mxc_nand: convert to mtd_device_register()\n mtd: mpc5121_nfc: convert to mtd_device_register()\n mtd: jz4740_nand: convert to mtd_device_register()\n mtd: h1910: convert to mtd_device_register()\n mtd: fsmc_nand: convert to mtd_device_register()\n ...\n\nFixed up trivial conflicts in\n - drivers/mtd/maps/integrator-flash.c: removed in ARM tree\n - drivers/mtd/maps/physmap.c: addition of afs partition probe type\n clashing with removal of CONFIG_MTD_PARTITIONS\n" }, { "commit": "ee0e87b174bb41f0310cf089262bf5dd8f95a212", "tree": "444b7eb1cc1a807561889a4cffe15fde11761645", "parents": [ "6b57c11601c8fa4bfa046513c4df155b3b58ea89" ], "author": { "name": "Jamie Iles", "email": "jamie@jamieiles.com", "time": "Mon May 23 10:23:40 2011 +0100" }, "committer": { "name": "David Woodhouse", "email": "David.Woodhouse@intel.com", "time": "Wed May 25 02:25:00 2011 +0100" }, "message": "mtd: convert remaining users to mtd_device_register()\n\nThe older add_mtd_device()/add_mtd_partitions() and their removal\ncounterparts will soon be gone. Replace uses with mtd_device_register()\nand mtd_device_unregister().\n\nSigned-off-by: Jamie Iles \u003cjamie@jamieiles.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nSigned-off-by: David Woodhouse \u003cDavid.Woodhouse@intel.com\u003e\n" }, { "commit": "ab50ff684707031ed4bad2fdd313208ae392e5bb", "tree": "8e16e8f0c858a2b3dde789aafd3a98167cf0193b", "parents": [ "ebfce01a6dfa2bd30a16efdeb417862ea0355d3b" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue May 17 15:46:01 2011 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri May 20 08:30:34 2011 +0300" }, "message": "UBI: switch to dynamic printks\n\nRemove custom dynamic prints and the module parameter to toggle them and use\nthe generic kernel dynamic printk infrastructure.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "ebfce01a6dfa2bd30a16efdeb417862ea0355d3b", "tree": "5fb8690771e2b75a1945b150124271aa250f91dc", "parents": [ "3802243a189b0f153acb03f531daf35d40bc38bb" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue May 17 13:26:24 2011 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri May 20 08:30:34 2011 +0300" }, "message": "UBI: turn some macros into static inline\n\nSimilarly as we have done for UBIFS in\n1dcffad74183bb00e8129ba1c5bb2c9931d31bd7, turn the debugging macros into static\ninline functions, for the same reasons: to avoid gcc 4.5 warnings.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "3802243a189b0f153acb03f531daf35d40bc38bb", "tree": "073eb1a7fcc3304ab048eb540d87565feadc0bfb", "parents": [ "1426414431a8d37a6e631e0b5e2ad6186b81876a" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue May 17 12:50:08 2011 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri May 20 08:30:34 2011 +0300" }, "message": "UBI: improve checking in debugging prints\n\nWhen debugging is disabled, define debugging prints as if (0) printk() to make\nsure that the compiler still the format string in debugging messages even if\ndebugging is disabled.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "1426414431a8d37a6e631e0b5e2ad6186b81876a", "tree": "dc4426f2ff8d56b96b3d4f2de1bb998638e86494", "parents": [ "feddbb34ebd75e9b6bf573b852079e327a88c07a" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Apr 14 11:36:31 2011 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Apr 14 13:38:59 2011 +0300" }, "message": "UBI: fix typo in a message\n\nWhen a PEB passes the torture test, UBI prints\n\"do not mark it a bad\", but should print \"do not mark it as bad\".\nThis patch corrects the typo.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "feddbb34ebd75e9b6bf573b852079e327a88c07a", "tree": "99b11dc996e6ce633c332a165a073e76cb05601c", "parents": [ "3627924acf70a9a26587712e4888ee7144489678" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Mar 28 10:12:25 2011 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Apr 14 11:34:41 2011 +0300" }, "message": "UBI: fix minor stylistic issues\n\nFix checkpatch.pl errors and warnings:\n\n* space before tab\n* line over 80 characters\n* include linux/ioctl.h instead of asm/ioctl.h\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "3627924acf70a9a26587712e4888ee7144489678", "tree": "1804290f2659105b2bd00e4d31f677f43f7d0dc1", "parents": [ "e8e088de305d7cc00b2c8b2a857ceb62d5fa68d3" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Mar 28 10:04:09 2011 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Apr 14 11:34:41 2011 +0300" }, "message": "UBI: use __packed instead of __attribute__((packed))\n\nThere was an attempt to standartize various \"__attribute__\" and\nother macros in order to have potentially portable and more\nconsistent code, see commit 82ddcb040570411fc2d421d96b3e69711c670328.\n\nNote, that commit refers Rober Love\u0027s blog post, but the URL\nis broken, the valid URL is:\nhttp://blog.rlove.org/2005/10/with-little-help-from-your-compiler.html\n\nMoreover, nowadays checkpatch.pl warns about using\n__attribute__((packed)):\n\n\"WARNING: __packed is preferred over __attribute__((packed))\"\n\nIt is not a big deal for UBI to use __packed, so let\u0027s do it.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "6748482f4153fc0e095aa3dc831d5edac5656a80", "tree": "7b1a276c26ac3b4171d56100e059d693a9963be3", "parents": [ "e10b376e98332edcc2530aaed384a7e248477052" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Mar 15 16:25:38 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Apr 14 11:34:40 2011 +0300" }, "message": "UBI: re-name set volume properties ioctl\n\nRename the ioctl which sets volume properties from \u0027UBI_IOCSETPROP\u0027 to\n\u0027UBI_IOCSETVOLPROP\u0027 to reflect the fact that this ioctl is about volume\nproperties, not device properties. This is also consistent with the\nother volume ioctl name - \u0027UBI_IOCVOLUP\u0027.\n\nThe main motivation for the re-name, however, is that we are going\nto introduce the per-UBI device \"set properties\" ioctl, so we need\ngood and logical naming.\n\nAt the same time, re-name the \"set volume properties request\" data\nstructure from \u0027struct ubi_set_prop_req\u0027 to\n\u0027struct ubi_set_vol_prop_req\u0027.\n\nAnd re-name \u0027UBI_PROP_DIRECT_WRITE\u0027 to \u0027UBI_VOL_PROP_DIRECT_WRITE\u0027.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "e10b376e98332edcc2530aaed384a7e248477052", "tree": "2a84d4397dec41c4e298738c704ab694d30326ad", "parents": [ "a6360dd37e1a144ed11e6548371bade559a1e4df" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Mar 21 16:50:18 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Apr 14 11:34:40 2011 +0300" }, "message": "UBI: make the control character device non-seekable\n\nThis patch makes the UBI control device (/dev/ubi_ctrl) non-seekable.\nThe seek operation does is not applicable to this file, so it is\ncleaner to explicitly return error (which the added \u0027no_llseek()\u0027)\ndoes than trying to change the position (which the removed\n\u0027default_llseek()\u0027 does).\n\nThis is an API break, but the only known user of this interface is\nmtd-utils which does not need the seeking functionality. And any app\nwhich relies on this is broken, but I\u0027m not aware of such apps.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "ccfeef0ff76ebd632ae51bc56700f0072c4f1864", "tree": "83ec5932dac631a455a69212b505f68f57f68cdf", "parents": [ "42933bac11e811f02200c944d8562a15f8ec4ff0", "6bef0b67474d71e0d6484cbabcc87657a1176d8d" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Apr 07 11:31:03 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Apr 07 11:31:03 2011 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.infradead.org/ubifs-2.6\n\n* \u0027for-linus\u0027 of git://git.infradead.org/ubifs-2.6:\n UBI: do not select KALLSYMS_ALL\n UBI: do not compare array with NULL\n UBI: check if we are in RO mode in the erase routine\n UBIFS: fix debugging failure in dbg_check_space_info\n UBIFS: fix error path in dbg_debugfs_init_fs\n UBIFS: unify error path dbg_debugfs_init_fs\n UBIFS: do not select KALLSYMS_ALL\n UBIFS: fix assertion warnings\n UBIFS: fix oops on error path in read_pnode\n UBIFS: do not read flash unnecessarily\n" }, { "commit": "6bef0b67474d71e0d6484cbabcc87657a1176d8d", "tree": "1d61b5b23842cfdeb1c91fd2825fc5edab627f71", "parents": [ "6e5133cc757912e7ba2bfbbfb384667707f45ec3" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 30 11:27:08 2011 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Apr 05 11:08:59 2011 +0300" }, "message": "UBI: do not select KALLSYMS_ALL\n\nAll UBI needs is to make sure we stacktraces when UBI debugging\nis enabled. It is enough to select KALLSYMS for this, KALLSYMS_ALL\nis not necessary.\n\nAnd the current Kconfig line we have:\n\nselect KALLSYMS_ALL if KALLSYMS \u0026\u0026 DEBUG_KERNEL\n\nis just too complex to be sane and right. But this \"if\" part there\nis needed to prevent \"unmet direct dependency\" warnings, because\nKALLSYMS_ALL depends on KALLSYMS and DEBUG_KERNEL, so we cannot\njust select KALLSYMS_ALL.\n\nAnyway, this feels messy, and we do not seem to really need KALLSYMS_ALL,\nso select KALLSYMS instead.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nAcked-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\n" }, { "commit": "6e5133cc757912e7ba2bfbbfb384667707f45ec3", "tree": "62ff05e2d236d7c107633fd2e27e2d1155c7088d", "parents": [ "3efe509070e3d27e6d5dbc4bf8588e9453e9b949" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Mar 25 18:48:59 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Apr 05 11:08:58 2011 +0300" }, "message": "UBI: do not compare array with NULL\n\nCoverity spotted that UBI debugging code tries to compare\nan array and NULL, which obviously makes little sense. Kill\nthis check.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "3efe509070e3d27e6d5dbc4bf8588e9453e9b949", "tree": "d383315c70461be6e6366c6e8142db95e113a12b", "parents": [ "7da6443aca9be29c6948dcbd636ad50154d0bc0c" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Mar 18 18:11:42 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Apr 05 11:08:58 2011 +0300" }, "message": "UBI: check if we are in RO mode in the erase routine\n\n\u0027do_sync_erase()\u0027 has to check whether we are in R/O mode before\nerasing the PEB. This patch adds the check and while on it, adds an\nassertion which validates the \u0027pnum\u0027 argument, as well as removes\na check which is always true because it has already been done\nfew lines before.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "25985edcedea6396277003854657b5f3cb31a628", "tree": "f026e810210a2ee7290caeb737c23cb6472b7c38", "parents": [ "6aba74f2791287ec407e0f92487a725a25908067" ], "author": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Wed Mar 30 22:57:33 2011 -0300" }, "committer": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Thu Mar 31 11:26:23 2011 -0300" }, "message": "Fix common misspellings\n\nFixes generated by \u0027codespell\u0027 and manually reviewed.\n\nSigned-off-by: Lucas De Marchi \u003clucas.demarchi@profusion.mobi\u003e\n" }, { "commit": "3d46b316bcc1ecb17f8e7874a8ec620c64864898", "tree": "0f451eb03a5f2d972842917099aead265687d0ef", "parents": [ "4bbba111d94781d34081c37856bbc5eb33f6c72a" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Mar 24 16:09:56 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Mar 24 16:09:56 2011 +0200" }, "message": "UBI: use GFP_NOFS properly\n\nThis patch fixes a brown-paperbag bug which was introduced by me:\nI used incorrect \"GFP_KERNEL | GFP_NOFS\" allocation flags to make\nsure my allocations do not cause write-back. But the correct form\nis \"GFP_NOFS\".\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "8f627a8a881481598c2591c3acc122fb9be7bac4", "tree": "06497d25e30824500aeaf8c736c45b070f121234", "parents": [ "fd57ed021990157ee5b3997c3f21c734093a9e23", "5d630e43284fdb0613e4e7e7dd906f27bc25b6af" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 18 10:50:27 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 18 10:50:27 2011 -0700" }, "message": "Merge branch \u0027linux-next\u0027 of git://git.infradead.org/ubifs-2.6\n\n* \u0027linux-next\u0027 of git://git.infradead.org/ubifs-2.6: (25 commits)\n UBIFS: clean-up commentaries\n UBIFS: save 128KiB or more RAM\n UBIFS: allocate orphans scan buffer on demand\n UBIFS: allocate lpt dump buffer on demand\n UBIFS: allocate ltab checking buffer on demand\n UBIFS: allocate scanning buffer on demand\n UBIFS: allocate dump buffer on demand\n UBIFS: do not check data crc by default\n UBIFS: simplify UBIFS Kconfig menu\n UBIFS: print max. index node size\n UBIFS: handle allocation failures in UBIFS write path\n UBIFS: use max_write_size during recovery\n UBIFS: use max_write_size for write-buffers\n UBIFS: introduce write-buffer size field\n UBI: incorporate LEB offset information\n UBIFS: incorporate maximum write size\n UBI: provide LEB offset information\n UBI: incorporate maximum write size\n UBIFS: fix LEB number in printk\n UBIFS: restrict world-writable debugfs files\n ...\n" }, { "commit": "fd57ed021990157ee5b3997c3f21c734093a9e23", "tree": "a32d18647b1becc1ee81fb04c5dd5dedeed4e1d7", "parents": [ "ca749e2af01bb3e6b94d441696903dc26c357443", "28237e4583604818294dc1ce7881db5f53377b9c" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 18 10:50:02 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 18 10:50:02 2011 -0700" }, "message": "Merge branch \u0027linux-next\u0027 of git://git.infradead.org/ubi-2.6\n\n* \u0027linux-next\u0027 of git://git.infradead.org/ubi-2.6:\n UBI: make tests modes dynamic\n UBI: make self-checks dynamic\n UBI: make debugging messages dynamic\n UBI: remove UBI_IO_DEBUG macro\n UBI: kill debugging buffer\n UBI: allocate erase checking buffer on demand\n UBI: allocate write checking buffer on demand\n UBI: always re-read in case of read failures\n UBI: cleanup comments about corrupted PEBs\n UBI: add slab cache for ubi_scan_leb objects\n UBI: use raw mtd read function in debugging code\n UBI: try to reveal buggy MTD drivers\n UBI: add a commentary about allocating VID header buffer on stack\n UBI: cleanup LEB start calculations\n UBI: fix NOR erase preparation quirk\n" }, { "commit": "28237e4583604818294dc1ce7881db5f53377b9c", "tree": "84fc5d22a1f4213824445253cc9702be96069b47", "parents": [ "92d124f5314913a21f7fa98b22ee457dab171edd" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Mar 15 10:30:40 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 16 13:50:17 2011 +0200" }, "message": "UBI: make tests modes dynamic\n\nSimilarly to the debugging checks and message, make the test modes\nbe dynamically selected via the \"debug_tsts\" module parameter or\nvia the \"/sys/module/ubi/parameters/debug_tsts\" sysfs file. This\nis consistent with UBIFS as well.\n\nAnd now, since all the Kconfig knobs became dynamic, we can remove\nthe Kconfig.debug file completely.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "92d124f5314913a21f7fa98b22ee457dab171edd", "tree": "8a8c187f12290424419f0a05caeb202cd73253cc", "parents": [ "b342efd4a49cef9cf1a260c1814aad97722f38f8" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Mar 14 18:17:40 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 16 13:50:16 2011 +0200" }, "message": "UBI: make self-checks dynamic\n\nThis patch adds a possibility to dynamically switch UBI self-checks\non and off, instead of toggling them compile-time from the configuration\nmenu. This is much more flexible, and consistent with UBIFS, and this\nalso simplifies UBI Kconfig menu and the code.\n\nThis patch introduces two levels of self-checks - general, which\nincludes all self-checks which are relatively fast, and I/O, which\nincludes write-verify checks and erase-verify checks, which are\nrelatively slow and involve flash I/O.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "b342efd4a49cef9cf1a260c1814aad97722f38f8", "tree": "c2fecee23143d6939a719c9e6c5728c6f8798665", "parents": [ "6f9fdf62db64b1e52e5b7a9f785554e8b877b65c" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Mar 11 14:33:23 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 16 13:50:16 2011 +0200" }, "message": "UBI: make debugging messages dynamic\n\nThis patch adds a possibility to dynamically select UBI debugging\nmessages, instead of selecting them compile-time from the configuration\nmenu. This is much more flexible, and consistent with UBIFS, and this\nalso simplifies UBI Kconfig menu and the code.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "6f9fdf62db64b1e52e5b7a9f785554e8b877b65c", "tree": "a42ef328d49bbdafac26c4d84eb4c8afba8be874", "parents": [ "6edb9793959fb547a15d5ffe6b142d9f0b3e41a6" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Mar 11 13:08:51 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 16 13:50:16 2011 +0200" }, "message": "UBI: remove UBI_IO_DEBUG macro\n\nThis additional little macro is used to print a bit more messages\nwhile scanning the media. However, we have the \u0027dbg_bld()\u0027 macro\nfor this, so we better us \u0027dbg_bld()\u0027 and kill UBI_IO_DEBUG. This\nsimplifies the code a tiny bit.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "6edb9793959fb547a15d5ffe6b142d9f0b3e41a6", "tree": "85ce150c27b728dbddcfbd303154c7e59c95e548", "parents": [ "332873d60b943c9bf53957c6e334038ac5e9dc6b" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Mar 14 17:11:29 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 16 13:50:16 2011 +0200" }, "message": "UBI: kill debugging buffer\n\nThis patch kills the \u0027ubi-\u003edbg_peb_buf\u0027 debugging buffer and the\nassociated mutex, because all users of this buffer are now gone.\nWe are killing this buffer because we are going to switch to\ndynamic debugging control, just like in UBIFS, which means that\nCONFIG_MTD_UBI_DEBUG_PARANOID will be removed. In this case we\u0027d\nend up always allocating \u0027ubi-\u003edbg_peb_buf\u0027, which is rather large\n(128KiB or more), and this would be wasteful. Thus, we are just\nkilling it.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "332873d60b943c9bf53957c6e334038ac5e9dc6b", "tree": "f930b7e315d478901d567ec4057cc78e789d3e2c", "parents": [ "a75867432a7eb2cdcaa8613a3b72b1d0594dd930" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Mar 14 17:09:40 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 16 13:50:16 2011 +0200" }, "message": "UBI: allocate erase checking buffer on demand\n\nInstead of using pre-allocated \u0027ubi-\u003edbg_peb_buf\u0027 buffer in\n\u0027ubi_dbg_check_all_ff()\u0027, dynamically allocate it when needed. The\nintend is to get rid of the pre-allocated \u0027ubi-\u003edbg_peb_buf\u0027 buffer\ncompletely. And the need for this arises because we want to change\nto dynamic debugging control instead of compile-time control, i.e.,\nwe are going to kill the CONFIG_MTD_UBI_DEBUG_PARANOID Kconfig\noption, which would mean that \u0027ubi-\u003edbg_peb_buf\u0027 is always allocated,\nwhich would be wasteful.\n\nThus, we are getting rid of \u0027ubi-\u003edbg_peb_buf\u0027, and this is a\npreparation for that.\n\nsigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "a75867432a7eb2cdcaa8613a3b72b1d0594dd930", "tree": "f83b3760bf171a3402ff48616726c6c71d01df2c", "parents": [ "a87f29cbbcbd5bd1e4990367cd18967e9bbeacff" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Mar 14 17:06:52 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Mar 16 13:50:16 2011 +0200" }, "message": "UBI: allocate write checking buffer on demand\n\nInstead of using pre-allocated \u0027ubi-\u003edbg_peb_buf\u0027 buffer in\n\u0027ubi_dbg_check_write()\u0027, dynamically allocate it when needed. The\nintend is to get rid of the pre-allocated \u0027ubi-\u003edbg_peb_buf\u0027 buffer\ncompletely. And the need for this arises because we want to change\nto dynamic debugging control instead of compile-time control, i.e.,\nwe are going to kill the CONFIG_MTD_UBI_DEBUG_PARANOID Kconfig\noption, which would mean that \u0027ubi-\u003edbg_peb_buf\u0027 is always allocated,\nwhich would be wasteful.\n\nThus, we are getting rid of \u0027ubi-\u003edbg_peb_buf\u0027, and this is a\npreparation for that.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "f43ec882b8b65de0ebde2e1ad52e8de0349d83ae", "tree": "6d71ba08bf9069c354a01a2b2e78ea867bab604a", "parents": [ "30b542ef453e6832ff682170b2db95d7bca2fe70" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Feb 14 13:36:54 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Mar 08 10:12:48 2011 +0200" }, "message": "UBI: provide LEB offset information\n\nProvide the LEB offset information in the UBI device information data\nstructure. This piece of information is required by UBIFS to find out\nwhat are the LEB offsets which are aligned to the max. write size.\n\nIf LEB offset not aligned to max. write size, then UBIFS has to take\nthis into account to write more optimally.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "30b542ef453e6832ff682170b2db95d7bca2fe70", "tree": "a74ccffb6a4eebb6ec8f9d3aa2d152700f995830", "parents": [ "10ac27970274e9094aee84a6452a25bf1b7e59e1" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Jan 30 18:37:33 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Mar 08 10:12:48 2011 +0200" }, "message": "UBI: incorporate maximum write size\n\nIncorporate MTD write buffer size into UBI device information\nbecause UBIFS needs this field. UBI does not use it ATM, just\nprovides to upper layers in \u0027struct ubi_device_info\u0027.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "a87f29cbbcbd5bd1e4990367cd18967e9bbeacff", "tree": "637789d9f982c7b92d4ee0aafcb1a3060e35a628", "parents": [ "fef2deb31f6523203a3fa1af485a5f1fef19cf6b" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Oct 31 18:55:30 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Feb 06 19:35:26 2011 +0200" }, "message": "UBI: always re-read in case of read failures\n\nWhen the read operation fails, UBI tries to re-read several times in\na hope that one of the subsequent reads may succeed. However, currently\nUBI re-reads only if MTD failed to read all data, but does not re-reads\nif all the data were read, but with an integrity error (-EBADMSB). This\npatch makes UBI to always re-try reading.\n\nThis should be useful for reading NAND pages with unstable bits -\nre-reading may help to get correct data.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "fef2deb31f6523203a3fa1af485a5f1fef19cf6b", "tree": "d29f44eeb247b72329e0e16e19399dd4b83e5e26", "parents": [ "6c1e875ca6f3a47b40dce715bd07fdfdb8388d55" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Oct 29 08:34:50 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Feb 06 19:19:41 2011 +0200" }, "message": "UBI: cleanup comments about corrupted PEBs\n\nJust make them a bit more readable and explanatory.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "6c1e875ca6f3a47b40dce715bd07fdfdb8388d55", "tree": "b30ab51df8df7c4ed9ef3adb68b234064816497f", "parents": [ "7950d023c562823345892aac2e7c6a49f8de9ad1" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Oct 31 17:54:14 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Feb 06 19:19:23 2011 +0200" }, "message": "UBI: add slab cache for ubi_scan_leb objects\n\nDuring scanning UBI allocates one struct ubi_scan_leb object for each PEB,\nso it can end up allocating thousands of them. Use slab cache to reduce\nmemory consumption for these 48-byte objects, because currently used\n\u0027kmalloc()\u0027 ends up allocating 64 bytes per object, instead of 48.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "7950d023c562823345892aac2e7c6a49f8de9ad1", "tree": "b1482e3a88c69742a2b39ecba7a71e471a105a57", "parents": [ "276832d878d8a892ac7b40fd0ee07fe757e080c7" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Nov 19 17:05:36 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Feb 06 19:18:58 2011 +0200" }, "message": "UBI: use raw mtd read function in debugging code\n\nThis change affects only the debugging code. Namely, use mtd-\u003eread()\nfunction instead of ubi_io_read() to avoid bit-flips injection\n(ubi_dbg_is_bitflip()) which we do not want on the debugging path.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "276832d878d8a892ac7b40fd0ee07fe757e080c7", "tree": "8da446b3091065c088f1aae8b35833eba9b91308", "parents": [ "2fff570e7c8f97e411cd852d64b77b92d9ab8da9" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sat Nov 13 15:08:29 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Feb 06 19:18:32 2011 +0200" }, "message": "UBI: try to reveal buggy MTD drivers\n\nWhen reading data from the flash, corrupt the buffer we are about to\nread to. The idea is to fix the following possible situation:\n\n1. The buffer contains data from previous operation, e.g., read from\n another PEB previously. The data looks like expected, e.g., if we\n just do not read anything and return - the caller would not\n notice this. E.g., if we are reading a VID header, the buffer may\n contain a valid VID header from another PEB.\n2. The driver is buggy and returns use success or -EBADMSG or\n -EUCLEAN, but it does not actually put any data to the buffer.\n\nThis may confuse UBI or upper layers - they may think the buffer\ncontains valid data while in fact it is just old data.\n\nThus, try to reveal such buggy MTD drivers with simple debugging\ncode which fills the read buffer with 0x12 constant.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "3a0592b1dd69cfc403ee7514c47b9d57d8bbd6d7", "tree": "b98f0e7cc2dc25cdffcd52dee0f070ab033b5f05", "parents": [ "1bae4ce27c9c90344f23c65ea6966c50ffeae2f5" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sat Jan 29 18:27:13 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sat Jan 29 18:27:13 2011 +0200" }, "message": "Revert \"UBI: use mtd-\u003ewritebufsize to set minimal I/O unit size\"\n\nThis reverts commit a121f643993474548fe98144514c50dd4f3dbe76.\n\nUnfortunately, this commit breaks UBIFS backward compatibility and\nmakes new UBIFS refuse older UBIFS-formatted media:\n\nUBIFS error: validate_sb: min. I/O unit mismatch: 8 in superblock, 64 real\n\nThus, we have to revert this patch and work on a better solution.\n\nReported-by: Holger Brunck \u003cholger.brunck@keymile.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "2fff570e7c8f97e411cd852d64b77b92d9ab8da9", "tree": "4f5ad38da8a911a887422c9ecef97644cf8e07d9", "parents": [ "e8cfe009436f9ab6f4234e1f7c406c231747925c" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Dec 03 15:32:21 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Jan 26 10:14:23 2011 +0200" }, "message": "UBI: add a commentary about allocating VID header buffer on stack\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "e8cfe009436f9ab6f4234e1f7c406c231747925c", "tree": "5b076f5ea37c12cfaf17770647f2ecd4cae1a202", "parents": [ "d4c6381303163e774a72db8c172cdc5c23f01588" ], "author": { "name": "John Ogness", "email": "john.ogness@linutronix.de", "time": "Fri Jan 21 15:39:02 2011 +0100" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Jan 26 10:14:17 2011 +0200" }, "message": "UBI: cleanup LEB start calculations\n\nWrong macro was used in calculating the data offset: UBI_EC_HDR_SIZE instead of\nUBI_VID_HDR_SIZE. The data offset should be VID header offset + VID header size\n(aligned to the minimum I/O unit).\n\nThis was not a bug only because currently UBI_EC_HDR_SIZE and UBI_VID_HDR_SIZE\nhave the same value of 64 bytes.\n\nCommit message was amended by Artem.\n\nSigned-off-by: John Ogness \u003cjohn.ogness@linutronix.de\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "d4c6381303163e774a72db8c172cdc5c23f01588", "tree": "2b7a6986c3168b1fc1f17113ac5cc3ef8c320bf1", "parents": [ "3c0eee3fe6a3a1c745379547c7e7c904aa64f6d5" ], "author": { "name": "Holger Brunck", "email": "holger.brunck@keymile.com", "time": "Tue Jan 25 13:04:11 2011 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Jan 26 10:13:52 2011 +0200" }, "message": "UBI: fix NOR erase preparation quirk\n\nIn \u0027nor_erase_prepare()\u0027 we want to make sure the UBI headers are\ncorrupted. But it is possible that one of the headers just contains\nall 0xFFs, which is also OK, because UBI will erase it in case of\na power cut.\n\nSigned-off-by: Holger Brunck \u003cholger.brunck@keymile.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "ab2020f2f11fc7fb81e6c71298b0830d85412011", "tree": "c9a6342063461dcf31278d65585bca73bdda4a84", "parents": [ "235646a486d10891bd86af28d8eac75d9f22bd2d", "154bf89f5e3e3dc59666926f27ca4a0866f39157" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jan 17 11:15:30 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jan 17 11:15:30 2011 -0800" }, "message": "Merge git://git.infradead.org/mtd-2.6\n\n* git://git.infradead.org/mtd-2.6: (59 commits)\n mtd: mtdpart: disallow reading OOB past the end of the partition\n mtd: pxa3xx_nand: NULL dereference in pxa3xx_nand_probe\n UBI: use mtd-\u003ewritebufsize to set minimal I/O unit size\n mtd: initialize writebufsize in the MTD object of a partition\n mtd: onenand: add mtd-\u003ewritebufsize initialization\n mtd: nand: add mtd-\u003ewritebufsize initialization\n mtd: cfi: add writebufsize initialization\n mtd: add writebufsize field to mtd_info struct\n mtd: OneNAND: OMAP2/3: prevent regulator sleeping while OneNAND is in use\n mtd: OneNAND: add enable / disable methods to onenand_chip\n mtd: m25p80: Fix JEDEC ID for AT26DF321\n mtd: txx9ndfmc: limit transfer bytes to 512 (ECC provides 6 bytes max)\n mtd: cfi_cmdset_0002: add support for Samsung K8D3x16UxC NOR chips\n mtd: cfi_cmdset_0002: add support for Samsung K8D6x16UxM NOR chips\n mtd: nand: ams-delta: drop omap_read/write, use ioremap\n mtd: m25p80: add debugging trace in sst_write\n mtd: nand: ams-delta: select for built-in by default\n mtd: OneNAND: lighten scary initial bad block messages\n mtd: OneNAND: OMAP2/3: add support for command line partitioning\n mtd: nand: rearrange ONFI revision checking, add ONFI 2.3\n ...\n\nFix up trivial conflict in drivers/mtd/Kconfig as per DavidW.\n" }, { "commit": "a121f643993474548fe98144514c50dd4f3dbe76", "tree": "184d54ebdbd62977ed4d672058293638fe2e0b2a", "parents": [ "7fa33ac0a7e24a1b8bd71be5c47a17423c62fbda" ], "author": { "name": "Anatolij Gustschin", "email": "agust@denx.de", "time": "Thu Dec 16 23:42:19 2010 +0100" }, "committer": { "name": "David Woodhouse", "email": "David.Woodhouse@intel.com", "time": "Thu Jan 06 15:36:00 2011 +0000" }, "message": "UBI: use mtd-\u003ewritebufsize to set minimal I/O unit size\n\nPreviously we used mtd-\u003ewritesize field to set UBI\u0027s minimal\nI/O unit size. This sometimes caused UBIFS recovery issues\nwhen mounting an uncleanly unmounted UBIFS partition on NOR\nflash since mtd-\u003ewritesize is 1 byte for NOR flash. The\nMTD CFI driver however often performs writing multiple\nbytes in one programming operation using the chip\u0027s write\nbuffer. We have to use the size of this write buffer as\na minimal I/O unit size for UBI on NOR flash to fix the\nobserved UBIFS recovery issues.\n\nSigned-off-by: Anatolij Gustschin \u003cagust@denx.de\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nSigned-off-by: David Woodhouse \u003cDavid.Woodhouse@intel.com\u003e\n" }, { "commit": "309b5e4e4154721f8079bc250d2233fd4b3aa039", "tree": "b004e9ff14c82a247e34e6050d1477ac750f7c54", "parents": [ "08b3af3092bb2c284796e4e823c5309c2d0a9bca" ], "author": { "name": "Joe Perches", "email": "joe@perches.com", "time": "Thu Nov 04 20:07:40 2010 -0700" }, "committer": { "name": "David Woodhouse", "email": "David.Woodhouse@intel.com", "time": "Fri Dec 03 16:27:29 2010 +0000" }, "message": "mtd: use vzalloc\n\nSigned-off-by: Joe Perches \u003cjoe@perches.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nSigned-off-by: David Woodhouse \u003cDavid.Woodhouse@intel.com\u003e\n" }, { "commit": "7ac760c2f78ddd8e1bd633767b01becfbbf96720", "tree": "424592684b1e91ba7189195b0c89abd2f68b515b", "parents": [ "e8a7e48bb248a1196484d3f8afa53bded2b24e71" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Dec 02 06:34:01 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Dec 03 15:49:21 2010 +0200" }, "message": "UBI: fix corrupted PEB detection for NOR flash\n\nMy new shiny code for corrupted PEB detection has NOR specific bug.\nWe tread PEB as corrupted and preserve it, if\n\n1. EC header is OK.\n2. VID header is corrupted.\n3. data area is not \"all 0xFFs\"\n\nIn case of NOR we have \u0027nor_erase_prepare()\u0027 quirk, which invalidates\nthe headers before erasing the PEB. And we invalidate first the VID\nheader, and then the EC header. So if a power cut happens after we have\ninvalidated the VID header, but before we have invalidated the EC\nheader, we end up with a PEB which satisfies the above 3 conditions,\nand the scanning code will treat it as corrupted, and will print\nscary warnings, wrongly.\n\nThis patch fixes the issue by firt invalidating the EC header, then\ninvalidating the VID header. In case of power cut inbetween, we still\njust lose the EC header, and UBI can deal with this situation gracefully.\n\nThanks to Anatolij Gustschin \u003cagust@denx.de\u003e for tracking this down.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nReported-by: Anatolij Gustschin \u003cagust@denx.de\u003e\nTested-by: Anatolij Gustschin \u003cagust@denx.de\u003e\n" }, { "commit": "1b1d76e2df2a0aa965d6a227db7061d3fff029b1", "tree": "c0ebf23becdbe770d9c8e8ab3b8613e5143d4bb3", "parents": [ "e53beacd23d9cb47590da6a7a7f6d417b941a994" ], "author": { "name": "Dan Carpenter", "email": "error27@gmail.com", "time": "Thu Nov 18 06:58:04 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Nov 19 15:19:40 2010 +0200" }, "message": "UBI: release locks in check_corruption\n\nCommit 45aafd32996e27 \"UBI: tighten the corrupted PEB criteria\"\nintroduced some return paths that didn\u0027t release the ubi-\u003ebuf_mutex\n\nSigned-off-by: Dan Carpenter \u003cerror27@gmail.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "4f3a29dadaf999a273f1e7fe2476595d0283eef3", "tree": "3776499f6edc95a36533c7efe32a7b6038c88b61", "parents": [ "06d362931a530e0d48c1a9554a752da4ed240f0b", "45aafd32996e27bfc4862654ff31231bdddbe200" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 16:34:23 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 16:34:23 2010 -0700" }, "message": "Merge branch \u0027linux-next\u0027 of git://git.infradead.org/ubi-2.6\n\n* \u0027linux-next\u0027 of git://git.infradead.org/ubi-2.6:\n UBI: tighten the corrupted PEB criteria\n UBI: fix check_data_ff return code\n UBI: remember copy_flag while scanning\n UBI: preserve corrupted PEBs\n UBI: add truly corrupted PEBs to corrupted list\n UBI: introduce debugging helper function\n UBI: make check_pattern function non-static\n UBI: do not put eraseblocks to the corrupted list unnecessarily\n UBI: separate out corrupted list\n UBI: change cascade of ifs to switch statements\n UBI: rename a local variable\n UBI: handle bit-flips when no header found\n UBI: remove duplicate IO error codes\n UBI: rename IO error code\n UBI: fix small 80 characters limit style issue\n UBI: cleanup and simplify Kconfig\n" }, { "commit": "45aafd32996e27bfc4862654ff31231bdddbe200", "tree": "fd2dee47b52135e9532d479a0378a8ec52af6eba", "parents": [ "df3fca4cdddfa6e1f51b65214d4342660649bd1f" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Oct 20 11:54:58 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Oct 21 11:20:37 2010 +0300" }, "message": "UBI: tighten the corrupted PEB criteria\n\nIf we get a bit-flip of ECC error while reading the data area, do not add it to\ncorrupted list, because it is possible that this is just unstable PEB with\ncorruptions caused by unclean reboots.\n\nThis patch also improves commentaries.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "df3fca4cdddfa6e1f51b65214d4342660649bd1f", "tree": "13389da659d8d2235f8a56f7e263f11bced19fc6", "parents": [ "fb22b59b2c38054cc847f6acc5c46daa26dc6dd3" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Oct 20 11:51:21 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Oct 21 11:20:37 2010 +0300" }, "message": "UBI: fix check_data_ff return code\n\nWhen the data does not contain all 0xFF bytes, \u0027check_data_ff()\u0027 should return\n1, not -EINVAL; Also, the caller (\u0027process_eb()\u0027) should not add the PEB to the\n\"corrupted\" list if there was a read error.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "fb22b59b2c38054cc847f6acc5c46daa26dc6dd3", "tree": "76025e34f0934b8e7186972e43849dc4a8f19606", "parents": [ "5fc01ab6934c43b42c41bc753fe1123c16d7f38f" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 22:00:11 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu Oct 21 11:20:36 2010 +0300" }, "message": "UBI: remember copy_flag while scanning\n\nWhile scanning the flash we read all VID headers and store some important\ninformation in \u0027struct ubi_scan_leb\u0027. Store also the \u0027copy_flag\u0027 value there\nas it is needed when comparing LEBs. We do not increase memory consumption\nbecause this is just one bit and we have plenty of spare bits in\n\u0027struct ubi_scan_leb\u0027 (sizeof(struct ubi_scan_leb) is 48 both with and\nwithout this patch).\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "5fc01ab6934c43b42c41bc753fe1123c16d7f38f", "tree": "174a1e1a0d283ec8ca4c756d7f354a475ac46a36", "parents": [ "feeba4b872e5166ca64c44fbb5bbec234dfce199" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 23:08:15 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:57 2010 +0300" }, "message": "UBI: preserve corrupted PEBs\n\nCurrently UBI erases all corrupted eraseblocks, irrespectively of the nature\nof corruption: corruption due to power cuts and non-power cut corruption.\nThe former case is OK, but the latter is not, because UBI may destroy\npotentially important data.\n\nWith this patch, during scanning, when UBI hits a PEB with corrupted VID\nheader, it checks whether this PEB contains only 0xFF data. If yes, it is\nsafe to erase this PEB and it is put to the \u0027erase\u0027 list. If not, this may\nbe important data and it is better to avoid erasing this PEB. Instead,\nUBI puts it to the corr list and moves out of the pool of available PEB.\nIOW, UBI preserves this PEB.\n\nSuch corrupted PEB lessen the amount of available PEBs. So the more of them\nwe accumulate, the less PEBs are available. The maximum amount of non-power\ncut corrupted PEBs is 8.\n\nThis patch is a response to UBIFS problem where reporter\n(Matthew L. Creech \u003cmlcreech@gmail.com\u003e) observes that UBIFS index points\nto an unmapped LEB. The theory is that corresponding PEB somehow got\ncorrupted and UBI wiped it. This patch (actually a series of patches)\ntries to make sure such PEBs are preserved - this would make it is easier\nto analyze the corruption.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "feeba4b872e5166ca64c44fbb5bbec234dfce199", "tree": "5120bbb0e351ae3f108e33f70281e53b9eebe5c5", "parents": [ "315324947cbc7264af86b7ecdc2d5fb5f8556222" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 22:50:53 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:57 2010 +0300" }, "message": "UBI: add truly corrupted PEBs to corrupted list\n\nStart using the \u0027corr\u0027 list and add there PEBs which look truly corrupted,\nwhich means they have corrupted VID header and the data which follows the\ncorrupted header does not contain all 0xFF bytes.\n\nAt the moment, this does not change UBI functionality much because these\nPEBs will be erase when scanning finishes. But the plan is to teach UBI\npreserving them.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "315324947cbc7264af86b7ecdc2d5fb5f8556222", "tree": "2e4ded851b2138e5e46ca43458564d3408c68c68", "parents": [ "bb00e180a93a6c8e89c3b2d1f9473781e1e2d2a4" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 22:27:46 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:57 2010 +0300" }, "message": "UBI: introduce debugging helper function\n\nIntroduce a helper function to print hexdump: \u0027ubi_dbg_print_hex_dump()\u0027.\nIt is compiled out if debugging is enabled. Will be used in the next patch.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "bb00e180a93a6c8e89c3b2d1f9473781e1e2d2a4", "tree": "87bd9ddc65052014e36e4fb0565e73d7ff4dd992", "parents": [ "0525dac9fd31e5a12fb934238abd09e2752a5967" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sat Jul 31 09:37:34 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:57 2010 +0300" }, "message": "UBI: make check_pattern function non-static\n\nThis patch turns static function \u0027check_pattern()\u0027 into a non-static\n\u0027ubi_check_pattern()\u0027. This is just a preparation for the chages which\nare coming in the next patches.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "0525dac9fd31e5a12fb934238abd09e2752a5967", "tree": "9f5f04c0d073d9c4e628afdaa5ad00a1504061f0", "parents": [ "3fb34124da9d5e37576d9f87d7a5005ba1d82dd7" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 17:11:37 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:57 2010 +0300" }, "message": "UBI: do not put eraseblocks to the corrupted list unnecessarily\n\nCurrently UBI maintains 2 lists of PEBs during scanning:\n1. \u0027erase\u0027 list - PEBs which have no corruptions but should be erased\n2. \u0027corr\u0027 list - PEBs which have some corruptions and should be erased\n\nBut we do not really need 2 lists for PEBs which should be erased after\nscanning is done - this is redundant. So this patch makes sure all PEBs\nwhich are corrupted are moved to the head of the \u0027erase\u0027 list. We add\nthem to the head to make sure they are erased first and we get rid of\ncorruption ASAP.\n\nHowever, we do not remove the \u0027corr\u0027 list and realted functions, because\nthe plan is to use this list for other purposes. Namely, we plan to\nput eraseblocks with corruption which does not look like it was caused\nby unclean power cut. Then we\u0027ll preserve thes PEBs in order to avoid\nkilling potentially valuable user data.\n\nThis patch also amends PEBs accounting, because it was closely tight to\nthe \u0027erase\u0027/\u0027corr\u0027 lists separation.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "3fb34124da9d5e37576d9f87d7a5005ba1d82dd7", "tree": "98f4ff2696b1cd480e3fb43b93f637e2b5bc8bd0", "parents": [ "b33215084c1c06258a2d9deb035d343aafa4066e" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 15:36:12 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:56 2010 +0300" }, "message": "UBI: separate out corrupted list\n\nThis patch introduces \u0027add_corrupted()\u0027 function and separates out \u0027corr\u0027 list\nmanipulation from the common \u0027add_to_list()\u0027 function. This is just a\npreparation for further changes - this patch does not change functionality.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "b33215084c1c06258a2d9deb035d343aafa4066e", "tree": "fd93b1b76a871e85b680c83de2ec4a81c3444025", "parents": [ "e0e718c281ce1dd4006681e1255535cc6f2857d4" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 14:40:55 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:56 2010 +0300" }, "message": "UBI: change cascade of ifs to switch statements\n\nThis patch improves readability and simplifies scanning code by changing a\nlong cascade of \u0027if\u0027 statements to a switch statement. This should presumably\nbe a little faster as well.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "e0e718c281ce1dd4006681e1255535cc6f2857d4", "tree": "14958b8e11ffd676360f8709501924ff0963b7ae", "parents": [ "92e1a7d9e7e07fb1cf0cbbcdf202938d0819b54d" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 14:53:23 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:56 2010 +0300" }, "message": "UBI: rename a local variable\n\nRename local variable \u0027ec_corr\u0027 into \u0027ec_err\u0027 to make the code a little bit\nmore readable. \u0027ec_err\u0027 is more appropriate because it sounds more like \u0027error\nwhen EC was read\u0027 and it looks more logical because we use it together with\n\u0027err\u0027. Just a minor nicification which should improve the rather complex\nscanning code.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "92e1a7d9e7e07fb1cf0cbbcdf202938d0819b54d", "tree": "48930121fa2f1cf9ebd36f85118fe433d1ecd01f", "parents": [ "74d82d2660058e32644f0c673656b2a1d01d3688" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 14:22:17 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:56 2010 +0300" }, "message": "UBI: handle bit-flips when no header found\n\nCurrently UBI has one small flaw - when we read EC or VID header, but find only\n0xFF bytes, we return UBI_IO_FF and do not report whether we had bit-flips or\nnot. In case of the VID header, the scanning code adds this PEB to the free list,\neven though there were bit-flips.\n\nImagine the following situation: we start writing VID header to a PEB and have a\npower cut, so the PEB becomes unstable. When we scan and read the PEB, we get\na bit-flip. Currently, UBI would just ignore this and treat the PEB as free. This\npatch changes UBI behavior and now UBI will schedule this PEB for erasure.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "74d82d2660058e32644f0c673656b2a1d01d3688", "tree": "8b90af4d6014ea50d74c504dc832cf3a9c3fff10", "parents": [ "756e1df1d2b8b572a92dd1b82d2a432d5b280b1c" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 02:11:20 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:56 2010 +0300" }, "message": "UBI: remove duplicate IO error codes\n\nThe \u0027UBI_IO_PEB_EMPTY\u0027 and \u0027UBI_IO_PEB_FREE\u0027 are essentially the same\nand mean that there are only 0xFF bytes instead of headers. Simplify\nUBI a little by turning them into a single \u0027UBI_IO_FF\u0027 error code.\n\nAlso, stop maintaining commentaries in \u0027ubi_io_read_vid_hdr()\u0027 which are\nalmost identical to commentaries in \u0027ubi_io_read_ec_hdr()\u0027.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "756e1df1d2b8b572a92dd1b82d2a432d5b280b1c", "tree": "e1109e8a31f72bb3c50b2001f28983c1bdfba1a9", "parents": [ "c174a08c72ae77a05be59d0d810dc13239b81e8e" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 01:30:16 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:56 2010 +0300" }, "message": "UBI: rename IO error code\n\nRename UBI_IO_BAD_HDR_READ into UBI_IO_BAD_HDR_EBADMSG which is presumably more\nself-documenting and readable. Indeed, the \u0027_READ\u0027 suffix does not tell much and\neven confuses, while \u0027_EBADMSG\u0027 tells about uncorrectable ECC error, because we\nuse -EBADMSG all over the place to represent ECC errors.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "c174a08c72ae77a05be59d0d810dc13239b81e8e", "tree": "be1b596614c912e29890c24bfcecf48beff6965e", "parents": [ "1dd786328ff42b239a00fdbcdaed6fd9709042bf" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Sep 03 15:11:17 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:55 2010 +0300" }, "message": "UBI: fix small 80 characters limit style issue\n\nOne line was longer than 80 lines, make it shorter.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "1dd786328ff42b239a00fdbcdaed6fd9709042bf", "tree": "2bdf0f5d47f47c87c9c976900ab9d47716b4bb66", "parents": [ "cd07202cc8262e1669edff0d97715f3dd9260917" ], "author": { "name": "H Hartley Sweeten", "email": "hartleys@visionengravers.com", "time": "Thu Aug 19 17:28:50 2010 -0700" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 19 17:19:55 2010 +0300" }, "message": "UBI: cleanup and simplify Kconfig\n\nCleanup the Kconfig for UBI by using menuconfig to enable/disable the entire\ndriver. Remove the dependency checks for MTD_UBI and MTD_UBI_DEBUG by\nwrapping the options in if/endif blocks and remove any redundant checks.\nRemove all default n since that is the Kconfig default. Change menu \"Additional\nUBI debugging messages\" into a comment to remove one menu level.\n\nSigned-off-by: H Hartley Sweeten \u003chsweeten@visionengravers.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "6038f373a3dc1f1c26496e60b6c40b164716f07e", "tree": "a0d3bbd026eea41b9fc36b8c722cbaf56cd9f825", "parents": [ "1ec5584e3edf9c4bf2c88c846534d19cf986ba11" ], "author": { "name": "Arnd Bergmann", "email": "arnd@arndb.de", "time": "Sun Aug 15 18:52:59 2010 +0200" }, "committer": { "name": "Arnd Bergmann", "email": "arnd@arndb.de", "time": "Fri Oct 15 15:53:27 2010 +0200" }, "message": "llseek: automatically add .llseek fop\n\nAll file_operations should get a .llseek operation so we can make\nnonseekable_open the default for future file operations without a\n.llseek pointer.\n\nThe three cases that we can automatically detect are no_llseek, seq_lseek\nand default_llseek. For cases where we can we can automatically prove that\nthe file offset is always ignored, we use noop_llseek, which maintains\nthe current behavior of not returning an error from a seek.\n\nNew drivers should normally not use noop_llseek but instead use no_llseek\nand call nonseekable_open at open time. Existing drivers can be converted\nto do the same when the maintainer knows for certain that no user code\nrelies on calling seek on the device file.\n\nThe generated code is often incorrectly indented and right now contains\ncomments that clarify for each added line why a specific variant was\nchosen. In the version that gets submitted upstream, the comments will\nbe gone and I will manually fix the indentation, because there does not\nseem to be a way to do that using coccinelle.\n\nSome amount of new code is currently sitting in linux-next that should get\nthe same modifications, which I will do at the end of the merge window.\n\nMany thanks to Julia Lawall for helping me learn to write a semantic\npatch that does all this.\n\n\u003d\u003d\u003d\u003d\u003d begin semantic patch \u003d\u003d\u003d\u003d\u003d\n// This adds an llseek\u003d method to all file operations,\n// as a preparation for making no_llseek the default.\n//\n// The rules are\n// - use no_llseek explicitly if we do nonseekable_open\n// - use seq_lseek for sequential files\n// - use default_llseek if we know we access f_pos\n// - use noop_llseek if we know we don\u0027t access f_pos,\n// but we still want to allow users to call lseek\n//\n@ open1 exists @\nidentifier nested_open;\n@@\nnested_open(...)\n{\n\u003c+...\nnonseekable_open(...)\n...+\u003e\n}\n\n@ open exists@\nidentifier open_f;\nidentifier i, f;\nidentifier open1.nested_open;\n@@\nint open_f(struct inode *i, struct file *f)\n{\n\u003c+...\n(\nnonseekable_open(...)\n|\nnested_open(...)\n)\n...+\u003e\n}\n\n@ read disable optional_qualifier exists @\nidentifier read_f;\nidentifier f, p, s, off;\ntype ssize_t, size_t, loff_t;\nexpression E;\nidentifier func;\n@@\nssize_t read_f(struct file *f, char *p, size_t s, loff_t *off)\n{\n\u003c+...\n(\n *off \u003d E\n|\n *off +\u003d E\n|\n func(..., off, ...)\n|\n E \u003d *off\n)\n...+\u003e\n}\n\n@ read_no_fpos disable optional_qualifier exists @\nidentifier read_f;\nidentifier f, p, s, off;\ntype ssize_t, size_t, loff_t;\n@@\nssize_t read_f(struct file *f, char *p, size_t s, loff_t *off)\n{\n... when !\u003d off\n}\n\n@ write @\nidentifier write_f;\nidentifier f, p, s, off;\ntype ssize_t, size_t, loff_t;\nexpression E;\nidentifier func;\n@@\nssize_t write_f(struct file *f, const char *p, size_t s, loff_t *off)\n{\n\u003c+...\n(\n *off \u003d E\n|\n *off +\u003d E\n|\n func(..., off, ...)\n|\n E \u003d *off\n)\n...+\u003e\n}\n\n@ write_no_fpos @\nidentifier write_f;\nidentifier f, p, s, off;\ntype ssize_t, size_t, loff_t;\n@@\nssize_t write_f(struct file *f, const char *p, size_t s, loff_t *off)\n{\n... when !\u003d off\n}\n\n@ fops0 @\nidentifier fops;\n@@\nstruct file_operations fops \u003d {\n ...\n};\n\n@ has_llseek depends on fops0 @\nidentifier fops0.fops;\nidentifier llseek_f;\n@@\nstruct file_operations fops \u003d {\n...\n .llseek \u003d llseek_f,\n...\n};\n\n@ has_read depends on fops0 @\nidentifier fops0.fops;\nidentifier read_f;\n@@\nstruct file_operations fops \u003d {\n...\n .read \u003d read_f,\n...\n};\n\n@ has_write depends on fops0 @\nidentifier fops0.fops;\nidentifier write_f;\n@@\nstruct file_operations fops \u003d {\n...\n .write \u003d write_f,\n...\n};\n\n@ has_open depends on fops0 @\nidentifier fops0.fops;\nidentifier open_f;\n@@\nstruct file_operations fops \u003d {\n...\n .open \u003d open_f,\n...\n};\n\n// use no_llseek if we call nonseekable_open\n////////////////////////////////////////////\n@ nonseekable1 depends on !has_llseek \u0026\u0026 has_open @\nidentifier fops0.fops;\nidentifier nso ~\u003d \"nonseekable_open\";\n@@\nstruct file_operations fops \u003d {\n... .open \u003d nso, ...\n+.llseek \u003d no_llseek, /* nonseekable */\n};\n\n@ nonseekable2 depends on !has_llseek @\nidentifier fops0.fops;\nidentifier open.open_f;\n@@\nstruct file_operations fops \u003d {\n... .open \u003d open_f, ...\n+.llseek \u003d no_llseek, /* open uses nonseekable */\n};\n\n// use seq_lseek for sequential files\n/////////////////////////////////////\n@ seq depends on !has_llseek @\nidentifier fops0.fops;\nidentifier sr ~\u003d \"seq_read\";\n@@\nstruct file_operations fops \u003d {\n... .read \u003d sr, ...\n+.llseek \u003d seq_lseek, /* we have seq_read */\n};\n\n// use default_llseek if there is a readdir\n///////////////////////////////////////////\n@ fops1 depends on !has_llseek \u0026\u0026 !nonseekable1 \u0026\u0026 !nonseekable2 \u0026\u0026 !seq @\nidentifier fops0.fops;\nidentifier readdir_e;\n@@\n// any other fop is used that changes pos\nstruct file_operations fops \u003d {\n... .readdir \u003d readdir_e, ...\n+.llseek \u003d default_llseek, /* readdir is present */\n};\n\n// use default_llseek if at least one of read/write touches f_pos\n/////////////////////////////////////////////////////////////////\n@ fops2 depends on !fops1 \u0026\u0026 !has_llseek \u0026\u0026 !nonseekable1 \u0026\u0026 !nonseekable2 \u0026\u0026 !seq @\nidentifier fops0.fops;\nidentifier read.read_f;\n@@\n// read fops use offset\nstruct file_operations fops \u003d {\n... .read \u003d read_f, ...\n+.llseek \u003d default_llseek, /* read accesses f_pos */\n};\n\n@ fops3 depends on !fops1 \u0026\u0026 !fops2 \u0026\u0026 !has_llseek \u0026\u0026 !nonseekable1 \u0026\u0026 !nonseekable2 \u0026\u0026 !seq @\nidentifier fops0.fops;\nidentifier write.write_f;\n@@\n// write fops use offset\nstruct file_operations fops \u003d {\n... .write \u003d write_f, ...\n+\t.llseek \u003d default_llseek, /* write accesses f_pos */\n};\n\n// Use noop_llseek if neither read nor write accesses f_pos\n///////////////////////////////////////////////////////////\n\n@ fops4 depends on !fops1 \u0026\u0026 !fops2 \u0026\u0026 !fops3 \u0026\u0026 !has_llseek \u0026\u0026 !nonseekable1 \u0026\u0026 !nonseekable2 \u0026\u0026 !seq @\nidentifier fops0.fops;\nidentifier read_no_fpos.read_f;\nidentifier write_no_fpos.write_f;\n@@\n// write fops use offset\nstruct file_operations fops \u003d {\n...\n .write \u003d write_f,\n .read \u003d read_f,\n...\n+.llseek \u003d noop_llseek, /* read and write both use no f_pos */\n};\n\n@ depends on has_write \u0026\u0026 !has_read \u0026\u0026 !fops1 \u0026\u0026 !fops2 \u0026\u0026 !has_llseek \u0026\u0026 !nonseekable1 \u0026\u0026 !nonseekable2 \u0026\u0026 !seq @\nidentifier fops0.fops;\nidentifier write_no_fpos.write_f;\n@@\nstruct file_operations fops \u003d {\n... .write \u003d write_f, ...\n+.llseek \u003d noop_llseek, /* write uses no f_pos */\n};\n\n@ depends on has_read \u0026\u0026 !has_write \u0026\u0026 !fops1 \u0026\u0026 !fops2 \u0026\u0026 !has_llseek \u0026\u0026 !nonseekable1 \u0026\u0026 !nonseekable2 \u0026\u0026 !seq @\nidentifier fops0.fops;\nidentifier read_no_fpos.read_f;\n@@\nstruct file_operations fops \u003d {\n... .read \u003d read_f, ...\n+.llseek \u003d noop_llseek, /* read uses no f_pos */\n};\n\n@ depends on !has_read \u0026\u0026 !has_write \u0026\u0026 !fops1 \u0026\u0026 !fops2 \u0026\u0026 !has_llseek \u0026\u0026 !nonseekable1 \u0026\u0026 !nonseekable2 \u0026\u0026 !seq @\nidentifier fops0.fops;\n@@\nstruct file_operations fops \u003d {\n...\n+.llseek \u003d noop_llseek, /* no read or write fn */\n};\n\u003d\u003d\u003d\u003d\u003d End semantic patch \u003d\u003d\u003d\u003d\u003d\n\nSigned-off-by: Arnd Bergmann \u003carnd@arndb.de\u003e\nCc: Julia Lawall \u003cjulia@diku.dk\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\n" }, { "commit": "d3f6e6c666c0f68991d785177c4c62fcd1d651f2", "tree": "136f546ea12366f262b2d112b0795bf85c7e707c", "parents": [ "1deacd7a1d4bb22f812908c34e3254de76e4faa8" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sun Aug 29 23:34:44 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Aug 30 13:32:31 2010 +0300" }, "message": "UBI: do not oops when erroneous PEB is scheduled for scrubbing\n\nWhen an erroneous PEB is scheduling for scrubbing, we end up with the\nfollowing oops:\n\n[\u003cc0162404\u003e] (prot_queue_del+0x0/0x50) from [\u003cc01635b4\u003e] (ubi_wl_scrub_peb+0xec/0x13c)\n[\u003cc01634c8\u003e] (ubi_wl_scrub_peb+0x0/0x13c) from [\u003cc01603bc\u003e] (ubi_eba_read_leb+0x200/0x428)\n[\u003cc01601bc\u003e] (ubi_eba_read_leb+0x0/0x428) from [\u003cc015e3c0\u003e] (ubi_leb_read+0xe8/0x138)\n[\u003cc015e2d8\u003e] (ubi_leb_read+0x0/0x138) from [\u003cc00d6918\u003e] (ubifs_start_scan+0x7c/0xf4)\n[\u003cc00d689c\u003e] (ubifs_start_scan+0x0/0xf4) from [\u003cc00e3650\u003e] (ubifs_recover_leb+0x3c/0x730)\n[\u003cc00e3614\u003e] (ubifs_recover_leb+0x0/0x730) from [\u003cc00e444c\u003e] (ubifs_recover_log_leb+0xc8/0x2dc)\n[\u003cc00e4384\u003e] (ubifs_recover_log_leb+0x0/0x2dc) from [\u003cc00d7c20\u003e] (ubifs_replay_journal+0xb90/0x13a4)\n[\u003cc00d7090\u003e] (ubifs_replay_journal+0x0/0x13a4) from [\u003cc00cdd68\u003e] (ubifs_fill_super+0xb84/0x1054)\n[\u003cc00cd1e4\u003e] (ubifs_fill_super+0x0/0x1054) from [\u003cc00ced04\u003e] (ubifs_get_sb+0xc4/0x2ac)\n[\u003cc00cec40\u003e] (ubifs_get_sb+0x0/0x2ac) from [\u003cc007f04c\u003e] (vfs_kern_mount+0x58/0x94)\n[\u003cc007eff4\u003e] (vfs_kern_mount+0x0/0x94) from [\u003cc007f0e8\u003e] (do_kern_mount+0x40/0xe8)\n[\u003cc007f0a8\u003e] (do_kern_mount+0x0/0xe8) from [\u003cc0095628\u003e] (do_new_mount+0x68/0x8c)\n[\u003cc00955c0\u003e] (do_new_mount+0x0/0x8c) from [\u003cc00957a8\u003e] (do_mount+0x15c/0x1b8)\n[\u003cc009564c\u003e] (do_mount+0x0/0x1b8) from [\u003cc0095890\u003e] (sys_mount+0x8c/0xd4)\n[\u003cc0095804\u003e] (sys_mount+0x0/0xd4) from [\u003cc0023c00\u003e] (ret_fast_syscall+0x0/0x2c)\nKernel panic - not syncing: Fatal exception\n\nThe problem is that \u0027ubi_wl_scrub_peb()\u0027 does not expect that PEBs may\nbe in the erroneous tree, which is a bug. This patch fixes the bug\nand adds corresponding check to \u0027ubi_wl_scrub_peb()\u0027. Now it will simply\nignore erroneous PEBs, instead of causing an oops.\n\nReported-by: Matthieu CASTET \u003cmatthieu.castet@parrot.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "1deacd7a1d4bb22f812908c34e3254de76e4faa8", "tree": "a15a8b2b6083e9eb56077a1f874c5114e00a58bd", "parents": [ "80c1c16fb89e7055acb560e6d90c881b5c7496c1" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Thu Aug 12 12:10:59 2010 -0700" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Aug 30 13:31:34 2010 +0300" }, "message": "UBI: fix kconfig unmet dependency\n\nwarning: (OPTPROBES \u0026\u0026 KPROBES \u0026\u0026 HAVE_OPTPROBES \u0026\u0026 !PREEMPT \u0026\u0026 DEBUG_KERNEL ||\nMTD_UBI_DEBUG \u0026\u0026 MTD \u0026\u0026 SYSFS \u0026\u0026 MTD_UBI || UBIFS_FS_DEBUG \u0026\u0026 MISC_FILESYSTEMS \u0026\u0026\nUBIFS_FS || LOCKDEP \u0026\u0026 DEBUG_KERNEL \u0026\u0026 TRACE_IRQFLAGS_SUPPORT \u0026\u0026\nSTACKTRACE_SUPPORT \u0026\u0026 LOCKDEP_SUPPORT || LATENCYTOP \u0026\u0026 HAVE_LATENCYTOP_SUPPORT \u0026\u0026\nDEBUG_KERNEL \u0026\u0026 STACKTRACE_SUPPORT \u0026\u0026 PROC_FS) selects KALLSYMS_ALL\nwhich has unmet direct dependencies (DEBUG_KERNEL \u0026\u0026 KALLSYMS)\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "80c1c16fb89e7055acb560e6d90c881b5c7496c1", "tree": "f84727cff89a3ac0cf7c0e502a189d3d20c289b1", "parents": [ "01ebc12f5f2e88a1c6a5436b71a506ac2bf66d6b" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Sat Aug 28 12:11:40 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Aug 30 10:19:11 2010 +0300" }, "message": "UBI: fix forward compatibility\n\nCommit 0798cea8c2e1afee59686c51d27d0e96b05e42d1 \"UBI: improve corrupted flash handling\"\nbroke delet-compatible volumes handling - it introduced a limit of 8 eraseblocks which\nmay be corrupted. And delete-compatible eraseblocks are added to the \"corrupted\" list,\nso if we\u0027d have a large delete-compatible volume, UBI would refuse it.\n\nThe fix is to add delete-compatible volumes to the erase list instead. Indeed, they are\ncorrupted, we just have to erase them.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "01ebc12f5f2e88a1c6a5436b71a506ac2bf66d6b", "tree": "7c3e748518efe1f42c82eb922d1d6625964282a2", "parents": [ "2bfc96a127bc1cc94d26bfaa40159966064f9c8c" ], "author": { "name": "Julia Lawall", "email": "julia@diku.dk", "time": "Sat Aug 07 11:09:29 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Aug 30 10:19:11 2010 +0300" }, "message": "UBI: eliminate update of list_for_each_entry loop cursor\n\nlist_for_each_entry uses its first argument to move from one element to the\nnext, so modifying it can break the iteration. The variable re1 is already\nused within the loop as a temporary variable, and is not live here.\n\nThe semantic match that finds this problem is as follows:\n(http://coccinelle.lip6.fr/)\n\n// \u003csmpl\u003e\n@r@\niterator name list_for_each_entry;\nexpression x,E;\nposition p1,p2;\n@@\n\nlist_for_each_entry@p1(x,...) { \u003c... x \u003d@p2 E ...\u003e }\n\n@@\nexpression x,E;\nposition r.p1,r.p2;\nstatement S;\n@@\n\n*x \u003d@p2 E\n...\nlist_for_each_entry@p1(x,...) S\n// \u003c/smpl\u003e\n\nSigned-off-by: Julia Lawall \u003cjulia@diku.dk\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "64d4b4c90a876401e503c3a3260e9d0ed066f271", "tree": "4da6a688f44cdec560792a29e24031a00a0696ef", "parents": [ "7cdb996ee4d04b5bb2cd544b14e8eb91cfd7fe64" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jul 30 14:59:50 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Aug 02 07:21:19 2010 +0300" }, "message": "UBI: do not warn unnecessarily\n\nCurrently, when UBI attaches an MTD device and cannot reserve all 1% (by\ndefault) of PEBs for bad eraseblocks handling, it prints a warning. However,\nMatthew L. Creech \u003cmlcreech@gmail.com\u003e is not very happy to see this warning,\nbecause he did reserve enough of PEB at the beginning, but with time some\nPEBs became bad. The warning is not necessary in this case.\n\nThis patch makes UBI print the warning\n o if this is a new image\n o of this is used image and the amount of reserved PEBs is only 10% (or less)\n of the size of the reserved PEB pool.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "7cdb996ee4d04b5bb2cd544b14e8eb91cfd7fe64", "tree": "a533532fd99fb4445727344d8656586deb5a8094", "parents": [ "158132c9abbccce802def10e5ffaf044b266a6e1" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jul 30 16:31:38 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Aug 02 07:21:19 2010 +0300" }, "message": "UBI: do not print message about corruptes PEBs if we have none of them\n\nCurrently UBI prints\n\nUBI: corrupted PEBs will be formatted\n\neven if there are not corrupted PEBs. Fix this.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "158132c9abbccce802def10e5ffaf044b266a6e1", "tree": "d7d1532ace65bc89f9f4cf6e11c74ea745d079c4", "parents": [ "f5d5b1f8c12a7637ee1145f2f00358eb375edb54" ], "author": { "name": "Brijesh Singh", "email": "brijesh.s.singh@gmail.com", "time": "Wed Jun 16 09:28:26 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Jun 16 09:39:49 2010 +0300" }, "message": "UBI: improve delete-compatible volumes handling\n\nWhen a delete-compatible volume is found, it is first added to the\n\u0027corr\u0027 list, which contains \"corrupted\" PEBs which should be erased,\nand then it is added to the used volumes tree. However, the second\nstep should not be done. This does not cause problems in practice,\nbecause we never access delete-compattible volumes, but it is still\nnot the right thing to do.\n\n[Artem: amended the commit message and few prints]\n\nSigned-off-by: Brijesh Singh \u003cbrijesh.s.singh@gmail.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "f5d5b1f8c12a7637ee1145f2f00358eb375edb54", "tree": "5d4b5ec5ef6eba5fb0f5cc1e5127115da33c2759", "parents": [ "095751a6e0838a712393a74eb0b7b6559dbdbe81" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Jun 14 08:15:39 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Jun 14 08:47:49 2010 +0300" }, "message": "UBI: fix error message and compilation warnings\n\nFix the followong compilation warnings introduced by commit\n095751a6e0838a712393a74eb0b7b6559dbdbe81:\n\ndrivers/mtd/ubi/scan.c: In function \u0027check_what_we_have\u0027:\ndrivers/mtd/ubi/scan.c:960: warning: passing argument 1 of \u0027get_random_bytes\u0027 discards qualifiers from pointer target type\n\nFix the following compilation warnings introduced by commit\n1a49af2ca019dcb4614c32f832bbcb814b61409c:\n\ndrivers/mtd/ubi/io.c: In function \u0027ubi_io_read\u0027:\ndrivers/mtd/ubi/io.c:153: warning: initialization makes integer from pointer without a cast\ndrivers/mtd/ubi/io.c:170: warning: format \u0027%s\u0027 expects type \u0027char *\u0027, but argument 5 has type \u0027int\u0027\ndrivers/mtd/ubi/io.c:177: warning: format \u0027%zd\u0027 expects type \u0027signed size_t\u0027, but argument 7 has type \u0027int\u0027\ndrivers/mtd/ubi/io.c:177: warning: too many arguments for format\n\nAlso, amend the ECC error code string and add brackets and whitespace\nthere - this should make the message readable.\n\nReported-by: Stephen Rothwell \u003csfr@canb.auug.org.au\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "095751a6e0838a712393a74eb0b7b6559dbdbe81", "tree": "286ae0ee280545b85b7636cca91fb8e7f4efc859", "parents": [ "1a49af2ca019dcb4614c32f832bbcb814b61409c" ], "author": { "name": "Matthieu CASTET", "email": "matthieu.castet@parrot.com", "time": "Thu Jun 03 16:14:27 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jun 11 17:23:37 2010 +0300" }, "message": "UBI: generate random image_seq when formatting MTD devices\n\nGenerate random image_seq when attaching empty MTD device (kernel do the\nubi formating).\n\nSigned-off-by: Matthieu CASTET \u003cmatthieu.castet@parrot.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "1a49af2ca019dcb4614c32f832bbcb814b61409c", "tree": "1ab9382a4f2433018b9a38c2449baa835373b51b", "parents": [ "0798cea8c2e1afee59686c51d27d0e96b05e42d1" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Jun 08 10:59:07 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jun 11 17:23:37 2010 +0300" }, "message": "UBI: improve ECC error message\n\nECC errors are quite typical errors on NAND, so it is worth improving\nthe UBI message and print something like\n\nubi_io_read: error -74 (ECC error) while reading 4096 bytes from PEB 1:4 ...\n\nrather than\n\nubi_io_read: error -74 while reading 4096 bytes from PEB 1:4 ...\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "0798cea8c2e1afee59686c51d27d0e96b05e42d1", "tree": "7bc5702e9bdeb37699f3cce0d4641d15676e74f2", "parents": [ "33789fb9d4470e27d18596c0966339e2ca8865a9" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Apr 30 13:02:33 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jun 04 11:30:44 2010 +0300" }, "message": "UBI: improve corrupted flash handling\n\nThis patch improves the way UBI handles corrupted flash, or flash\ncontaining garbage or non-UBI data, which is the same from UBI POW.\nNamely, we do the following:\n\n* if 5% or more PEBs are corrupted, refuse the flash\n* if less than 5% PEBs are corrupted, do not refuse the flash\n and format these PEBs\n* if less than 8 PEBs are corrupted, format them silently, otherwise\n print a warning message.\n\nReported-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nReviewed-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\nTested-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\n" }, { "commit": "33789fb9d4470e27d18596c0966339e2ca8865a9", "tree": "82b5025c7e0b40289977f1ccc9173c1702a4b036", "parents": [ "eb89580e1a8388d206bf143c6c39d001095106ba" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Apr 30 12:31:26 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jun 04 11:30:43 2010 +0300" }, "message": "UBI: introduce eraseblock counter variables\n\nThis is just a preparation patch which introduces several\n\u0027struct ubi_scan_info\u0027 fields which count eraseblocks of different\ntypes. This will be used later on to decide whether it is safe to\nformat the flash or not. No functional changes so far.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nReviewed-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\nTested-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\n" }, { "commit": "eb89580e1a8388d206bf143c6c39d001095106ba", "tree": "a0cee0ac8e62a9731ade3d8aa400b621252ea3f4", "parents": [ "786d78318586cbdc8aec539fe5a4942490267fef" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon May 03 09:04:39 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jun 04 11:30:42 2010 +0300" }, "message": "UBI: introduce a new IO return code\n\nThis patch introduces the %UBI_IO_BAD_HDR_READ return code for\nthe I/O level function. We will use this code in order to distinguish\nbetween \"corrupted header possibly because this is non-ubi data\" and\n\"corrupted header possibly because of real data corruption and ECC error\".\n\nSo far this patch does not introduce any functional change, just a\npreparation.\n\nThis patch is pased on a patch from\nSebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nReviewed-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\nTested-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\n" }, { "commit": "786d78318586cbdc8aec539fe5a4942490267fef", "tree": "3ca3144ce0186dea041782b8a9789caf16d89cbb", "parents": [ "67a3e12b05e055c0415c556a315a3d3eb637e29e" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Apr 30 16:50:22 2010 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jun 04 11:30:42 2010 +0300" }, "message": "UBI: simplify IO error codes\n\nWe do not really need 2 separate error codes for indicating bad VID\nand bad EC headers (UBI_IO_BAD_EC_HDR, UBI_IO_BAD_VID_HDR), it is\nenough to have only one UBI_IO_BAD_HDR return code.\n\nThis patch does not introduce any functional change, only some\ncode simplification.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nReviewed-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\nTested-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\n" }, { "commit": "7ea8085910ef3dd4f3cad6845aaa2b580d39b115", "tree": "d9c1edb5906f943f7d70bfb4b65106e29772d379", "parents": [ "cc967be54710d97c05229b2e5ba2d00df84ddd64" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Wed May 26 17:53:25 2010 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu May 27 22:05:02 2010 -0400" }, "message": "drop unused dentry argument to -\u003efsync\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "be436f6238a17b8432b9de0212bcfc838afb1f85", "tree": "59b685247d6d3647cc82920092759b4b09599aa5", "parents": [ "3f5026222e8a16daaa830eec4d72c6745b74407e" ], "author": { "name": "Shinya Kuribayashi", "email": "shinya.kuribayashi.px@renesas.com", "time": "Thu May 06 19:22:09 2010 +0900" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri May 07 08:33:32 2010 +0300" }, "message": "UBI: misc comment fixes\n\nSigned-off-by: Shinya Kuribayashi \u003cshinya.kuribayashi.px@renesas.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "3f5026222e8a16daaa830eec4d72c6745b74407e", "tree": "a3274b82c86184e8b6e39795dbf11ad5d243f4c3", "parents": [ "af7ad7a0a6c0c1d8497a25b6b8b3b2ce9f52ff04" ], "author": { "name": "Shinya Kuribayashi", "email": "shinya.kuribayashi.px@renesas.com", "time": "Thu May 06 19:21:47 2010 +0900" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri May 07 08:33:10 2010 +0300" }, "message": "UBI: fix s/then/than/ typos\n\nSigned-off-by: Shinya Kuribayashi \u003cshinya.kuribayashi.px@renesas.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "af7ad7a0a6c0c1d8497a25b6b8b3b2ce9f52ff04", "tree": "3073f9591d82feb01e6138e6b5bbac5d76f78e5e", "parents": [ "70d38b9625edba52a809308427f78831c3963f52" ], "author": { "name": "Marc Kleine-Budde", "email": "mkl@pengutronix.de", "time": "Wed May 05 10:17:25 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Thu May 06 09:12:11 2010 +0300" }, "message": "UBI: init even if MTD device cannot be attached, if built into kernel\n\nUBI can be built into the kernel or be compiled as a kernel module.\nFurther on the command line one can specify MTD devices to be attach to\nUBI while loading. In the current implementation the UBI driver refuses\nto load if one of the MTD devices cannot be attached.\n\nConsider:\n1) UBI compiled into the kernel and\n2) a MTD device specified on the command line and\n3) this MTD device contains bogus data (for whatever reason).\n\nDuring init UBI tries to attach the MTD device is this fails the whole\nUBI subsystem isn\u0027t initialized. Later the userspace cannot attach any\nMTD to UBI because UBI isn\u0027t loaded.\n\nThis patch keeps the current behaviour: if UBI is compiled as a module\nand a MTD device cannot be attached the UBI module cannot be loaded,\nbut changes it for the UBI-is-built-into-the-kernel usecase.\n\nIf UBI is builtin, a not attachable MTD device doen\u0027t stop UBI from\ninitializing. This slightly modifies the behaviour if multiple MTD\ndevices are specified on the command line. Now every MTD device is\nprobed and, if possible, attached, i.e. a faulty MTD device doesn\u0027t\nstop the others from being attached.\n\nArtem: tweaked the patch\n\nSigned-off-by: Marc Kleine-Budde \u003cmkl@pengutronix.de\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "70d38b9625edba52a809308427f78831c3963f52", "tree": "ef5e519bd672c4ec2a80b7a78adfb66251a1193c", "parents": [ "01bf0b64579ead8a82e7cfc32ae44bc667e7ad0f" ], "author": { "name": "Kevin Cernekee", "email": "cernekee@gmail.com", "time": "Tue Apr 13 13:30:12 2010 -0700" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon May 03 09:08:13 2010 +0300" }, "message": "UBI: remove reboot notifier\n\nThe UBI reboot notifier causes problems with hibernation. Move this\nfunctionality into the low-level MTD driver instead.\n\nSigned-off-by: Kevin Cernekee \u003ccernekee@gmail.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "5a0e3ad6af8660be21ca98a971cd00f331318c05", "tree": "5bfb7be11a03176a87296a43ac6647975c00a1d1", "parents": [ "ed391f4ebf8f701d3566423ce8f17e614cde9806" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 24 17:04:11 2010 +0900" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Mar 30 22:02:32 2010 +0900" }, "message": "include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h\n\npercpu.h is included by sched.h and module.h and thus ends up being\nincluded when building most .c files. percpu.h includes slab.h which\nin turn includes gfp.h making everything defined by the two files\nuniversally available and complicating inclusion dependencies.\n\npercpu.h -\u003e slab.h dependency is about to be removed. Prepare for\nthis change by updating users of gfp and slab facilities include those\nheaders directly instead of assuming availability. As this conversion\nneeds to touch large number of source files, the following script is\nused as the basis of conversion.\n\n http://userweb.kernel.org/~tj/misc/slabh-sweep.py\n\nThe script does the followings.\n\n* Scan files for gfp and slab usages and update includes such that\n only the necessary includes are there. ie. if only gfp is used,\n gfp.h, if slab is used, slab.h.\n\n* When the script inserts a new include, it looks at the include\n blocks and try to put the new include such that its order conforms\n to its surrounding. It\u0027s put in the include block which contains\n core kernel includes, in the same order that the rest are ordered -\n alphabetical, Christmas tree, rev-Xmas-tree or at the end if there\n doesn\u0027t seem to be any matching order.\n\n* If the script can\u0027t find a place to put a new include (mostly\n because the file doesn\u0027t have fitting include block), it prints out\n an error message indicating which .h file needs to be added to the\n file.\n\nThe conversion was done in the following steps.\n\n1. The initial automatic conversion of all .c files updated slightly\n over 4000 files, deleting around 700 includes and adding ~480 gfp.h\n and ~3000 slab.h inclusions. The script emitted errors for ~400\n files.\n\n2. Each error was manually checked. Some didn\u0027t need the inclusion,\n some needed manual addition while adding it to implementation .h or\n embedding .c file was more appropriate for others. This step added\n inclusions to around 150 files.\n\n3. The script was run again and the output was compared to the edits\n from #2 to make sure no file was left behind.\n\n4. Several build tests were done and a couple of problems were fixed.\n e.g. lib/decompress_*.c used malloc/free() wrappers around slab\n APIs requiring slab.h to be added manually.\n\n5. The script was run on all .h files but without automatically\n editing them as sprinkling gfp.h and slab.h inclusions around .h\n files could easily lead to inclusion dependency hell. Most gfp.h\n inclusion directives were ignored as stuff from gfp.h was usually\n wildly available and often used in preprocessor macros. Each\n slab.h inclusion directive was examined and added manually as\n necessary.\n\n6. percpu.h was updated not to include slab.h.\n\n7. Build test were done on the following configurations and failures\n were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my\n distributed build env didn\u0027t work with gcov compiles) and a few\n more options had to be turned off depending on archs to make things\n build (like ipr on powerpc/64 which failed due to missing writeq).\n\n * x86 and x86_64 UP and SMP allmodconfig and a custom test config.\n * powerpc and powerpc64 SMP allmodconfig\n * sparc and sparc64 SMP allmodconfig\n * ia64 SMP allmodconfig\n * s390 SMP allmodconfig\n * alpha SMP allmodconfig\n * um on x86_64 SMP allmodconfig\n\n8. percpu.h modifications were reverted so that it could be applied as\n a separate patch and serve as bisection point.\n\nGiven the fact that I had only a couple of failures from tests on step\n6, I\u0027m fairly confident about the coverage of this conversion patch.\nIf there is a breakage, it\u0027s likely to be something in one of the arch\nheaders which should be easily discoverable easily on most builds of\nthe specific arch.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nGuess-its-ok-by: Christoph Lameter \u003ccl@linux-foundation.org\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nCc: Lee Schermerhorn \u003cLee.Schermerhorn@hp.com\u003e\n" }, { "commit": "28812fe11a21826ba4c97c6c7971a619987cd912", "tree": "c78daefd595989bbc660ea48d8bc2dc5750321d3", "parents": [ "5fbcae57db69128c14080a7a5a42d0626bfe155c" ], "author": { "name": "Andi Kleen", "email": "andi@firstfloor.org", "time": "Tue Jan 05 12:48:07 2010 +0100" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Sun Mar 07 17:04:48 2010 -0800" }, "message": "driver-core: Add attribute argument to class_attribute show/store\n\nPassing the attribute to the low level IO functions allows all kinds\nof cleanups, by sharing low level IO code without requiring\nan own function for every piece of data.\n\nAlso drivers can extend the attributes with own data fields\nand use that in the low level function.\n\nThis makes the class attributes the same as sysdev_class attributes\nand plain attributes.\n\nThis will allow further cleanups in drivers.\n\nFull tree sweep converting all users.\n\nSigned-off-by: Andi Kleen \u003cak@linux.intel.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "6e9065d756df5dac6dc02b94b82b4f5dbbf38caf", "tree": "ed2f34b18bb9c9af1b772e25127ec1ea1ce4f2cf", "parents": [ "adbf05e3ec6ea380ba145b6fa89d9125ed76eb98" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Jan 25 17:09:30 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Feb 01 15:16:37 2010 +0200" }, "message": "UBI: add write checking\n\nAdd an extra debugging check function which validates writes.\nAfter every write it reads the data back, compares it with the\noriginal data, and complains if they mismatch.\n\nUseful for debugging. No-op if extra debugging checks are disabled.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "adbf05e3ec6ea380ba145b6fa89d9125ed76eb98", "tree": "9da867755d3b915ec1a759658397aaa31bc68263", "parents": [ "0bf1c4399afee6a2031b0ee943a4c016e53f727c" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Jan 20 10:28:58 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Feb 01 15:16:37 2010 +0200" }, "message": "UBI: simplify debugging return codes\n\nUBI debugging functions were a little bit over-engineered and\nreturned more error codes than needed, and the callers had to\ndo useless checks. Simplify the return codes.\n\nImpact: only debugging code is affected, which means that for\n non-developers this is a no-op patch.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "0bf1c4399afee6a2031b0ee943a4c016e53f727c", "tree": "c5d7a62b2c965e65a42f17f6206c28f3c4e57a81", "parents": [ "f9b0080e10e0ce3b8acbe91ae6a50da4f2ed7339" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Jan 12 12:26:42 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Feb 01 15:16:36 2010 +0200" }, "message": "UBI: fix attaching error path\n\nIn the error path of \u0027ubi_attach_mtd_dev()\u0027 we have a tricky situation:\nwe have to release things differently depending on at which point\nthe failure happening. Namely, if @ubi-\u003edev is not initialized, we have\nto free everything ourselves. But if it was, we should not free the @ubi\nobject, because it will be freed in the \u0027dev_release()\u0027 function. And\nwe did not get this situation right.\n\nThis patch introduces additional argument to the \u0027uif_init()\u0027 function.\nOn exit, this argument indicates whether the final \u0027free(ubi)\u0027 will\nhappen in \u0027dev_release()\u0027 or not. So the caller always knows how to\nproperly release the resources.\n\nImpact: all memory is now correctly released when UBI fails to attach\n an MTD device.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "f9b0080e10e0ce3b8acbe91ae6a50da4f2ed7339", "tree": "7f9f45f0e689b1e3a84bdf944c166ce206563caf", "parents": [ "9e0c7ef3f6c95357ce359a4f9223f0dfcd21cef7" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Jan 05 16:48:40 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Feb 01 15:16:36 2010 +0200" }, "message": "UBI: support attaching by MTD character device name\n\nThis patch adds a capability to attach MTD devices by their character\ndevice paths. For example, one can do:\n\n$ modprobe ubi mtd\u003d/dev/mtd0\n\nto attach /dev/mtd0.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "9e0c7ef3f6c95357ce359a4f9223f0dfcd21cef7", "tree": "712c9ab76536d68dfe36299ec71e5f18829075c8", "parents": [ "64a028a6de08545a2c94f302bc7694bf48aee5b5" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Jan 06 14:20:31 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Feb 01 15:16:36 2010 +0200" }, "message": "UBI: mark few variables as __initdata\n\nThe @mtd_devs and @mtd_dev_param variables are used only during the\ninitialization, and all functions that use the variables have\nthe __init prefix. This means we can safely mark the variables\nas __initdata, which is a tiny optimization.\n\nImpact: tiny RAM consumption optimization when UBI is used as a kernel\n module.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "551e28dbe82f9de58993d7587201a2569b942341", "tree": "8f14618a4f5f016d41eb5b263784e3e2a4dd2bf3", "parents": [ "b39bda6e7329b1be6dfc2741ed298b6e814532a5", "c5ce5b46af76f52dea21f467397d24c4ae6cb3ff" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 28 12:57:50 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 28 12:57:50 2010 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.infradead.org/ubi-2.6\n\n* \u0027for-linus\u0027 of git://git.infradead.org/ubi-2.6:\n UBI: fix volume creation input checking\n" }, { "commit": "c5ce5b46af76f52dea21f467397d24c4ae6cb3ff", "tree": "cd45d66f718c6e632ab366e9847129c913821f41", "parents": [ "92dcffb916d309aa01778bf8963a6932e4014d07" ], "author": { "name": "Mika Westerberg", "email": "ext-mika.1.westerberg@nokia.com", "time": "Tue Jan 26 17:47:05 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Jan 27 11:55:43 2010 +0200" }, "message": "UBI: fix volume creation input checking\n\nDo not use an unchecked variable UBI_IOCMKVOL ioctl.\n\nSigned-off-by: Mika Westerberg \u003cext-mika.1.westerberg@nokia.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nCc: stable@vger.kernel.org\n" }, { "commit": "ebddd63b74dcf1cb676d14328d5852f1fee19a8a", "tree": "c924535e6ae791d514ec809c99ebb214d66d97b7", "parents": [ "b531b55a7bde8aa2bdf7023b3afc6df1bf3dcb67" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Jan 18 16:43:44 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Mon Jan 18 18:53:56 2010 +0200" }, "message": "UBI: fix memory leak in update path\n\nWhen truncating an UBI volume, UBI should allocates a PEB-sized\nbuffer but does not release it, which leads to memory leaks.\nThis patch fixes the issue.\n\nReported-by: Marek Skuczynski \u003cmareksk7@gmail.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nTested-by: Marek Skuczynski \u003cmareksk7@gmail.com\u003e\nCc: stable@kernel.org\n" }, { "commit": "b531b55a7bde8aa2bdf7023b3afc6df1bf3dcb67", "tree": "d6045d623d645f6481ae5b1cab5e8288d293f345", "parents": [ "ff998793288b49a3b22d929bf8e56362320905ff" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Jan 05 17:25:59 2010 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Jan 12 13:19:15 2010 +0200" }, "message": "UBI: add more checks to chdev open\n\nWhen opening UBI volumes by their character device names, make\nsure we are opening character devices, not block devices or any\nother inode type.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "ff998793288b49a3b22d929bf8e56362320905ff", "tree": "2f11b74d60bebec123fb26c7cb22af5c5eb0af81", "parents": [ "74d2e4f8d79ae0c4b6ec027958d5b18058662eea" ], "author": { "name": "Peter Horton", "email": "zero@colonel-panic.org", "time": "Tue Jan 05 11:14:36 2010 +0000" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Jan 12 13:19:08 2010 +0200" }, "message": "UBI: initialise update marker\n\nThe in kernel copy of a volume\u0027s update marker is not initialised from the\nvolume table. This means that volumes where an update was unfinnished will\nnot be treated as \"forbidden to use\". This is basically that the update\nfunctionality was broken.\n\nSigned-off-by: Peter Horton \u003czero@colonel-panic.org\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nCc: stable@kernel.org\n" }, { "commit": "fc1495bf99de6f65066b3234813180301ff8b693", "tree": "4858db540f3c57f6f998383056ad4714b969660c", "parents": [ "5476ffd2b78f06cce31a57f8611162918fe1ae3a", "b38882f5c066dc681679e90f1903eda323e605b1" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Dec 10 09:31:45 2009 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Dec 10 09:31:45 2009 -0800" }, "message": "Merge git://git.infradead.org/ubifs-2.6\n\n* git://git.infradead.org/ubifs-2.6:\n UBIFS: fix return code in check_leaf\n UBI: flush wl before clearing update marker\n MAINTAINERS: change e-mail of Artem Bityutskiy\n UBIFS: remove manual O_SYNC handling\n UBIFS: support mounting of UBI volume character devices\n UBI: Add ubi_open_volume_path\n" }, { "commit": "6afaf8a484cbbfd2ccf58a4e5396d1f280469789", "tree": "f32db3212e23a38c794df0c01fcf8505180ca51d", "parents": [ "949cb6232d5fc9fa77cfa441418e12d6f9de163e" ], "author": { "name": "Sebastian Andrzej Siewior", "email": "sebastian@breakpoint.cc", "time": "Sun Nov 29 19:46:02 2009 +0100" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Dec 04 07:47:11 2009 +0200" }, "message": "UBI: flush wl before clearing update marker\n\nubiupdatevol -t does the following:\n- ubi_start_update()\n - set_update_marker()\n - for all LEBs ubi_eba_unmap_leb()\n - clear_update_marker()\n - ubi_wl_flush()\n\nubi_wl_flush() physically erases all PEB, once it returns all PEBs are\nempty. clear_update_marker() has the update marker written after return.\nIf there is a power cut between the last two functions then the UBI\nvolume has no longer the \"update\" marker set and may have some valid\nLEBs while some of them may be gone.\nIf that volume in question happens to be a UBIFS volume, then mount\nwill fail with\n\n|UBIFS error (pid 1361): ubifs_read_node: bad node type (255 but expected 6)\n|UBIFS error (pid 1361): ubifs_read_node: bad node at LEB 0:0\n|Not a node, first 24 bytes:\n|00000000: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff\n\nif there is at least one valid LEB and the wear-leveling worker managed\nto clear LEB 0.\n\nThe patch waits for the wl worker to finish prior clearing the \"update\"\nmarker on flash. The two new LEB which are scheduled for erasing after\nclear_update_marker() should not matter because they are only visible to\nUBI.\n\nSigned-off-by: Sebastian Andrzej Siewior \u003csebastian@breakpoint.cc\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\nCc: stable@kernel.org\n" }, { "commit": "b57102841846d9840dcb1b8b308f6d7369b8e5c5", "tree": "5792c36c373eef0be91714e5a7a3b83d8cf62960", "parents": [ "648f4e3e50c4793d9dbf9a09afa193631f76fa26" ], "author": { "name": "Corentin Chary", "email": "corentincj@iksaif.net", "time": "Mon Sep 28 21:10:11 2009 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Nov 24 08:18:54 2009 +0200" }, "message": "UBI: Add ubi_open_volume_path\n\nAdd an \u0027ubi_open_volume_path(path, mode)\u0027 function which works like\n\u0027open_bdev_exclusive(path, mode, ...)\u0027 where path is the special file\nrepresenting the UBI volume, typically /dev/ubi0_0.\n\nThis is needed to teach UBIFS being able to mount UBI character devices.\n\n[Comments and the patch were amended a bit by Artem]\n\nSigned-off-by: Corentin Chary \u003ccorentincj@iksaif.net\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "fa3012318bfb395552baef69bb1ebe87e64945c8", "tree": "5095c1dda820a408609c6e0bd5bc2e7f4c359eae", "parents": [ "7beeec88e5f379680abeb4244b0781e102201c0e" ], "author": { "name": "Michael Roth", "email": "mroth@nessie.de", "time": "Sun Oct 04 18:14:29 2009 +0200" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Nov 09 09:40:56 2009 +0100" }, "message": "Kconfig: Remove useless and sometimes wrong comments\n\nAdditionally, some excessive newlines removed.\n\nSigned-off-by: Michael Roth \u003cmroth@nessie.de\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "774b1382106529ad963d011931c1b911db05be68", "tree": "4d3a71c4b894a6b92c32391f5c8d79a2289dd2d0", "parents": [ "2eadaad67b2b6bd132eda105128d2d466298b8e3" ], "author": { "name": "Roel Kluin", "email": "roel.kluin@gmail.com", "time": "Fri Oct 16 14:00:17 2009 +0200" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 20 10:13:49 2009 +0300" }, "message": "UBI: fix check on unsigned long\n\nresult is unsigned, the wrong check was used.\n\nSigned-off-by: Roel Kluin \u003croel.kluin@gmail.com\u003e\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "2eadaad67b2b6bd132eda105128d2d466298b8e3", "tree": "31ca1bf2d0b2500a0600deecb81cd5af5230d0f6", "parents": [ "012abeea669ea49636cf952d13298bb68654146a" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Wed Sep 30 10:01:28 2009 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Tue Oct 20 10:13:04 2009 +0300" }, "message": "UBI: fix backward compatibility\n\nCommit 32bc4820287a1a03982979515949e8ea56eac641 did not fully fix\nthe backward compatibility issues. We still fail to properly handle\nsituations when the first PEB contains non-zero image sequence\nnumber, but one of the following PEBs contains zero image sequence\nnumber. For example, this may happen if we mount a new image with\nan old kernel, and then try to mount it in the new kernel.\n\nThis patch should fix the issue.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" }, { "commit": "342ff1a1b558ebbdb8cbd55ab6a63eca8b2473ca", "tree": "1f967f283dade6e03897169bb29513354f49f910", "parents": [ "50223e486cabdcf7e540e519da1f26bab3084e5d", "24ed7a97464db44592495f98cff8bcee02f92bc2" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Sep 22 07:51:45 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Sep 22 07:51:45 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial: (34 commits)\n trivial: fix typo in aic7xxx comment\n trivial: fix comment typo in drivers/ata/pata_hpt37x.c\n trivial: typo in kernel-parameters.txt\n trivial: fix typo in tracing documentation\n trivial: add __init/__exit macros in drivers/gpio/bt8xxgpio.c\n trivial: add __init macro/ fix of __exit macro location in ipmi_poweroff.c\n trivial: remove unnecessary semicolons\n trivial: Fix duplicated word \"options\" in comment\n trivial: kbuild: remove extraneous blank line after declaration of usage()\n trivial: improve help text for mm debug config options\n trivial: doc: hpfall: accept disk device to unload as argument\n trivial: doc: hpfall: reduce risk that hpfall can do harm\n trivial: SubmittingPatches: Fix reference to renumbered step\n trivial: fix typos \"man[ae]g?ment\" -\u003e \"management\"\n trivial: media/video/cx88: add __init/__exit macros to cx88 drivers\n trivial: fix typo in CONFIG_DEBUG_FS in gcov doc\n trivial: fix missing printk space in amd_k7_smp_check\n trivial: fix typo s/ketymap/keymap/ in comment\n trivial: fix typo \"to to\" in multiple files\n trivial: fix typos in comments s/DGBU/DBGU/\n ...\n" }, { "commit": "fd589a8f0a13f53a2dd580b1fe170633cf6b095f", "tree": "942c50188ca58041b0453189e710eafcfebaea57", "parents": [ "4f37940d64a155c025968118849b596f6aaa8128" ], "author": { "name": "Anand Gadiyar", "email": "gadiyar@ti.com", "time": "Thu Jul 16 17:13:03 2009 +0200" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Sep 21 15:14:55 2009 +0200" }, "message": "trivial: fix typo \"to to\" in multiple files\n\nSigned-off-by: Anand Gadiyar \u003cgadiyar@ti.com\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "de75c771b4cc4da963164a538a8448128301bc35", "tree": "a8fea3a1a6c68324b0b425c482f3c2ef5c7d1b22", "parents": [ "867996b15c1f0a98d2c405bada907e97499ba8c2" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jul 24 16:18:04 2009 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Aug 14 20:02:20 2009 +0300" }, "message": "UBI: improve NOR flash erasure quirk\n\nMore testing of NOR flash against power cuts showed that sometimes\neraseblocks may be unwritable, and we cannot really invalidate\nthem before erasure. But in this case the eraseblock probably\ncontains garbage anyway, and we do not have to invalidate the\nheaders. This assumption might be not true, but this is at least\nwhat I have observed. So if we cannot invalidate the headers,\nwe make sure that the PEB does not contain valid VID header.\nIf this is true, everything is fine, otherwise we panic.\n" }, { "commit": "867996b15c1f0a98d2c405bada907e97499ba8c2", "tree": "3ca5bfb6e74ef5697bb89d29365f6c4df7ab4c03", "parents": [ "758d8e46347aee199e7025b8c571bab75d2de63f" ], "author": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Jul 24 15:31:33 2009 +0300" }, "committer": { "name": "Artem Bityutskiy", "email": "Artem.Bityutskiy@nokia.com", "time": "Fri Aug 14 20:02:20 2009 +0300" }, "message": "UBI: introduce flash dump helper\n\nUseful for debugging problems, compiled in only if UBI debugging\nis enabled. This patch also makes the UBI writing function dump\nthe flash if it fails to write.\n\nSigned-off-by: Artem Bityutskiy \u003cArtem.Bityutskiy@nokia.com\u003e\n" } ], "next": "758d8e46347aee199e7025b8c571bab75d2de63f" }