linux-kernel - Re: [rfc][patch] remove nopage

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20071112043644.GA20602@wotan.suse.de>
Date:	Mon, 12 Nov 2007 05:36:44 +0100
From:	Nick Piggin <npiggin@...e.de>
To:	Jeff Garzik <jgarzik@...ox.com>
Cc:	Linus Torvalds <torvalds@...ux-foundation.org>,
	Linux Memory Management List <linux-mm@...ck.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Hugh Dickins <hugh@...itas.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-ia64@...r.kernel.org, rth@...ddle.net,
	Jaya Kumar <jayakumar.lkml@...il.com>, krh@...hat.com,
	stefanr@...6.in-berlin.de, rolandd@...co.com,
	mshefty@...ips.intel.com, hal.rosenstock@...il.com,
	avi@...ranet.com, mchehab@...radead.org, dgilbert@...erlog.com,
	Greg Kroah-Hartman <greg@...ah.com>,
	Takashi Iwai <tiwai@...e.de>, perex@...ex.cz,
	Karsten Wiese <annabellesgarden@...oo.de>
Subject: Re: [rfc][patch] remove nopage

On Sun, Nov 11, 2007 at 09:04:29PM -0500, Jeff Garzik wrote:
> Nick Piggin wrote:
> >Index: linux-2.6/sound/oss/via82cxxx_audio.c
> >===================================================================
> >--- linux-2.6.orig/sound/oss/via82cxxx_audio.c
> >+++ linux-2.6/sound/oss/via82cxxx_audio.c
> >@@ -2099,8 +2099,7 @@ static void via_dsp_cleanup (struct via_
> > }
> > 
> > 
> >-static struct page * via_mm_nopage (struct vm_area_struct * vma,
> >-				    unsigned long address, int *type)
> >+static int via_mm_fault (struct vm_area_struct *vma, struct vm_fault *vmf)
> > {
> > 	struct via_info *card = vma->vm_private_data;
> > 	struct via_channel *chan = &card->ch_out;
> >@@ -2108,22 +2107,14 @@ static struct page * via_mm_nopage (stru
> > 	unsigned long pgoff;
> > 	int rd, wr;
> > 
> >-	DPRINTK ("ENTER, start %lXh, ofs %lXh, pgoff %ld, addr %lXh\n",
> >-		 vma->vm_start,
> >-		 address - vma->vm_start,
> >-		 (address - vma->vm_start) >> PAGE_SHIFT,
> >-		 address);
> >-
> >-        if (address > vma->vm_end) {
> >-		DPRINTK ("EXIT, returning NOPAGE_SIGBUS\n");
> >-		return NOPAGE_SIGBUS; /* Disallow mremap */
> >-	}
> >+	DPRINTK ("ENTER, pgoff %ld\n", vmf->pgoff);
> >+
> >         if (!card) {
> >-		DPRINTK ("EXIT, returning NOPAGE_SIGBUS\n");
> >-		return NOPAGE_SIGBUS;	/* Nothing allocated */
> >+		DPRINTK ("EXIT, returning VM_FAULT_SIGBUS\n");
> >+		return VM_FAULT_SIGBUS;	/* Nothing allocated */
> > 	}
> > 
> >-	pgoff = vma->vm_pgoff + ((address - vma->vm_start) >> PAGE_SHIFT);
> >+	pgoff = vmf->pgoff;
> > 	rd = card->ch_in.is_mapped;
> > 	wr = card->ch_out.is_mapped;
> > 
> >@@ -2150,9 +2141,8 @@ static struct page * via_mm_nopage (stru
> > 	DPRINTK ("EXIT, returning page %p for cpuaddr %lXh\n",
> > 		 dmapage, (unsigned long) chan->pgtbl[pgoff].cpuaddr);
> > 	get_page (dmapage);
> >-	if (type)
> >-		*type = VM_FAULT_MINOR;
> >-	return dmapage;
> >+	vmf->page = dmapage;
> >+	return 0;
> > }
> 
> 
> Although the overall concept looks nice (ACK, good work), the above 
> change does not build.  The code continues to reference via_mm_nopage(), 
> which has been renamed to via_mm_fault() in your patch.

Ah dang, it was just the .nopage in the same file that I missed.
Fixed. Thanks.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/