Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch



Unfortunately, the patch breaks building of a (non-XEN) GENERIC kernel.

Please see attached log...


On Thu, 12 Jun 2014, Cherry G. Mathew wrote:

"Paul" == Paul Goyette <paul%whooppee.com@localhost> writes:

   >> Module Name: src Committed By: cherry Date: Tue May 6 04:26:24
   >> UTC 2014
   >>
   >> Modified Files: src/sys/arch/x86/x86: pmap.c
   >> src/sys/arch/xen/include: xenpmap.h src/sys/arch/xen/x86:
   >> x86_xpmap.c
   >>
   >> Log Message: Use the hypervisor to copy/zero pages. This saves us
   >> the extra overheads of setting up temporary kernel
   >> mapping/unmapping.
   >>
   >> riz@ reports savings of about 2s on a 120s kernel build.

   Paul> This commit seems to have broken the ability to boot NetBSD
   Paul> under at least some hypervisors.  My VPS at prgmr.com boots
   Paul> fine with a kernel from just before this change.  However,
   Paul> with this commit, the following panic occurs:

Sorry, the first patch was incorrect ( it skips over some newer legit
versions! ) - thanks abs@.

Thanks to Gary Duzan for the initial suggestion and abs@ for catching
the error in my first patch.

--
Cherry



!DSPAM:53994eb3297321877012835!


-------------------------------------------------------------------------
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:       |
| Customer Service | FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com    |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at juniper.net |
| Kernel Developer |                          | pgoyette at netbsd.org  |
-------------------------------------------------------------------------

Attachment: release.amd64.log
Description: Binary data



Home | Main Index | Thread Index | Old Index