linux-ext4 - Re: RFC Per group orphan list

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]

Date:	Thu, 29 Apr 2010 00:17:13 +0200
From:	Jan Kara <jack@...e.cz>
To:	Dmitry Monakhov <dmonakhov@...nvz.org>
Cc:	linux-ext4@...r.kernel.org
Subject: Re: RFC Per group orphan list

  Hi,

> Currently orphan list is global, which require per-sb mutex
> to protect add/del from the list. But it is not really necessary.
> Just think about it we have to add inode to orphan list for each
> ->write_begin call(for each page) if i_size will grow.
  Really? I don't see we would add inode to orphan list on every
->write_begin. We only do it in case write fails (ENOSPC, page fault
or so) but that's not interesting performance wise. The place where
we do add inode to orphan list are truncate path and file extending
direct IO (which might be what you meant).

> For example btrfs support per-ctree orphan list.
> So IMHO per-group orphan list seems to be more natural here.
> So my proposal is following:
> 1) Introduce new incompat feature for per-group orphan list. 
> 2) Store per-group head of the list in group descriptor
>    (Where are several reserved fields in grp-desc).
> 3) Maintain per-group orphan which protected by per-group lst-mutex. 
> 4) Let fsck to scan group descriptors after unclean umount.Since they
>    are close to each other performance penalty will be negligible.
> 
> Seems that development complexity of this feature is not too high, 
> what do you think.
  If you can show it helps in some reasonably realistic load - several direct
IO writers might be interesting, I think it might be worth it. OTOH it will
complicate orphan recovery because that would have to scan all group
descriptors (and there is lots of them on a few TB filesystem). But that will
be probably fine since most of them will have empty list.

								Honza
-- 
Jan Kara <jack@...e.cz>
SuSE CR Labs
--
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