From mboxrd@z Thu Jan 1 00:00:00 1970 From: William Lee Irwin III Subject: Re: [PATCH 12/11] unpaged: fix sound Bad page states Date: Mon, 21 Nov 2005 13:57:10 -0800 Message-ID: <20051121215710.GR6916@holomorphy.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Hugh Dickins Cc: Andrew Morton , Takashi Iwai , Russell King , "David S. Miller" , Linus Torvalds , Nick Piggin , Dave Airlie , linux-kernel@vger.kernel.org, alsa-devel@lists.sourceforge.net List-Id: alsa-devel@alsa-project.org On Mon, Nov 21, 2005 at 06:12:38PM +0000, Hugh Dickins wrote: > Earlier I unifdefed PageCompound, so that snd_pcm_mmap_control_nopage > and others can give out a 0-order component of a higher-order page, > which won't be mistakenly freed when zap_pte_range unmaps it. But > many Bad page states reported a PG_reserved was freed after all: I had > missed that we need to say __GFP_COMP to get compound page behaviour. > Some of these higher-order pages are allocated by snd_malloc_pages, some > by snd_malloc_dev_pages; or if SBUS, by sbus_alloc_consistent - but that > has no gfp arg, so add __GFP_COMP into its sparc32/64 implementations. > I'm still rather puzzled that DRM seems not to need a similar change. > Signed-off-by: Hugh Dickins Acked-by: William Irwin -- wli