From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alasdair G Kergon Subject: Re: Re: [PATCH] raid1: use __GFP_NOFAIL Date: Fri, 1 May 2009 19:15:59 +0100 Message-ID: <20090501181559.GF23302@agk.fab.redhat.com> References: <20090430231443.GD23302@agk.fab.redhat.com> Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: device-mapper development List-Id: dm-devel.ids On Fri, May 01, 2009 at 08:04:12AM -0400, Mikulas Patocka wrote: > As I said on chat, don't remove that GFP_IO allocation. The code is > deadlocky anyway and allocation from a mempool with GFP_ATOMIC and, in > case of failure, allocation from memory with GFP_IO is the best that can > be done to reduce the probability of the deadlock. OK - I'll apply this one pending a more-complicated fix that uses the mempool correctly. Alasdair -- agk@redhat.com