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
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20100729121721.GC3757@quack.suse.cz>
Date:	Thu, 29 Jul 2010 14:17:21 +0200
From:	Jan Kara <jack@...e.cz>
To:	Christoph Hellwig <hch@...radead.org>
Cc:	Jan Kara <jack@...e.cz>, LKML <linux-kernel@...r.kernel.org>,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH 1/3] bdi: Use parent filesystem BDI for inodes not
 capable of writeback

On Thu 29-07-10 04:12:10, Christoph Hellwig wrote:
> Given that we only need the per-mapping one for the bdev fs what about
> doing:
> 
> /*
>  * Return the writeback-relevant backing device for this inode.
>  *
>  * For a normal filesystem this must always be the bdi hanging off the
>  * superblock, given that we only expect one bdi per filesystems in
>  * the per-superblock sync functions.  But the block device special
>  * filesystem requires a quick given that it contains the internal
>  * I/O inodes for block devices on a single superblock.  This works
>  * because the block deevice filesystem inodes are never user visible
>  * and we will never do a per-superblock sync on it.
>  */
> static struct backing_dev_info *inode_to_bdi(struct inode *inode)
> {
> 	if (inode->i_sb == blockdev_superblock)
> 		return inode->i_mapping->backing_dev_info;
> 	return inode->i_sb->s_bdi;
> }
> 
> that also avoids the need for doing the bdi capabilities audit ASAP.
  Well, but I'm not sure we'll help ourselves with having to do audit this
way. Some other pseudofilesystems can play tricks with backing_dev_info as
well and thus inode->i_mapping->backing_dev_info != inode->i_sb->s_bdi. In
particular MTD seems to do such things. As I'm looking at the code, it
looks like a similar case as blockdev filesystem.

								Honza
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ