Re: CVS commit: src

2009-10-15 Thread Jean-Yves Migeon
to drain the per-CPU caches before draining the global cache. - pool_cache_invalidate_local(): remove. Thanks! -- Jean-Yves Migeon jeanyves.mig...@free.fr

CVS commit: src/sys/arch/amd64/amd64

2009-12-30 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Thu Dec 31 01:11:28 UTC 2009 Modified Files: src/sys/arch/amd64/amd64: machdep.c Log Message: Use banner() instead of amd64's custom use of printf's for copyright notice and total/available memory. To generate a diff of this commit

CVS commit: src/sys/arch/x86/x86

2010-01-10 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Sun Jan 10 12:10:23 UTC 2010 Modified Files: src/sys/arch/x86/x86: pmap.c Log Message: Simplify pmap_init_tmp_pgtbl() a bit. The first level of the temporary page mappings are also done in the first iteration of the loop below, so n

CVS commit: src/sys/arch

2010-02-09 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Feb 9 22:51:14 UTC 2010 Modified Files: src/sys/arch/i386/include: pmap.h src/sys/arch/x86/include: pmap.h src/sys/arch/x86/x86: pmap.c Log Message: Fix typos in comments. To generate a diff of this commit: cvs

CVS commit: src/sys/arch/i386/i386

2010-02-09 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Feb 9 23:09:47 UTC 2010 Modified Files: src/sys/arch/i386/i386: mptramp.S Log Message: Use CR0_PE (enable protected mode) instead of hardcoding constant. To generate a diff of this commit: cvs rdiff -u -r1.19 -r1.20 src/sys/ar

CVS commit: src/sys/arch/x86/x86

2010-02-09 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Feb 9 23:47:10 UTC 2010 Modified Files: src/sys/arch/x86/x86: cpu.c Log Message: Use roundup2() instead of hardcoding the operation. To generate a diff of this commit: cvs rdiff -u -r1.66 -r1.67 src/sys/arch/x86/x86/cpu.c Ple

CVS commit: src/sys/arch/x86/x86

2010-02-09 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Feb 9 23:52:14 UTC 2010 Modified Files: src/sys/arch/x86/x86: cpu.c x86_machdep.c Log Message: Wrap a comment; add a space after a comma to another (align with next line) To generate a diff of this commit: cvs rdiff -u -r1.67

CVS commit: src/sys/arch/x86/x86

2010-02-09 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Wed Feb 10 00:39:31 UTC 2010 Modified Files: src/sys/arch/x86/x86: pmap.c Log Message: To properly account for the total number of pages allocated for PDP, use PDP_SIZE, as PAE (i386) requires 4 pages instead of 1. To generate a di

CVS commit: src/sys/arch

2010-02-11 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Fri Feb 12 01:55:46 UTC 2010 Modified Files: src/sys/arch/x86/x86: pmap.c src/sys/arch/xen/x86: x86_xpmap.c xen_bus_dma.c xenfunc.c Log Message: Starting with Xen 3 API, MMU_EXTENDED_COMMAND (tlb flush, cache flush, page pinn

CVS commit: src/sys/arch/x86/x86

2010-02-15 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Feb 16 00:48:17 UTC 2010 Modified Files: src/sys/arch/x86/x86: pmap.c Log Message: - re-factor code in pmap_map_ptes() slightly, and make it PAE-ready for native i386 by using PDP_SIZE - introduce pmap_unmap_apdp(), used to clea

CVS commit: src/sys/arch/i386/i386

2010-02-22 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Feb 22 23:52:17 UTC 2010 Modified Files: src/sys/arch/i386/i386: genassym.cf Log Message: Remove a duplicate entry for PTE_BASE. While here, introduce PDP_SIZE and PDE_SIZE. They will be used by asm code to make it more generic

CVS commit: src/sys/arch/i386/i386

2010-02-22 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Feb 22 23:52:17 UTC 2010 Modified Files: src/sys/arch/i386/i386: genassym.cf Log Message: Remove a duplicate entry for PTE_BASE. While here, introduce PDP_SIZE and PDE_SIZE. They will be used by asm code to make it more generic

CVS commit: src/sys

2010-02-23 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Wed Feb 24 00:01:12 UTC 2010 Modified Files: src/sys/dev/pci: agp.c src/sys/uvm: uvm_page.c Log Message: - Use ctob() instead of ptoa() to obtain physical addresses from frame numbers. Using ptoa() will cast to vaddr_t, which

CVS commit: src/sys/uvm

2010-02-25 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Thu Feb 25 23:10:49 UTC 2010 Modified Files: src/sys/uvm: uvm_glue.c Log Message: Change RSS (resident set size) limit. Instead of setting it arbitrarily to the total free memory available to the system, use the smallest value betwee

CVS commit: src/sys/uvm

2010-02-25 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Thu Feb 25 23:10:49 UTC 2010 Modified Files: src/sys/uvm: uvm_glue.c Log Message: Change RSS (resident set size) limit. Instead of setting it arbitrarily to the total free memory available to the system, use the smallest value betwee

CVS commit: src/sys/kern

2010-02-26 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Fri Feb 26 18:47:13 UTC 2010 Modified Files: src/sys/kern: kern_proc.c Log Message: Change RSS (resident set size) limit. Instead of setting it arbitrarily to the total free memory available to the system, use the smallest value betw

CVS commit: src/sys/kern

2010-02-26 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Fri Feb 26 18:47:13 UTC 2010 Modified Files: src/sys/kern: kern_proc.c Log Message: Change RSS (resident set size) limit. Instead of setting it arbitrarily to the total free memory available to the system, use the smallest value betw

CVS commit: src/sys/arch

2010-02-26 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Fri Feb 26 19:25:07 UTC 2010 Modified Files: src/sys/arch/amd64/include: pte.h src/sys/arch/i386/i386: dumpsys.c src/sys/arch/i386/include: pte.h src/sys/arch/i386/pci: elan520.c src/sys/arch/x86/x86: b

CVS commit: src/sys/arch

2010-02-26 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Fri Feb 26 19:25:07 UTC 2010 Modified Files: src/sys/arch/amd64/include: pte.h src/sys/arch/i386/i386: dumpsys.c src/sys/arch/i386/include: pte.h src/sys/arch/i386/pci: elan520.c src/sys/arch/x86/x86: b

CVS commit: src/sys/arch/xen/x86

2010-02-27 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Sat Feb 27 09:22:40 UTC 2010 Modified Files: src/sys/arch/xen/x86: xen_bus_dma.c Log Message: Make a XENMEM_decrease_reservation DEBUG printf() more meaningful. To generate a diff of this commit: cvs rdiff -u -r1.17 -r1.18 src/sys/

CVS commit: src/sys/arch/xen/x86

2010-02-27 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Sat Feb 27 09:22:40 UTC 2010 Modified Files: src/sys/arch/xen/x86: xen_bus_dma.c Log Message: Make a XENMEM_decrease_reservation DEBUG printf() more meaningful. To generate a diff of this commit: cvs rdiff -u -r1.17 -r1.18 src/sys/

CVS commit: src/sys/arch/i386/include

2010-02-28 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Mar 1 00:55:33 UTC 2010 Modified Files: src/sys/arch/i386/include: pmap.h Log Message: Use PDP_SIZE for NTOPLEVEL_PDES (number of top level PDEs) instead of #ifdef'ing PAE. To generate a diff of this commit: cvs rdiff -u -r1.1

CVS commit: src/sys/arch/i386/include

2010-02-28 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Mar 1 00:55:33 UTC 2010 Modified Files: src/sys/arch/i386/include: pmap.h Log Message: Use PDP_SIZE for NTOPLEVEL_PDES (number of top level PDEs) instead of #ifdef'ing PAE. To generate a diff of this commit: cvs rdiff -u -r1.1

CVS commit: src/sys/arch/i386

2010-02-28 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Mar 1 01:15:24 UTC 2010 Modified Files: src/sys/arch/i386/i386: machdep.c rbus_machdep.c src/sys/arch/i386/include: rbus_machdep.h Log Message: Change rbus_min_start_hint() semantic for i386. "ram" is now psize_t (instea

CVS commit: src/sys/arch/i386

2010-02-28 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Mar 1 01:15:24 UTC 2010 Modified Files: src/sys/arch/i386/i386: machdep.c rbus_machdep.c src/sys/arch/i386/include: rbus_machdep.h Log Message: Change rbus_min_start_hint() semantic for i386. "ram" is now psize_t (instea

CVS commit: src/sys/arch

2010-02-28 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Mar 1 01:35:11 UTC 2010 Modified Files: src/sys/arch/amd64/amd64: machdep.c src/sys/arch/i386/i386: machdep.c Log Message: Do not forget that ptoa() casts the result to vaddr_t, which is bad for paddr_t values under i386

CVS commit: src/sys/arch

2010-02-28 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Mon Mar 1 01:35:11 UTC 2010 Modified Files: src/sys/arch/amd64/amd64: machdep.c src/sys/arch/i386/i386: machdep.c Log Message: Do not forget that ptoa() casts the result to vaddr_t, which is bad for paddr_t values under i386

CVS commit: src/sys/arch/xen/x86

2010-03-01 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Mar 2 00:13:50 UTC 2010 Modified Files: src/sys/arch/xen/x86: xen_bus_dma.c Log Message: Catch the return value from the XENMEM_decrease_reservation hypercall, and not some error value stored earlier. While here, fix a typo in

CVS commit: src/sys/arch/xen/x86

2010-03-01 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Mar 2 00:13:50 UTC 2010 Modified Files: src/sys/arch/xen/x86: xen_bus_dma.c Log Message: Catch the return value from the XENMEM_decrease_reservation hypercall, and not some error value stored earlier. While here, fix a typo in

CVS commit: src/sys/arch/xen/x86

2010-03-02 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Wed Mar 3 00:09:03 UTC 2010 Modified Files: src/sys/arch/xen/x86: cpu.c Log Message: Use roundup2() instead of hardcoding the CACHE_LINE_SIZE rounding operation. To generate a diff of this commit: cvs rdiff -u -r1.41 -r1.42 src/sy

CVS commit: src/sys/arch/xen/x86

2010-03-02 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Wed Mar 3 00:09:03 UTC 2010 Modified Files: src/sys/arch/xen/x86: cpu.c Log Message: Use roundup2() instead of hardcoding the CACHE_LINE_SIZE rounding operation. To generate a diff of this commit: cvs rdiff -u -r1.41 -r1.42 src/sy

CVS commit: src/sys/arch/xen/x86

2010-03-09 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Mar 9 23:12:06 UTC 2010 Modified Files: src/sys/arch/xen/x86: xen_bus_dma.c Log Message: Although Xen's documentation states that the address_bits field is not used by XENMEM_decrease_reservation, it is checked by the hypervisor

CVS commit: src/sys/arch/xen/x86

2010-03-09 Thread Jean-Yves Migeon
Module Name:src Committed By: jym Date: Tue Mar 9 23:12:06 UTC 2010 Modified Files: src/sys/arch/xen/x86: xen_bus_dma.c Log Message: Although Xen's documentation states that the address_bits field is not used by XENMEM_decrease_reservation, it is checked by the hypervisor

Re: CVS commit: src/sys

2010-04-18 Thread Jean-Yves Migeon
that could be labeled as security-sensitive (not exploitable directly though). Bumping revision makes it easier to say "-current under rev 5.99.x is affected, 5.99.x+1 is not". -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-07-24 Thread Jean-Yves Migeon
(pae_pl*_pi, pae_pd_entry_t, and so on). It becomes a matter of calling the proper code within kvm(3), by checking if PAE was enabled within the kernel dump (through kvm_nlist, for example). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-07-24 Thread Jean-Yves Migeon
Besides, I suspect that turning paddr_t to uint64_t will add overhead, even if the upper 32 bits are always 0 in GENERIC. > Pleaes fix the amd64 build error reported on current-us...@. > The build error is related to rump. Investigating. rumptest just finished for i386 and amd64, and no er

Re: CVS commit: src/sys/arch

2010-07-24 Thread Jean-Yves Migeon
hout breaking ABI, needs more thought that I currently could put in my patch. Cheers, -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-07-24 Thread Jean-Yves Migeon
idth, build.sh runs have no more than 3% overhead with PAE. So, hmm, between one specific measurement and real world use... there is quite a gap. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-07-24 Thread Jean-Yves Migeon
ght set > of modules is an incoming feature but i have no time line for it. > > i plan for it to be useful for ppc oea vs. 4xx, sparc32 vs sparc64 > in 32 bitmode, and there was someone else wanting it too. IMHO, it makes sense for these platforms. But here, i386 and PAE are so close in concept that it feels like a quirk (at least to me). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-07-24 Thread Jean-Yves Migeon
fixed with "pae_" (~ some kind of namespace), and alias the relevant function with them in _KERNEL when option PAE is enabled. For kvm(3), both could be used; I could extract a value from a symbol like "pae_enabled" out of a core file through kvm_nlist, then use the relevant vatop functions. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-08-16 Thread Jean-Yves Migeon
r with the booted kernel, not very practical...). Hence, the sysctl machdep. > (why the extern in cpu.h? i doesn't seem necessary.) Yes; I wanted to place it at the same level as i386_use_fxsave. i386_use_pae may get use elsewhere eventually, so I added it to cpu.h. -- Jean-Yves Migeon jeanyves.mig...@free.fr

re: CVS commit: src/sys/arch

2010-08-16 Thread Jean-Yves Migeon
(u_long). - i386 will use 64 bits paddr_t for kvm(3), so we will have to discuss on how to make it work without too much fuzz with pre-64 bits i386 dumps. - sparse dumps for memory above 4GB. Eventually, PAE and !PAE kernels core files should be handled by kvm, if that's what you are asking. PAE does not affect program core files. -- Jean-Yves Migeon jeanyves.mig...@free.fr

re: CVS commit: src/sys/arch

2010-08-17 Thread Jean-Yves Migeon
t message is not very clear: the " Will be used by i386 kvm(3) to know the functions that should [...]" sentence applies to i386_use_pae, not machdep.pae. It is only there for convenience, eg. when someone wants to know easily if its system is running under PAE. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-08-17 Thread Jean-Yves Migeon
ernel paddr_t will not automagically solve the "all PAs are unsigned long" assumption of kvm(3). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2010-08-17 Thread Jean-Yves Migeon
recisely the "problem" here :) > while maintaining binary compatibility. I don't think so; modules that manipulates 32 bits bus_addr_t will likely fail with PAE when you cross the 4GB boundary. I don't see how the union could solve that. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys

2010-09-27 Thread Jean-Yves Migeon
r_t being 32 bits. > > Thanks! -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src

2010-10-06 Thread Jean-Yves Migeon
On 06.10.2010 12:16, Manuel Bouyer wrote: > On Tue, Oct 05, 2010 at 11:48:17PM +0000, Jean-Yves Migeon wrote: >> [...] >> >> XXX Currently, savecore(8) will fail to dump a PAE kernel in a !PAE >> environment (and reciprocally). So you need to sync and reboot >&

Re: CVS commit: src/sys/dev/pci

2010-10-10 Thread Jean-Yves Migeon
anks; I was about to commit the fix. Second time I got burned on that one :/ -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/dev/mii

2010-11-27 Thread Jean-Yves Migeon
On 27.11.2010 18:38, Jean-Yves Migeon wrote: > Module Name: src > Committed By: jym > Date: Sat Nov 27 17:38:49 UTC 2010 > > Modified Files: > src/sys/dev/mii: miidevs.h > > Log Message: > Correct string for BCM6709S. s/BCM6709S/BCM5709S/ -- Jean-Yves Migeon j...@netbsd.org

Re: CVS commit: src/etc/powerd/scripts

2010-12-31 Thread Jean-Yves Migeon
in the right direction IMHO, will it be replaced by a MI interface to put a system into sleep, as it is not a feature specific to ACPI? -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/etc/powerd/scripts

2010-12-31 Thread Jean-Yves Migeon
On 31.12.2010 11:10, Jukka Ruohonen wrote: > On Fri, Dec 31, 2010 at 11:01:08AM +0100, Jean-Yves Migeon wrote: >> I am using machdep.sleep_state as node to put a domU into suspend mode. >> Up to now, putting sleep_state under machdep allowed powerd(8) >> sleep_button to be u

Re: CVS commit: src/etc/powerd/scripts

2010-12-31 Thread Jean-Yves Migeon
ed uses pmf(9)). > FWIW, the sleep states that ACPI names are not sufficient even to > describe all of the potential sleep states of ACPI hardware. I have a > laptop that's perfectly capable of an "S3-like" sleep, but the ACPI BIOS > doesn't support S3, and the HDD

Re: CVS commit: src/sys/arch/i386/conf

2011-02-10 Thread Jean-Yves Migeon
On 10.02.2011 22:23, David Laight wrote: > On Thu, Feb 10, 2011 at 04:49:19PM +0000, Jean-Yves Migeon wrote: >> Module Name: src >> Committed By:jym >> Date:Thu Feb 10 16:49:19 UTC 2011 >> >> Modified Files: >> src/sys/arch/i386/

Re: CVS commit: src/sys/arch/i386/conf

2011-02-10 Thread Jean-Yves Migeon
s where kernel gets replaced without its associated modules. IIRC, this is why MONOLITHIC was brought back, due to the number of complaints where GENERIC fails booting because ffs.kmod or exec_elf32.kmod could not be loaded. This is what amd64 is approx. doing, but with most file-systems included by default. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/i386/conf

2011-02-11 Thread Jean-Yves Migeon
On 11.02.2011 07:30, Matthias Scheler wrote: > On Fri, Feb 11, 2011 at 05:06:43AM +0100, Jean-Yves Migeon wrote: >> Indeed, it would be good to have at least some exec formats and >> file-systems builtin by default in GENERIC: >> >> EXEC_ELF32, _SCRIPT # obvious >

Re: CVS commit: src/sys/arch/i386/conf

2011-02-13 Thread Jean-Yves Migeon
On 13.02.2011 14:42, David Laight wrote: > On Sun, Feb 13, 2011 at 04:37:21AM +0000, Jean-Yves Migeon wrote: >> Module Name: src Committed By: jym Date: Sun Feb 13 >> 04:37:21 UTC >> 2011 >> >> Modified Files: src/sys/arch/i386/conf: GENERI

Re: CVS commit: src/sys/arch/i386/conf

2011-02-13 Thread Jean-Yves Migeon
On 13.02.2011 17:02, Paul Goyette wrote: > On Sun, 13 Feb 2011, Jean-Yves Migeon wrote: > >> ... >> For order of preference, see module(7): >> >>The loader will look first for a built-in module with the specified >>name that has not been disabl

Re: CVS commit: src/sys/arch/amd64/conf

2011-02-19 Thread Jean-Yves Migeon
On 19.02.2011 10:27, Bernd Ernesti wrote: > On Wed, Feb 16, 2011 at 03:16:58AM +0000, Jean-Yves Migeon wrote: >> Module Name: src >> Committed By:jym >> Date:Wed Feb 16 03:16:58 UTC 2011 >> >> Modified Files: >> src/sys/arc

Re: CVS commit: src/sys/arch/amd64/conf

2011-02-19 Thread Jean-Yves Migeon
LITHIC (but still having modules as possibility), and that did not seem to bother people. So either way, it's fine; but please -- i386 and amd64 should share common grounds. If some want a MONOLITHIC amd64, it's even possible, although I can't see the point given the arguments ab

Re: CVS commit: src/sys/arch/amd64/conf

2011-02-20 Thread Jean-Yves Migeon
kernel). Needs to be xchecked though. Note that /stand also includes modules that cannot be included inside a monolithic kernel, like zfs and solaris compat layer. And these makes up for ~ half the size of /stand (5MiB). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/amd64/conf

2011-02-20 Thread Jean-Yves Migeon
pport anymore). So, one more time: the current issue is to define what people consider as options that should be enabled by default, and what could stay out (or as a third party module if you urgently need it back). One example: accf_* is a questionable choice, whether its inside GENERIC (as a builtin), or enabled by default in MONOLITHIC. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/amd64/conf

2011-02-21 Thread Jean-Yves Migeon
of a MONOLITHIC kernel irrelevant. > So, one more time: the current issue is to define what people consider > as options that should be enabled by default, and what could stay out > (or as a third party module if you urgently need it back). One example: > accf_* is a questionable choice, whether its inside GENERIC (as a > builtin), or enabled by default in MONOLITHIC. I don't think that's the issue. Running GENERIC is supposed to make all stable features and functionality available. The question at hand is whether those features are compiled in (which would be "monolithic") or loaded as modules (which would be "modular"). Exactly. [1] http://en.wikipedia.org/wiki/Voting_paradox -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/i386/conf

2011-02-23 Thread Jean-Yves Migeon
ce not only "what to load", but also, "what is the state" of a driver module. Module's loading can change the state of devices, and rebooting/calling bootloader will _not_ reset that state. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen/xen

2011-04-04 Thread Jean-Yves Migeon
On 04.04.2011 23:21, Taylor R Campbell wrote: >Date: Sun, 3 Apr 2011 23:21:39 + >From: "Jean-Yves Migeon" > >Now that pkgsrc-2011Q1 has arrived, and before -6 chimes in, change >ifxname for xvif(4) from xvif%d.%d to xvif%d-%d. This is needed >

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Jean-Yves Migeon
On Wed, 6 Apr 2011 16:29:22 +, Taylor R Campbell wrote: Date: Mon, 04 Apr 2011 23:46:19 +0200 From: Jean-Yves Migeon The newer scripts for Xen read the interface value from the vifname entry in Xenstore, so changing the name is not that critical. But this should be

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Jean-Yves Migeon
On 06.04.2011 20:01, Manuel Bouyer wrote: > You could also use > xvifxiy (e.g. xvif5i2, where i stands for 'index'). > or any other letter ... Huh hmm indeed... I wonder why I did not think about this approach before... -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
mentioned in the thread above, ie; Uho, I forgot to mention in my commit log that I fixed it. I am allocating bpg_entries via pool_cache(9), and the constructor bpge_ctor() will return an error if uvm(9) fails to find a free page. In that case, the thread will just bail out and start waiting again. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
ied for all available memoryallocators(9) especially as the code behind can evolve (hey, Lars :) ). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
xed - we are no longer checking for big ones as well. > > This applies to all allocators. So, any thread sleeping for an allocation cannot be interrupted? -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen/xen

2011-05-15 Thread Jean-Yves Migeon
change, but forgot about the possible concurrency regarding refcnt fetch (not actually possible as port-xen is not MP, but will become soonish) I'll remove the volatile declaration too, only xbdi_put/_get use the refcnt for G/C anyway. Thanks for pointing that out! -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/usr.bin/pmap

2011-06-25 Thread Jean-Yves Migeon
eck (any executable that has kmem sgid rights is a target), and there are other potential tools usable out there (lsof(1), maybe?). Isn't it something that rather fits the kauth(9) ACLs? -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/crypto/external/bsd/openssl/lib/libcrypto/arch/x86_64

2011-07-17 Thread Jean-Yves Migeon
he .byte streams are required for the inclusion of the AES NI instructions, which are not supported with our current gcc version. Should be fixed once we have stabilized gcc 4.5 (dunno about other compilers though, especially pcc). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/crypto/external/bsd/openssl/lib/libcrypto/arch/x86_64

2011-07-17 Thread Jean-Yves Migeon
d by the Perl scripts in openssl (byte streams and the resulting ugliness are neither my own nor spz@). I tend to steer away from manipulating code (particularly crypto) when I don't have good knowledge of it. And this is far from being the case for me with OpenSSL. Anyway, I'll look into it next week for cleanup. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2011-07-17 Thread Jean-Yves Migeon
e to ANSI style), I had to do it manually. It's even the opposite, spatch has issues when parsing non-ANSI declarations, so you have to do the conversion all by yourself first... -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2011-07-18 Thread Jean-Yves Migeon
On Mon, 18 Jul 2011 07:41:30 +0100, David Laight wrote: On Mon, Jul 18, 2011 at 02:18:54AM +0200, Jean-Yves Migeon wrote: On 18.07.2011 02:00, David Young wrote: >> Can we please use ansi function definitions in newly committed code? > > This was tedious enough without conver

Re: CVS commit: src/sys

2011-07-30 Thread Jean-Yves Migeon
rful" lockless algorithms are, except in situation where the additional bus locking/atomic ops involved did not really improve the situation in highly concurrent systems (and could even make it worse). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen

2011-08-21 Thread Jean-Yves Migeon
e amount of cycles to complete, like TLB flushes). I'd expect all Xen MMU hypercalls to be reentrant. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen

2011-08-21 Thread Jean-Yves Migeon
On 21.08.2011 12:26, Jean-Yves Migeon wrote: > - second, the lock is not placed at the correct abstraction level IMHO, > it is way too high in the caller/callee hierarchy. It should remain > hidden from most consumers of the xpq_queue API, and should only be used > to protect the xpq

Re: CVS commit: src/sys/arch/xen

2011-08-21 Thread Jean-Yves Migeon
e; shadow page tables were mainly designed with Linux in mind, and the way it manages virtual memory is completely different to ours: it maps the entire physical memory in the kernel virtual space (with tricks when there's more than 1GiB involved), while we use recursive mappings. And Xen has problems validating these. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen

2011-08-22 Thread Jean-Yves Migeon
ow page tables you can't use a CAS to assure atomic operation with the hardware TLB, as this is, precisely, a shadow PT and not the one used by hardware. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/usermode/usermode

2011-08-27 Thread Jean-Yves Migeon
orward to add to common/, and I am more at peace knowing that we have a "valid" strnlen() in kernel rather than a bogus macro that may spread elsewhere... -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/usermode/usermode

2011-08-27 Thread Jean-Yves Migeon
On 27.08.2011 20:28, Joerg Sonnenberger wrote: > On Sat, Aug 27, 2011 at 08:13:28PM +0200, Jean-Yves Migeon wrote: >> On 27.08.2011 19:57, Reinoud Zandijk wrote: >>> Fix copystring routines to NOT just copy all since not all space might be >>> writable. This can be fixe

Re: CVS commit: src/share/man/man4

2011-08-29 Thread Jean-Yves Migeon
On 29.08.2011 06:30, Christoph Egger wrote: > On 29.08.11 06:26, Christoph Egger wrote: >> On 29.08.11 00:09, Jean-Yves Migeon wrote: >>> Module Name:src >>> Committed By: jym >>> Date: Sun Aug 28 22:09:37 UTC 2011 >>> >&g

Re: CVS commit: src/share/man/man4

2011-08-29 Thread Jean-Yves Migeon
On Mon, 29 Aug 2011 11:46:06 +0200, Manuel Bouyer wrote: On Mon, Aug 29, 2011 at 09:01:47AM +0200, Jean-Yves Migeon wrote: What kind of console is attaching for you in dom0? I can't see how '+' would get wired in by default. At least when either started from bare

Re: CVS commit: src/share/man/man4

2011-08-29 Thread Jean-Yves Migeon
On Mon, 29 Aug 2011 15:25:03 +0200, Manuel Bouyer wrote: On Mon, Aug 29, 2011 at 12:45:27PM +0100, Jean-Yves Migeon wrote: Hmm, I'll have to cross-check that one this afternoon. IIRC, I am also using the the default "break" command when I am running the dom0 inside QEMU, and

Re: CVS commit: src/sys/arch/xen

2011-08-29 Thread Jean-Yves Migeon
#x27;t put a giant lock around xpq_queue. This doesn't make any sense in a MP system, especially for operations that are frequently called and still may take a while to complete. Just imagine: all CPUs waiting for one to finish its TLB flush before they can edit their PD/PT again... ouch. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/share/man/man4

2011-08-29 Thread Jean-Yves Migeon
On 29.08.2011 22:27, Manuel Bouyer wrote: > On Mon, Aug 29, 2011 at 03:17:16PM +0100, Jean-Yves Migeon wrote: >> Both; usually I am using the VGA-emulated display, but sometimes >> (like I did lately) I switch to console, so I can keep a reasonable >> amount of history of

Re: CVS commit: [cherry-xenmp] src/sys/arch

2011-08-30 Thread Jean-Yves Migeon
t; > Log Message: > Add per-cpu mmu queues Thanks for looking into it! -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/etc

2011-09-06 Thread Jean-Yves Migeon
On 06.09.2011 12:54, Thomas Klausner wrote: > On Mon, Aug 22, 2011 at 06:54:07PM +0000, Jean-Yves Migeon wrote: >> Module Name: src >> Committed By:jym >> Date:Mon Aug 22 18:54:06 UTC 2011 >> >> [snip] >> >> Log Message: >>

Re: CVS commit: src/etc

2011-09-06 Thread Jean-Yves Migeon
y default DESTDIR), so this went unnoticed during my tests as the correct /etc/defaults gets parsed. I'll add the required code to handle this; thanks for reporting! -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2011-09-20 Thread Jean-Yves Migeon
On 20.09.2011 02:12, Jean-Yves Migeon wrote: Module Name:src Committed By: jym Date: Tue Sep 20 00:12:25 UTC 2011 Modified Files: [snip] Log Message: Merge jym-xensuspend branch in -current. ok bouyer@. Goal: save/restore support in NetBSD domUs, for i386, i386 PAE and amd64

Re: CVS commit: src/sys/arch

2011-09-20 Thread Jean-Yves Migeon
ried to circumvent the invalidation limitation (eg. not handling per-CPU pools, like x86 pmap_create "try_again" loop). -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src

2011-09-29 Thread Jean-Yves Migeon
means that the lock will block other LWPs from running even when they do not need to allocate memory. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/kern

2011-10-24 Thread Jean-Yves Migeon
based on a context; the old code does not permit that, unless you build and store the string somewhere, forcing the caller to *know* that it only keeps a pointer and does not copy the content. This will get misused, sooner or later. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2011-11-07 Thread Jean-Yves Migeon
rpa[2]) | PG_V; > +#endif /* PAE */ - are you sure that these have to be "defined(PAE) || defined(__x86_64__)" ? - please use "for (i = 0; i < PTP_LEVELS - 1; i++ ) { ... }". I will have to identify these blocks later when GENERIC will include both PAE and !PAE pmaps. PTP_LEVELS makes it a lot easier. That's all for the first quick review :) Thanks for starting the merge of your branch. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2011-11-08 Thread Jean-Yves Migeon
S makes it a lot easier. ok, will watchout for it - do you want to do this one yourself ? Please do :) -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2011-11-08 Thread Jean-Yves Migeon
On 08.11.2011 14:53, Cherry G. Mathew wrote: On 8 November 2011 15:20, Jean-Yves Migeon wrote: On Tue, 8 Nov 2011 08:49:22 +0530, Cherry G. Mathew wrote: Please keep ci_pae_l3_pdir to a uint32_t and back out its paddr_t type. Unless you can convince me that your code will do the right thing

Re: CVS commit: src/sys/arch

2011-11-09 Thread Jean-Yves Migeon
etter. I always wondered whether i386_use_pae should be set for amd64. Strictly speaking, PAE is enabled when we are running in long mode. I set the sysctl(7) a while ago for NX regression tests in ATF, but not the variable. Maybe I should. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/lib/libutil

2011-11-17 Thread Jean-Yves Migeon
a different rule applicable to man pages for 4-clause vs 2-clause BSD? I occasionally see new man pages written with a 4-clause BSD, however, most newly written code is 2-clause. -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch

2011-11-20 Thread Jean-Yves Migeon
ps to reproduce: - connect to a dom0 system - try ballooning up or down, with: sysctl -w machdep.xen.balloon.target=10 All newly created processes will then stay in tstile, and the sysctl never returns, waiting on "rplq" wait channel. Observed in QEMU, Virtualbox, and my amd64 spare host

  1   2   >