Re: CVS commit: src
On Thu, Feb 24, 2011 at 04:45:30PM +0100, Joerg Sonnenberger wrote: pretty much unchanged. Feel free to perform benchmarks if you want. There are also changes to EST in that time window... This must be a bad joke. The things I did were just simple refactoring. As you should be well aware of, with est(4) and powernow(4) the frequencies are touched only by an explicit request from the system administrator (or possibly, estd(1)). Nothing like this could have happened from my changes. - Jukka. PS. Besides, I think Paul runs his tests on AMD.
Re: CVS commit: src/sys/net80211
On 02/25/11 11:45, Matthias Scheler wrote: Module Name:src Committed By: tron Date: Fri Feb 25 10:45:57 UTC 2011 Modified Files: src/sys/net80211: ieee80211_radiotap.h Log Message: Remove duplicate definitions which break the build. Thanks. I was about testing a fix but your release build finished earlier than mine. Christoph
Re: CVS commit: src/sys/arch
On 02/24/11 12:02, Jukka Ruohonen wrote: On Thu, Feb 24, 2011 at 10:56:04AM +, Jukka Ruohonen wrote: Log Message: Move PowerNow! to the cpufeaturebus. I tried to be very careful, but please test for any fall over. acpicpu(4) work on both hp laptops. powernow only tries to attach on my K8 machine where initialization fails as before = no regression. Christoph
Re: CVS commit: src
On Fri Feb 25 2011 at 12:36:07 +0200, Jukka Ruohonen wrote: On Thu, Feb 24, 2011 at 04:45:30PM +0100, Joerg Sonnenberger wrote: pretty much unchanged. Feel free to perform benchmarks if you want. There are also changes to EST in that time window... This must be a bad joke. The things I did were just simple refactoring. As you should be well aware of, with est(4) and powernow(4) the frequencies are touched only by an explicit request from the system administrator (or possibly, estd(1)). Nothing like this could have happened from my changes. Don't worry, it is. Andreas performed a run with a more exact timestamp, and everyone can see for themselves: http://www.gson.org/netbsd/bugs/build/commits-2011.02.html Scroll down to 2004.02.24. Our commit guidelines say that changes which cause unhandled test regressions will be backed out, so I want to see the issue investigated and addressed instead of indifference and excuses. -- älä karot toivorikkauttas, kyl rätei ja lumpui piisaa
Re: CVS commit: src
On Thu, Feb 24, 2011 at 04:28:50AM +, Joerg Sonnenberger wrote: Module Name: src Committed By: joerg Date: Thu Feb 24 04:28:49 UTC 2011 Modified Files: src/sys/arch/amd64/include: mcontext.h types.h This part breaks building textproc/icu: c++ -I/usr/include -D_REENTRANT -I. -I../common-DU_I18N_IMPLEMENTATION -O2 -I/usr/include -W -Wall -ansi -pedantic -Wpointer-arith -Wwrite-strings -Wno-long-long -fvisibility=hidden -c -DPIC -fPIC -o fmtable.o fmtable.cpp In file included from /usr/include/machine/signal.h:49, from /usr/include/sys/siginfo.h:35, from /usr/include/sys/signal.h:114, from /usr/include/signal.h:45, from decContext.h:61, from decContext.c:25: /usr/include/machine/mcontext.h:87: error: expected '=', ',', ';', 'asm' or '__a ttribute__' before 'void' Thomas
Re: CVS commit: src
On Fri Feb 25 2011 at 15:19:30 +0100, Joerg Sonnenberger wrote: I get time outs for stress_long and stress_short (rump/rumpkern/t_sp). Those are because of, from what I could tell, this: 0xbbbd45c5 fcntl+21: mov%gs:0x0,%edi == segfault Notably, those tests exercise threads and processes more heavily than any other test we have: they create 256 concurrent racing threads over several processes which are killed and recreated. I see a number of failing rumphijack test cases (t_vfs cpcopy, mv_nox, mv_x, paxcopy). Those I didn't investigate because they appear amd64-only. They are new tests, so there might be something else wrong with them too, but I can't see anything else immediately suspicious in the commit window after which they started crashing. -- älä karot toivorikkauttas, kyl rätei ja lumpui piisaa
Re: CVS commit: src
... I see a failing test for SSP (lib/libc/ssp/t_ssp raw), which is just a broken test case. ... I have been meaning to look into this and figure out what's wrong with this test. It was simply imported from the old test framework without a lot of scrutiny. I hope to get this resolved over the weekend. - | 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 | -
Re: CVS commit: src/sys/net80211
On Fri, Feb 25, 2011 at 08:01:50AM +, Christoph Egger wrote: Module Name: src Committed By: cegger Date: Fri Feb 25 08:01:49 UTC 2011 Modified Files: src/sys/net80211: ieee80211_radiotap.h Log Message: sync with FreeBSD rev 1.11. No binary changes. Careful: NetBSD follows the standard, http://www.radiotap.net/defined-fields, and field 18 has not been standardized. Please back out this change (and all of the subsequent changes) and send a patch for review to tech-net. Dave -- David Young OJC Technologies dyo...@ojctech.com Urbana, IL * (217) 344-0444 x24
Re: CVS commit: src/sys/dev/acpi
On Fri, Feb 25, 2011 at 07:55:07PM +, Jukka Ruohonen wrote: Log Message: Start to derive the percpu(9) (or per-domain) state coordination mechanisms by parsing the _CSD, _PSD, and _TSD objects by default. This is quite interesting development affecting the whole x86; the sleep and performance states are no longer necessarily shared by all CPUs in the system. Currently we do a xcall(9) and write the MSRs to all CPUs. But new x86 CPUs can do percpu(9) -- or rather per-core or per CPU domain -- coordination. Note that these are not software constructs or BIOS constructs; as usual, ACPI just supplies the data for us. While I need to study this more, the basic scenario is: set 1 = { CPU0, CPU1 } and set 2 = { CPU2, CPU3 }. Write the MSR for CPU0 - the whole set 1 transforms. This is also important due to the recent TurboBoost and AMD's equivalent; The set 1 enters C3 - the set 2 can enter high-performance P-state. Are there any plans for kernel equivalent to cpuset(3)? - Jukka.
Re: CVS commit: src
On Fri, Feb 25, 2011 at 04:38:54PM +0200, Antti Kantee wrote: On Fri Feb 25 2011 at 15:19:30 +0100, Joerg Sonnenberger wrote: I get time outs for stress_long and stress_short (rump/rumpkern/t_sp). Those are because of, from what I could tell, this: 0xbbbd45c5 fcntl+21: mov%gs:0x0,%edi == segfault Notably, those tests exercise threads and processes more heavily than any other test we have: they create 256 concurrent racing threads over several processes which are killed and recreated. Are you sure about that? libpthread should be the only thing (before the back out) that touches %gs and the kernel is normally using %fs for CPU local memory. Joerg
re: CVS commit: src/usr.sbin/pcictl
Module Name: src Committed By: jmcneill Date: Fri Feb 25 13:50:18 UTC 2011 Modified Files: src/usr.sbin/pcictl: pcictl.8 Log Message: Today I learned that 'pcictl pci0 list -b -1' lists devices on all busses. isn't this already documented? PCI bus specified on the command line. Any other locator not specified defaults to a wildcard, or may be explicitly wildcarded by specifying ``any''. i agree having an explicit example is better, but could you make it use any instead of -1? thanks. .mrg.
Re: CVS commit: src/sys/net80211
On 25.02.11 17:15, David Young wrote: On Fri, Feb 25, 2011 at 08:01:50AM +, Christoph Egger wrote: Module Name: src Committed By:cegger Date:Fri Feb 25 08:01:49 UTC 2011 Modified Files: src/sys/net80211: ieee80211_radiotap.h Log Message: sync with FreeBSD rev 1.11. No binary changes. Careful: NetBSD follows the standard, http://www.radiotap.net/defined-fields, and field 18 has not been standardized. Please back out this change (and all of the subsequent changes) and send a patch for review to tech-net. Do you mean that bit field or the whole revision ? Christoph