Blackbird™ Linux sources for OpenPOWER https://git.raptorcs.com/git/blackbird-op-linux/atom?h=v4.9.3 2016-10-15T19:09:13+00:00 2016-10-15T19:09:13+00:00 Linus Torvalds torvalds@linux-foundation.org 2016-10-15T19:09:13+00:00 urn:sha1:df34d04a6f09ba41037e58c3df6d6fbed0ffcde9 Pull befs fixes from Luis de Bethencourt: "I recently took maintainership of the befs file system [0]. This is the first time I send you a git pull request, so please let me know if all the below is OK. Salah Triki and myself have been cleaning the code and fixing a few small bugs. Sorry I couldn't send this sooner in the merge window, I was waiting to have my GPG key signed by kernel members at ELCE in Berlin a few days ago." [0] https://lkml.org/lkml/2016/7/27/502 * tag 'befs-v4.9-rc1' of git://github.com/luisbg/linux-befs: (39 commits) befs: befs: fix style issues in datastream.c befs: improve documentation in datastream.c befs: fix typos in datastream.c befs: fix typos in btree.c befs: fix style issues in super.c befs: fix comment style befs: add check for ag_shift in superblock befs: dump inode_size superblock information befs: remove unnecessary initialization befs: fix typo in befs_sb_info befs: add flags field to validate superblock state befs: fix typo in befs_find_key befs: remove unused BEFS_BT_PARMATCH fs: befs: remove ret variable fs: befs: remove in vain variable assignment fs: befs: remove unnecessary *befs_sb variable fs: befs: remove useless initialization to zero fs: befs: remove in vain variable assignment fs: befs: Insert NULL inode to dentry fs: befs: Remove useless calls to brelse in befs_find_brun_dblindirect ... 2016-10-08T09:01:36+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-13T17:11:21+00:00 urn:sha1:a17e7d2010b44103e3b8e00d0c8c510606457bc6 Fixing the following checkpatch.pl errors: ERROR: "foo * bar" should be "foo *bar" + befs_blocknr_t blockno, befs_block_run * run); WARNING: Missing a blank line after declarations + struct buffer_head *bh; + befs_debug(sb, "---> %s length: %llu", __func__, len); WARNING: Block comments use * on subsequent lines + /* + Double indir block, plus all the indirect blocks it maps. (and other instances of these) Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:36+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-13T17:11:20+00:00 urn:sha1:a20af5f9eaac083e2865e94f37e47af74d70f187 Convert function descriptions to kernel-doc style. Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:35+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-13T17:11:19+00:00 urn:sha1:d327e612bd1f854f08bc4c419122a41440455cbe Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:34+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-11T21:15:16+00:00 urn:sha1:02d91f97fdd62e1d01191e3bb1c42092bd807951 Fixing typos in kernel-doc function descriptions in fs/befs/btree.c. Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:34+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-09T21:51:44+00:00 urn:sha1:103c0fb3400b3efc74c806db2fd18e9eedc510d5 Fixing the following checkpatch.pl error: ERROR: "foo * bar" should be "foo *bar" +befs_load_sb(struct super_block *sb, befs_super_block * disk_sb) And the following warnings: WARNING: suspect code indent for conditional statements (8, 12) + if (disk_sb->fs_byte_order == BEFS_BYTEORDER_NATIVE_LE) + befs_sb->byte_order = BEFS_BYTESEX_LE; WARNING: suspect code indent for conditional statements (8, 12) + else if (disk_sb->fs_byte_order == BEFS_BYTEORDER_NATIVE_BE) + befs_sb->byte_order = BEFS_BYTESEX_BE; WARNING: break quoted strings at a space character + befs_error(sb, "blocksize(%u) cannot be larger" + "than system pagesize(%lu)", befs_sb->block_size, WARNING: line over 80 characters + if (befs_sb->log_start != befs_sb->log_end || befs_sb->flags == BEFS_DIRTY) { Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:33+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-09T21:24:09+00:00 urn:sha1:11674239f927c900bc50f8f2f5a4ec3e4a3df89b The description of befs_load_sb was confusing the kernel-doc system since, because it starts with /**, it thinks it will document the function with kernel-doc formatting. Which it isn't. Fix other comment style issues in the file while we are at it. Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:31+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-09T21:23:36+00:00 urn:sha1:bbe1bd0b6bba138eaf441c6c964bde9866da8808 ag_shift and blocks_per_ag contain the same information in different ways, same as block_shift and block_size do. It is worth checking this two are consistent, but since blocks_per_ag isn't documented as mandatory to use some implementations of befs don't enforce this, so making it non-fatal if they don't match and just having it as a warning. Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:29+00:00 Luis de Bethencourt luisbg@osg.samsung.com 2016-08-09T21:20:23+00:00 urn:sha1:d1a8c70676c7e2cd4b80f494aed17b5813f2be44 befs_dump_super_block() wasn't giving the inode_size information when dumping all elements of the superblock. Add this element to have complete information of the superblock. Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com> Signed-off-by: Salah Triki <salah.triki@gmail.com> 2016-10-08T09:01:28+00:00 Salah Triki salah.triki@gmail.com 2016-08-11T11:04:52+00:00 urn:sha1:78f647c27f17b10bd7e1ce5d66a3755b1848efa5 There is no need to init block, since it will be overwitten later by iaddr2blockno(). Signed-off-by: Salah Triki <salah.triki@gmail.com> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>