| To: | David Chinner <dgc@xxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 4/7][TAKE5] support new modes in fallocate |
| From: | Nathan Scott <nscott@xxxxxxxxxx> |
| Date: | Thu, 28 Jun 2007 10:53:08 +1000 |
| Cc: | Andreas Dilger <adilger@xxxxxxxxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx>, "Amit K. Arora" <aarora@xxxxxxxxxxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, suparna@xxxxxxxxxx, cmm@xxxxxxxxxx |
| In-reply-to: | <20070628003921.GW989688@sgi.com> |
| Organization: | Aconex |
| References: | <20070614120413.GD86004887@sgi.com> <20070614193347.GN5181@schatzie.adilger.int> <20070625132810.GA1951@amitarora.in.ibm.com> <20070625134500.GE1951@amitarora.in.ibm.com> <20070625150320.GA8686@amitarora.in.ibm.com> <20070625214626.GJ5181@schatzie.adilger.int> <20070626231431.GO31489@sgi.com> <20070627034915.GR6652@schatzie.adilger.int> <20070627133657.GQ989688@sgi.com> <1182986916.15488.88.camel@edge.yarra.acx> <20070628003921.GW989688@sgi.com> |
| Reply-to: | nscott@xxxxxxxxxx |
| Sender: | xfs-bounce@xxxxxxxxxxx |
On Thu, 2007-06-28 at 10:39 +1000, David Chinner wrote: > > > I don't think it does - swapfile I/O looks like it goes direct to > bio without passing through the filesystem. When the swapfile is > mapped, it scans and records the extent map of the entire swapfile > in a separate structure and AFAICT the swap code uses that built map > without touching the filesystem at all. > > If that is true then the written/unwritten state of the extents is > irrelevant; all we need is allocated disk space for the file and > swapping should work. And it's not like anyone should be reading > the contents of that swapfile through the filesystem, either. ;) Ah, yes, good point - thats true. Unwritten extents are ideal for this then, as attempts to read swap via the regular interfaces will return zeros instead of random swapped out memory contents. cheers. |
| Previous by Date: | Re: [PATCH 4/7][TAKE5] support new modes in fallocate, David Chinner |
|---|---|
| Next by Date: | Re: [patch 1/3] Fix XFS_IOC_FSGEOMETRY_V1 in compat mode, David Chinner |
| Previous by Thread: | Re: [PATCH 4/7][TAKE5] support new modes in fallocate, David Chinner |
| Next by Thread: | Re: [PATCH 4/7][TAKE5] support new modes in fallocate, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |