lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20131204234435.GF19914@thunk.org> Date: Wed, 4 Dec 2013 18:44:35 -0500 From: Theodore Ts'o <tytso@....edu> To: "Darrick J. Wong" <darrick.wong@...cle.com> Cc: Akira Fujita <a-fujita@...jp.nec.com>, 'ext4 development' <linux-ext4@...r.kernel.org> Subject: Re: [PATCH] mke2fs: Fix block bitmaps initalization with -O ^resize_inode On Sat, Nov 30, 2013 at 12:06:24PM -0800, Darrick J. Wong wrote: > Hi Ted, I was hoping you might resolve a question for us: > > And there is a bug that block group which has backup super block > > and group descriptor block are not write out to device in wrtie_bitmaps() > > when BLOCK_UNINIT is set. > > So at this time, I attempted to fix this by cleaning BLOCK_UNIIT. > > Is it the case that a group should only have BLOCK_UNINIT set if the group is > totally empty (no group metadata, no blocks allocated to files) as Akira says? > > Or is it the case that a group can have BLOCK_UNINIT set if the group contains > group metadata but no blocks are allocated to files? The meaning of BLOCK_UNINIT is that the contents of that block group's block allocation bitmap is not initialized. This causes libext2fs to skip writing the block allocation bitmap, and to also skip reading the block allocation bitmap (which is why it substitutes all zero's instead of reading the allocation bitmap block). Before allocating a block from a block group that has the BLOCK_UNINIT flag set, it is important that the kernel or the userspace library first initialize the block allocation bitmap and clear the BLOCK_UNINIT flag. When allocating blocks, implementations MUST be able to initiaize the allocation bitmap for block groups which has the block group's own metadata blocks (backup superblock and bg descriptor blocks if any, reserved bg blocks, the allocation bitmaps, and inode table blocks) in use. Implementations SHOULD be able to initialize bitmaps for block groups that have metadata blocks from other block groups if the case of flex_bg. However, historically there were some implementations that didn't handle this correctly, which is why mke2fs initializes the block bitmap and clears BLOCK_UNINIT in block groups that have metadata blocks for other block groups. Optionally, implementations MAY set the BLOCK_UNINIT bit after data blocks have been deallocated from a block group such that the only blocks in use are the block group's metadata groups. Also, some implementations MAY clear the BLOCK_UNINIT bit and initialize the block allocation bitmap early --- for example, when allocating an inode in the block group. This shouldn't be required, however, and so implementations SHOULD correctly handle a situation where an inode has been allocated in the inode table, but BLOCK_UNINIT is set. All of this basically boils down to the two rules of thumb: 1) The BLOCK_UNINIT bit is fundamentally about whether the block allocation bitmap is valid, and whether mke2fs can skip needing to initialize the block, and whether e2fsck, dumpe2fs, debugfs, etc. can skip reading said allocation bitmap. 2) The IETF rule of "be conservative in what you send, and liberal in what you accept" applies. Does that help? Thanks, - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists