{"push_id":1104410472,"size":40,"distinct_size":40,"ref":"refs/heads/master","head":"6dcd8a742df38f4ec4dd288e83c9d917fc7a2255","before":"3d9879c4f7e27cad55e30b054ce570e601cf431a","commits":[{"sha":"4519c9ca6f8e85713989a4faf113f6468d5a1152","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Move pass5 to immediately follow pass1\n\nThe purpose of pass1 is to make sure all bits are represented\nproperly for inodes. After that, the bitmaps should be kept in\nsync by subsequent passes. The purpose of the old pass5 was really\nto free any blocks that were not linked somehow to dinodes.\n\nThis patch moves that reconciliation to immediately follow pass1.\nThat allows us to free the bitmap memory sooner, making subsequent\npasses use the rgrp bitmaps for their work. The purpose is to\nreduce the memory footprint of fsck.gfs2.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/4519c9ca6f8e85713989a4faf113f6468d5a1152"},{"sha":"faeebbcdc222b554ad066e704e0eea11bd2cdc27","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Convert block_type to bitmap_type after pass1 and 5\n\nThis patch changes all the passes that happen after pass1 and 5\nfrom using block_type to using bitmap_type. This is a step toward\neliminating the blockmap after pass1 and pass5 are complete.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/faeebbcdc222b554ad066e704e0eea11bd2cdc27"},{"sha":"54d819d3eb39e2a2b4a519dc6d9366baf4efdd39","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Change bitmap_type variables to int\n\nThis patch changes all references of \"q\" from a uint8_t to a normal\nint. This will allow us to check for bad return codes at some point.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/54d819d3eb39e2a2b4a519dc6d9366baf4efdd39"},{"sha":"e352d9865ee230de00c12a03ba0d6a3b812abbda","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Use di_entries to determine if lost+found was created\n\nBefore this patch, function make_sure_lf_exists compared the bitmap\nto the blockmap to determine if the lost+found directory was created.\nSince we're phasing out the blockmap, and only want to use the\nbitmap, we need an alternative. This patch changes it to check the\nnumber of directory entries in the root directory has increased.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/e352d9865ee230de00c12a03ba0d6a3b812abbda"},{"sha":"5c3f62df970cefe922318223d7862389ec8e85e7","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: pass1b shouldn't complain about non-bitmap blocks\n\nOne of the previous patches switched pass1b to stop using the\nblockmap and start using the bitmap. One major difference between\nthem is that the blockmap has a value (0 - 3) for every block on\nthe device, whereas the bitmap has a value (0 - 3) for every block\nin the file system. It does not have a value for blocks that are\non the device but aren't in the file system, such as the superblock\nand the blocks preceding it, the blocks reserved for the rgrps\nand their respective bitmaps.\n\nThat means the block_map function returned 0 for non-fs blocks,\nbut the bitmap function returned -1 (meaning invalid block).\nThe result was that pass1b complained about every non-fs block.\n\nThis patch makes pass1b check the return code from function\nbitmap_type() and if it's a negative number, it skips over it.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/5c3f62df970cefe922318223d7862389ec8e85e7"},{"sha":"8a299130dbaa5f5a6c3120b426a21f1e235e1a72","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Change all fsck_blockmap_set to fsck_bitmap_set\n\nThis patch changes all functions past pass1 so that they set the\nbits in the bitmap rather than bits in the blockmap. This is a\nstep toward being able to free the blockmap after pass1.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/8a299130dbaa5f5a6c3120b426a21f1e235e1a72"},{"sha":"3552c6b9eb77eabcd59fa22172f5209df0e6f900","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Move set_ip_blockmap to pass1\n\nSince function set_ip_blockmap is now only called from pass1,\nmove it to pass1 and make it static.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/3552c6b9eb77eabcd59fa22172f5209df0e6f900"},{"sha":"2744575f37549f1b73343e9196297b6a88f43bf9","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Remove unneeded parameter instree from set_ip_blockmap\n\nThis patch removes parameter instree from function set_ip_blockmap,\nsince it is only called in one place.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/2744575f37549f1b73343e9196297b6a88f43bf9"},{"sha":"e662521a03290bd5542a65b0191cdf75c0e46139","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Move leaf repair to pass2\n\nWe don't want to try to repair directory leaf blocks from pass1\nbecause we haven't verified the bitmap yet, so we cannot do\nblock allocations if necessary. Therefore, we postpone leaf\nrepair to pass2, where all the other directory integrity checks\nare done. We also change fsck_blockmap_set to fsck_bitmap_set.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/e662521a03290bd5542a65b0191cdf75c0e46139"},{"sha":"4a1e4d8c2a9ab5eff3f3f54364c2d482a9e783c8","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Eliminate astate code\n\nSince all the passes after pass1 now use the rgrp bitmaps for the\ndefinitive state of a block, there's no more need to re-sync the\nblockmap with the bitmap. Therefore, we can completely eliminate\nthe astate save/restore and reprocessing code.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/4a1e4d8c2a9ab5eff3f3f54364c2d482a9e783c8"},{"sha":"c201ff82ff58a1e04212badce010e00bd1def140","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Move reprocess code to pass1\n\nSince all the passes after pass1 no longer care about the blockmap,\nwe only need to keep the blockmap in sync with the bitmap in pass1.\nTherefore, we can safely move the \"reprocess_inode\" stuff to pass1.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/c201ff82ff58a1e04212badce010e00bd1def140"},{"sha":"7b8d1b6e050cec8634dc23136beebdbef8e18bf4","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Separate out functions that may only be done after pass1\n\nThis patch adds a new header file and source file which has the\npurpose of separating out the functions that may only be called\nafter pass1 from the functions that pass1 may call. This is to\nensure that there will never be inconsistencies between blockmap\n(which pass1 uses) and bitmap (which subsequent passes use).\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/7b8d1b6e050cec8634dc23136beebdbef8e18bf4"},{"sha":"b605ea0036696513db3b377f23576c3bbda5e611","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Divest check_metatree from fsck_blockmap_set\n\nThis patch replaces the call to function fsck_blockmap_set in\nfunction check_metatree (which is used in all passes) to a\ncorresponding call to fsck_bitmap_set. This is another step to\nhaving pass1 be the only function to manipulate the blockmap.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/b605ea0036696513db3b377f23576c3bbda5e611"},{"sha":"f8203271fac38964850c36f7bb8fb363faeb75f9","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: eliminate fsck_blockmap_set from check_eattr_entries\n\nBefore this patch, metawalk function check_eattr_entries was calling\nfunction fsck_blockmap_set, but only on error conditions.\nThis patch makes it the caller's responsibility. Since pass1 called\ncheck_eattr_entries, it needed to set the blockmap and bitmap, so\nthat's now included in the pass1->check_eattr_extentry caller.\nSubsequent passes don't need to set the blockmap, but they do need\nto set the bitmap. That's also moved to the pass->check_eattr_extentry\nbut in reality, it's only delete_eattr_extentry that needs to do this.\nThe only other caller is check_eattr_extentry_refs in pass1b, but\nthat's a special case in which we don't want to delete the eattrs.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/f8203271fac38964850c36f7bb8fb363faeb75f9"},{"sha":"97696af431be9682f16dd0f73d2ace62af77eac6","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Move blockmap stuff to pass1.c\n\nNow that we've eliminated all blockmap dependencies from the\nother passes, we can move the blockmap-related functions from\nmetawalk to pass1.c. This will ensure there won't be new calls\nadded.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/97696af431be9682f16dd0f73d2ace62af77eac6"},{"sha":"eb83797b704666e7baba6fe8e3d61835cc63e096","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck: make pass1 call bitmap reconciliation AKA pass5\n\nThis patch makes pass1 call what used to be pass5. That enables\npass1 to pass its private blockmap in as a private variable.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/eb83797b704666e7baba6fe8e3d61835cc63e096"},{"sha":"8b3a83763bcbc35ff5ad6d9f1e1b7ee57aa9fdc2","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: make blockmap global variable only to pass1\n\nThis patch changes the variable \"bl\" from being a global variable\nto being global only to pass1. This is another step toward allowing\npass1 to free the blockmap when it's done.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/8b3a83763bcbc35ff5ad6d9f1e1b7ee57aa9fdc2"},{"sha":"1dc0025223232470c871153319c53a05fca688dd","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: Add wrapper function pass1_check_metatree\n\nThis patch adds a wrapper function pass1_check_metatree, which\nsimply calls the normal check_metatree. Since the regular function\nnow frees inodes from the bitmap but not the blockmap, pass1 needs\nthe wrapper function to keep its blockmap in sync with the bitmap.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/1dc0025223232470c871153319c53a05fca688dd"},{"sha":"994d97724a2214b3b13a1b47c5e34fcd98e49ce2","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: pass counted_links into fix_link_count in pass4\n\nThis patch changes function fix_link_count so that it accepts the\ncounted_links rather than a pointer to the inode info structure.\nThat makes it more versatile for future expansion.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/994d97724a2214b3b13a1b47c5e34fcd98e49ce2"},{"sha":"7505d11eff02c57d2ed4447aa599ce015cd29e62","author":{"email":"0c71a8c45d1c28c49aceef3749b4faa9d3c35c54@redhat.com","name":"Bob Peterson"},"message":"fsck.gfs2: refactor pass4 function scan_inode_list\n\nThis patch makes no functional changes. It just cuts a large swath\nof function scan_inode_list and puts it in a separate function,\nhandle_unlinked. This makes it more versatile for later in order\nto save memory.\n\nSigned-off-by: Bob Peterson <rpeterso@redhat.com>","distinct":true,"url":"https://api.github.com/repos/andyprice/gfs2-utils/commits/7505d11eff02c57d2ed4447aa599ce015cd29e62"}]}