| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 2/6] vmap: flush vmap aliases when mapping fails |
| From: | Alex Elder <aelder@xxxxxxx> |
| Date: | Fri, 25 Mar 2011 16:00:43 -0500 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1300860870-15471-3-git-send-email-david@xxxxxxxxxxxxx> |
| References: | <1300860870-15471-1-git-send-email-david@xxxxxxxxxxxxx> <1300860870-15471-3-git-send-email-david@xxxxxxxxxxxxx> |
| Reply-to: | aelder@xxxxxxx |
On Wed, 2011-03-23 at 17:14 +1100, Dave Chinner wrote: > From: Dave Chinner <dchinner@xxxxxxxxxx> > > On 32 bit systems, vmalloc space is limited and XFS can chew through > it quickly as the vmalloc space is lazily freed. This can result in > failure to map buffers, even when there is apparently large amounts > of vmalloc space available. Hence, if we fail to map a buffer, purge > the aliases that have not yet been freed to hopefuly free up enough > vmalloc space to allow a retry to succeed. Looks good. Reviewed-by: Alex Elder <aelder@xxxxxxx> > Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx> > Reviewed-by: Christoph Hellwig <hch@xxxxxx> . . . |
| Previous by Date: | Re: [PATCH 1/6] xfs: preallocation transactions do not need to be synchronous, Alex Elder |
|---|---|
| Next by Date: | Re: [PATCH 3/6] xfs: introduce inode cluster buffer trylocks for xfs_iflush, Alex Elder |
| Previous by Thread: | [PATCH 2/6] vmap: flush vmap aliases when mapping fails, Dave Chinner |
| Next by Thread: | [PATCH 4/6] xfs: xfs_trans_read_buf() should return an error on failure, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |