linux-ext4 - Re: [PATCH 0/2 v8] XFS TESTS: ENOSPC Punch Hole Test

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <1311026953.2790.49.camel@doink>
Date:	Mon, 18 Jul 2011 17:09:13 -0500
From:	Alex Elder <aelder@....com>
To:	Allison Henderson <achender@...ux.vnet.ibm.com>
CC:	<linux-ext4@...r.kernel.org>, <linux-fsdevel@...r.kernel.org>,
	<xfs@....sgi.com>
Subject: Re: [PATCH 0/2 v8] XFS TESTS: ENOSPC Punch Hole Test

On Sun, 2011-07-17 at 16:52 -0700, Allison Henderson wrote:
> This patch set adds the ENOSPC test that was used for ext4 punch hole testing.
> This test will verify that a hole can be punched even when the disk is full.
> Reserved blocks should be used to complete the operation when there is not blocks
> to further fragment the file.
> 
> Because punching a hole does not always require extra blocks, there needs to 
> be several iterations of punching holes, and then filling the file system to 100% 
> usage before it is forced to grow the tree in order to handle the fragmentation. 
> The growing of the tree is what would cause ENOSPC if not for the use of reserved blocks.

I have reviewed both of your updated patches.  They
both look good to me.  I'll give it another day or
two for others to comment if they like, but unless
someone suggests changes I'll commit this for you.

Reviewed-by: Alex Elder <aelder@....com>




> Allison Henderson (2):
>   XFS TESTS: ENOSPC Punch Hole: Move su routines in 123 to common.rc
>   XFS TESTS: Add ENOSPC Hole Punch Test
> 
>  123       |   24 ------------
>  256       |  120 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  256.out   |    1 +
>  common.rc |   81 +++++++++++++++++++++++++++++++++++++++++
>  group     |   12 ++++--
>  5 files changed, 210 insertions(+), 28 deletions(-)
>  create mode 100644 256
>  create mode 100644 256.out
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



--
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