From owner-freebsd-stable@FreeBSD.ORG Sun Sep 6 16:12:17 2009 Return-Path: Delivered-To: stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id EEFB0106566B; Sun, 6 Sep 2009 16:12:17 +0000 (UTC) (envelope-from a.j.werven@student.utwente.nl) Received: from satellite.xs4all.nl (zoefsam.xs4all.nl [82.95.125.145]) by mx1.freebsd.org (Postfix) with ESMTP id 753238FC13; Sun, 6 Sep 2009 16:12:17 +0000 (UTC) Received: from satellite.xs4all.nl (localhost [127.0.0.1]) by satellite.xs4all.nl (8.14.3/8.14.3) with ESMTP id n86Fbq0x001618; Sun, 6 Sep 2009 17:37:52 +0200 (CEST) (envelope-from fonz@satellite.xs4all.nl) Received: (from fonz@localhost) by satellite.xs4all.nl (8.14.3/8.14.3/Submit) id n86FbqhP001617; Sun, 6 Sep 2009 17:37:52 +0200 (CEST) (envelope-from fonz) From: "A.J. \"Fonz\" van Werven" Message-Id: <200909061537.n86FbqhP001617@satellite.xs4all.nl> In-Reply-To: <20090906110238.GH47688@deviant.kiev.zoral.com.ua> To: Kostik Belousov Date: Sun, 6 Sep 2009 17:37:52 +0200 (CEST) X-Mailer: ELM [version 2.4ME+ PL124c (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="US-ASCII" Cc: Thierry Herbelot , stable@freebsd.org, jhb@freebsd.org Subject: Re: Panic in recent 7.2-Stable X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 06 Sep 2009 16:12:18 -0000 Kostik Belousov wrote: > I expect that the following patch, that is the partial merge of r194459, > would fix it. It patches sys/vm/vm_phys.c. > > Index: vm_phys.c > =================================================================== > --- vm_phys.c (revision 194458) > +++ vm_phys.c (revision 194459) > @@ -382,8 +382,7 @@ > if (pa >= seg->start && pa < seg->end) > return (&seg->first_page[atop(pa - seg->start)]); > } > - panic("vm_phys_paddr_to_vm_page: paddr %#jx is not in any segment", > - (uintmax_t)pa); > + return (NULL); > } > > /* Hi, A quick grep on the file in question revealed that there are two functions that may panic() with "page not in any segment": the vm_phys_paddr_to_vm_page() being patched and also the next function vm_phys_paddr_to_segind(). I'm not exactly current with the memory management code so this may be a very stupid question, but I'll ask it anyway: don't both functions need to be patched? My apologies if I'm way off the mark here, but I'm just trying to help. Regards, Alphons -- All right, that does it Bill [Donahue]. I'm pretty sure that killing Jesus is not very Christian. -- Pope Benedict XVI, Southpark season 11 episode 5