Re: CVS commit: [thorpej-i2c-spi-conf] src/sys/dev/i2c
I wonder if a summary report of detected RAM might be worthwhile, e.g., “4 RAM slots, 2 RAM DIMMs detected” - it’d be better still if the kernel could report the relationship between RAM present and memory controller channels (e.g., does it appear that RAM is spread across them for interleaving?). just a thought while the code is being tilled, Erik > On May 16, 2021, at 14:07, Jason Thorpe wrote: > >> >> On May 16, 2021, at 1:06 PM, matthew green wrote: >> >>> Modified Files: >>> src/sys/dev/i2c [thorpej-i2c-spi-conf]: spdmem_i2c.c >>> >>> Log Message: >>> The last change had an unfortunate side-effect on empty DIMM slots, so >>> roll that back. Instead, if we used direct config, then probe for the >>> module in the attach routine and report if the module is not present, >>> rather than assuming that it is. >> >> this seems odd to me. >> >> if we're using direct config, shouldn't match() be fully >> able to determine present or not, avoiding having to have >> an attach() that fake-fails? > > The problem is if a device tree has a node for a SPD DIMM, but the DIMM > *isn’t actually present* (i.e. the node is for “DIMM might live here, go > check!”). Returning “no match” in this situation would result in a “… not > configured” message for a non-existent DIMM. > >> >> thanks. >> >> >> .mrg. > > -- thorpej
Re: CVS commit: src/sys
Please pull up to netbsd-8 and netbsd-9 release branches. > On Nov 11, 2020, at 23:44, Simon Burge wrote: > > Module Name: src > Committed By: simonb > Date: Thu Nov 12 07:44:01 UTC 2020 > > Modified Files: > src/sys/conf: param.c > src/sys/kern: init_main.c > src/sys/sys: param.h > > Log Message: > Set a better default for MAXFILES on larger RAM machines if not > otherwise specified the kernel config file. Arbitary numbers are > 20,000 files for 16GB RAM or more and 10,000 files for 1GB RAM or > more. > > TODO: Adjust this and other values totally dynamically. > > > To generate a diff of this commit: > cvs rdiff -u -r1.68 -r1.69 src/sys/conf/param.c > cvs rdiff -u -r1.532 -r1.533 src/sys/kern/init_main.c > cvs rdiff -u -r1.678 -r1.679 src/sys/sys/param.h > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src/share/man/man4
These features, e.g. VLAN, Jumbo Frames, CRC offload, interrupt coalescing, should really be listed in the man page for each network interface device (and driver) that supports them (e.g. bge(4), wm(4), tlp(4), le(4)), along with any device-specific implementation bugs or caveats, and then the vlan(4) (and other) pages that describe these functions should be linked from those device driver pages. Erik Fair > On May 28, 2017, at 20:41, Ryota Ozaki wrote: > > Module Name: src > Committed By: ozaki-r > Date: Mon May 29 03:41:54 UTC 2017 > > Modified Files: > src/share/man/man4: vlan.4 > > Log Message: > Remove unmaintained (and probably unmaintainable) driver list > > ok msaitoh@ > > > To generate a diff of this commit: > cvs rdiff -u -r1.34 -r1.35 src/share/man/man4/vlan.4 > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src/lib/libc/net
So, what did we do by default: allow “_” in hostnames when that’s explicitly
against standard, or not? I read the patch in the PR and couldn’t make heads or
tails of it.
Erik
> On Oct 26, 2015, at 07:48, Christos Zoulas wrote:
>
> Module Name: src
> Committed By: christos
> Date: Mon Oct 26 14:48:04 UTC 2015
>
> Modified Files:
> src/lib/libc/net: getaddrinfo.c
>
> Log Message:
> PR/50367: Stefan Schaeckeler: libc resolver library does not resolve host
> names with underscores ("_"). According to resolv.conf(5) RES_NOCHECKNAME
> is on by default; well, it is and gethostbyname(3) obeys it
> (gethnamaddr.c:maybe_ok), but getaddrinfo(3) up till this commit not.
> XXX: pullup-7, pullup-6
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.109 -r1.110 src/lib/libc/net/getaddrinfo.c
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
Re: CVS commit: src/sys/dev/ic
Pull up to netbsd-7 and netbsd-6? Erik > On Jul 22, 2015, at 01:33, Juergen Hannken-Illjes wrote: > > Module Name: src > Committed By: hannken > Date: Wed Jul 22 08:33:51 UTC 2015 > > Modified Files: > src/sys/dev/ic: mpt_netbsd.c > > Log Message: > Adapter leaks requests when mpt_event_notify_reply() has to acknowledge > an event leading to "adapter resource shortage" messages when the scsipi > subsystem tries to use all adapt_openings. > > Change mpt_ctlop() to free the request on event MPI_FUNCTION_EVENT_ACK. > > Tested on a SunFire X4275 with Symbios Logic SAS1068E (1000:0058, rev. 4). > > > To generate a diff of this commit: > cvs rdiff -u -r1.31 -r1.32 src/sys/dev/ic/mpt_netbsd.c > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: [netbsd-7] src/distrib/atari/floppies/common
Why doesn’t adding “-m68020=60” to CPUFLAGS work? curious, Erik Fair > On Mar 29, 2015, at 00:44, Soren Jacobsen wrote: > > Module Name: src > Committed By: snj > Date: Sun Mar 29 07:44:46 UTC 2015 > > Modified Files: > src/distrib/atari/floppies/common [netbsd-7]: Makefile.images > > Log Message: > Pull up following revision(s) (requested by martin in ticket #652): > distrib/atari/floppies/common/Makefile.images: revision 1.9 > Use "-Os -m68020-60" for DBG. It seems to generate smaller objects than -Os. > gcc48 with "-Os": > -rwxr-xr-x 1 tsutsui wheel 1319596 Nov 16 20:50 obj.atari/instbin > gcc48 with "-Os -m68020-60" > -rwxr-xr-x 1 tsutsui wheel 1314516 Nov 16 20:49 obj.atari/instbin > This allows ever growing sysinst.fs still fit into 1440KB even with gcc48. > Acually we need a real solution (ustarfs based floppies etc.) soon > but we can work around at least for NetBSD 7.0. > Should be pulled up to netbsd-7 (if NetBSD/m68k 7.0 will switch to gcc48). > > > To generate a diff of this commit: > cvs rdiff -u -r1.8 -r1.8.26.1 \ >src/distrib/atari/floppies/common/Makefile.images > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: [netbsd-6-0] src/sys/dev/pci
On Nov 29, 2014, at 03:38, Martin Husemann wrote: > Module Name: src > Committed By: martin > Date: Sat Nov 29 11:38:46 UTC 2014 > > Modified Files: > src/sys/dev/pci [netbsd-6-0]: if_bge.c > > Log Message: > Pullup revision 1.260 (via patch), requested by msaitoh in #1200: > > Set maximum read byte count to 2048 for PCI-X BCM5703/5704 devices. > For PCI-X BCM5704, set maximum outstanding split transactions to 0. > This change fixes a serious stability problem on those chips. Why no pull-up to netbsd-6? curious, Erik
Re: CVS commit: [netbsd-6-0] src/sys
It appears that these changes were applied to "netbsd-6-0" and "netbsd-6-1" but not to "netbsd-6" (which is a problem for those of us following "netbsd-6"). Erik On Jun 18, 2014, at 02:35 , SAITOH Masanobu wrote: > Module Name: src > Committed By: msaitoh > Date: Wed Jun 18 09:35:40 UTC 2014 > > Modified Files: > src/sys/dist/ipf/netinet [netbsd-6-0]: ip_fil_netbsd.c > src/sys/net [netbsd-6-0]: if_ethersubr.c if_loop.c if_vlan.c > src/sys/netinet [netbsd-6-0]: if_arp.c ip_carp.c > src/sys/netinet6 [netbsd-6-0]: ip6_flow.c nd6.c > src/sys/rump/librump/rumpkern [netbsd-6-0]: klock.c > > Log Message: > Pull up following revision(s) (requested by bouyer in ticket #1067): > sys/dist/ipf/netinet/ip_fil_netbsd.c1.9 via patch > sys/net/if_ethersubr.c 1.197 via patch > sys/net/if_loop.c 1.77 via patch > sys/net/if_vlan.c 1.70 via patch > sys/netinet/if_arp.c1.158 > sys/netinet/ip_carp.c 1.54 via patch > sys/netinet6/ip6_flow.c 1.23 via patch > sys/netinet6/nd6.c 1.150 via patch > sys/rump/librump/rumpkern/klock.c 1.4 > > Make sure *(if_output)() is called with KERNEL_LOCK held to avoid mbuf leak. > See http://mail-index.netbsd.org/tech-net/2014/04/09/msg004511.html > for details. For netinet6, the problem report, fix and test were done > by njoly@ on current-users@ > > > To generate a diff of this commit: > cvs rdiff -u -r1.61.6.1 -r1.61.6.2 src/sys/dist/ipf/netinet/ip_fil_netbsd.c > cvs rdiff -u -r1.188.8.2 -r1.188.8.2.4.1 src/sys/net/if_ethersubr.c > cvs rdiff -u -r1.75 -r1.75.14.1 src/sys/net/if_loop.c > cvs rdiff -u -r1.69 -r1.69.14.1 src/sys/net/if_vlan.c > cvs rdiff -u -r1.154 -r1.154.8.1 src/sys/netinet/if_arp.c > cvs rdiff -u -r1.47.4.1 -r1.47.4.1.4.1 src/sys/netinet/ip_carp.c > cvs rdiff -u -r1.19 -r1.19.8.1 src/sys/netinet6/ip6_flow.c > cvs rdiff -u -r1.141.6.2 -r1.141.6.3 src/sys/netinet6/nd6.c > cvs rdiff -u -r1.3 -r1.3.18.1 src/sys/rump/librump/rumpkern/klock.c > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src
Can this driver be pulled up to netbsd-6? Erik On Jun 9, 2014, at 18:42 , Hikaru Abe wrote: > Module Name: src > Committed By: hikaru > Date: Tue Jun 10 01:42:39 UTC 2014 > > Modified Files: > src/distrib/sets/lists/man: mi > src/share/man/man4/man4.x86: Makefile > src/sys/arch/amd64/conf: ALL GENERIC > src/sys/arch/i386/conf: ALL GENERIC > src/sys/arch/x86/pci: files.pci > Added Files: > src/share/man/man4/man4.x86: vmx.4 > src/sys/arch/x86/pci: if_vmx.c if_vmxreg.h > > Log Message: > Add VMware VMXNET3 ethernet driver from OpenBSD, vmx(4). > > > To generate a diff of this commit: > cvs rdiff -u -r1.1474 -r1.1475 src/distrib/sets/lists/man/mi > cvs rdiff -u -r1.14 -r1.15 src/share/man/man4/man4.x86/Makefile > cvs rdiff -u -r0 -r1.1 src/share/man/man4/man4.x86/vmx.4 > cvs rdiff -u -r1.8 -r1.9 src/sys/arch/amd64/conf/ALL > cvs rdiff -u -r1.387 -r1.388 src/sys/arch/amd64/conf/GENERIC > cvs rdiff -u -r1.375 -r1.376 src/sys/arch/i386/conf/ALL > cvs rdiff -u -r1.1104 -r1.1105 src/sys/arch/i386/conf/GENERIC > cvs rdiff -u -r1.15 -r1.16 src/sys/arch/x86/pci/files.pci > cvs rdiff -u -r0 -r1.1 src/sys/arch/x86/pci/if_vmx.c \ >src/sys/arch/x86/pci/if_vmxreg.h > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src/sys/netinet
This should be pulled up to netbsd-6 On Apr 12, 2014, at 05:24, Greg Troxel wrote: > Module Name: src > Committed By: gdt > Date: Sat Apr 12 12:24:50 UTC 2014 > > Modified Files: > src/sys/netinet: if_arp.c > > Log Message: > revarprequest: Avoid leaking mbuf. > > In revarprequest, an mbuf could perhaps be leaked in an error path. > My reading of the code is that this is not possible, because ar_pro is > set to ETHERNET_IP, and ar_tha can only be null in the 1394 case. > But, better to have the free call anyway; ar_tha does not have a > documented interface contract :-) > > Pointed out by Maxime Villard. > > > To generate a diff of this commit: > cvs rdiff -u -r1.155 -r1.156 src/sys/netinet/if_arp.c > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src/etc
On Jan 8, 2014, at 10:25 , Greg Troxel wrote: > Why do you think the configured servers should be given query > permission? Is that a sense of courtesy to the pool operators that they > should be able to run "ntpdc -c monlist" and "ntpq -p" at machines that > are syncing from them? Yes, that's polite. It is suggested by the NTP peering guidelines (and I know "server" and "peer" are different). Erik
Re: CVS commit: src/etc
Unless I misunderstand NTP configuration semantics, your additional "restrict" statements for the NTP pool names will do the wrong thing, in that each reference to a given netbsd.pool.ntp.org name returns multiple IP addresses, in apparently random order, i.e. an attempt to guarantee no two queries return the same data. Ergo, those restrict statements will most probably not end up with the same IP address as their preceding "server" statements, as was presumably your intent. Erik
Re: CVS commit: [netbsd-5] src/doc
Do any of these changes apply to netbsd-6 also? curious, Erik
Re: CVS commit: src/sys/dev/scsipi
On Aug 9, 2013, at 12:58 , Frank Kardel wrote: > Module Name: src > Committed By: kardel > Date: Fri Aug 9 19:58:44 UTC 2013 > > Modified Files: > src/sys/dev/scsipi: ch.c > > Log Message: > bump command timeout to 5 minutes. several > types of changers (Overland PowerLoader, Dell > PowerVault) have been exceeding the 100 sec > limit aborting a perfectly (slowly) progressing > operation. Upon further reflection, I believe that this timeout value should be a device-specific tunable parameter because there is such wide variation in changer behavior/performance; perhaps by kernel config, or by sysctl(8) on a per-device-node basis. Or some other mechanism you prefer. I believe that baking it into the kernel as a constant is not good design because we'll just see another commit just like this one at some later date, continuing to patch around the design problem. Erik
Re: CVS commit: src/sys/dev/scsipi
On Aug 9, 2013, at 12:58 , Frank Kardel wrote: > Module Name: src > Committed By: kardel > Date: Fri Aug 9 19:58:44 UTC 2013 > > Modified Files: > src/sys/dev/scsipi: ch.c > > Log Message: > bump command timeout to 5 minutes. several > types of changers (Overland PowerLoader, Dell > PowerVault) have been exceeding the 100 sec > limit aborting a perfectly (slowly) progressing > operation. I think the kernel should uprintf(9) a notice to the effect that it has exceeded some (sooner than the 5 minute timeout) threshold and that it's really waiting for the device. Five minutes is a very long time for a timeout involving nominally local I/O devices. Without some progress indicator, users are likely to begin beating the system up (power cycle, etc) absent some persuasion to be patient. Erik
Re: CVS import: src/external/bsd/dhcpcd/dist
On Jul 19, 2013, at 04:52, "Roy Marples" wrote: > Module Name: src > Committed By: roy > Date: Fri Jul 19 11:52:58 UTC 2013 > > Update of /cvsroot/src/external/bsd/dhcpcd/dist > In directory ivanova.netbsd.org:/tmp/cvs-serv22216 > > Log Message: > Import dhcpcd-6.0.3 with the following changes: > * dhcpcd will now assign a short hostname by default > To use a FQDN hostname, set this in dhcpcd.conf(5) > env hostname_fqdn=YES This is the wrong default, too - hostname should always be FQDN. If you want the short form, throw away the extra information - don't force all other programs to work to get it. Erik
Re: CVS commit: src/bin/hostname
On Jul 19, 2013, at 03:34, "Roy Marples" wrote: > Module Name: src > Committed By: roy > Date: Fri Jul 19 10:34:51 UTC 2013 > > Modified Files: > src/bin/hostname: hostname.1 hostname.c > > Log Message: > Add the following options > -A Display the FQDN of each address on all interfaces. > -a Display alias name(s) of the host. > -d Display the DNS domain. > -f Display the FQDN for the hostname. > -I Display each IP address on all interfaces. > -i Display the IP address(es) for the hostname. > Not to go all Rob Pike on you (cf. "cat -v considered harmful"), but what the heck is all this for? The system's hostname is supposed to be the FQDN, not the short form (Sun got this wrong), and what the hell is hostname doing groveling around in network interfaces? Or talking to the DNS? hostname(1) has one job: set/get the system hostname. Does some other (*cough* Linux) system do these other things that we maybe might need to be … "compatible" with it for scripts? curious, Erik
Re: CVS commit: src/share/zoneinfo
Pullup request to netbsd-6 and netbsd-5? Erik
Re: CVS commit: src/sys/dev/pci
It would be very helpful to document in bge(4) just which chips support jumbo frames (and how big). Heck, it would be even more helpful if that was a capability that ifconfig(8) could report because drivers declare it. Erik On Jul 2, 2013, at 22:49 , SAITOH Masanobu wrote: > Module Name: src > Committed By: msaitoh > Date: Wed Jul 3 05:49:36 UTC 2013 > > Modified Files: > src/sys/dev/pci: if_bge.c > > Log Message: > Add BGE_JUMBO_CAPABLE flag to some chips. Before this commit, > the following chips support the jumbo frame function: > > 5700 > 5701 > 5702 > 5703 > 5704 > > With this commit, the following chip support it, too: > > 5714 > 5780 > 5717 > 5718 > 5719 (exclude rev. A0) > 5720 > 57765 > 57766 > > > To generate a diff of this commit: > cvs rdiff -u -r1.253 -r1.254 src/sys/dev/pci/if_bge.c > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: [netbsd-6-0] src/external/bsd/bind/dist
On Dec 13, 2012, at 09:42, Jeff Rizzo wrote: > Module Name: src > Committed By: riz > Date: Thu Dec 13 17:42:04 UTC 2012 > > Modified Files: > src/external/bsd/bind/dist [netbsd-6-0]: CHANGES version > src/external/bsd/bind/dist/bin/named [netbsd-6-0]: query.c > > Log Message: > external/bsd/bind/dist/CHANGESpatch > external/bsd/bind/dist/versionpatch > external/bsd/bind/dist/bin/named/query.c patch > > Address CVE-2012-5688: Named could die on specific queries with dns64 > enabled. > [spz, ticket #740] Is there some reason why this change was not also pulled up to "netbsd-6"? curious, Erik
Re: CVS commit: src/sys/kern
This & the immediate predecessor are definitely a netbsd-5 pull up items. Erik On Jun 3, 2012, at 09:23, David Laight wrote: > Module Name: src > Committed By: dsl > Date: Sun Jun 3 16:23:44 UTC 2012 > > Modified Files: > src/sys/kern: vfs_bio.c > > Log Message: > Use separate temporaries for the 'int' percentage and the 'long' > water marks. > Previous paniced on sparc64 due to a misaligned copy. > > > To generate a diff of this commit: > cvs rdiff -u -r1.238 -r1.239 src/sys/kern/vfs_bio.c > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src/sys/ufs/ffs
On Apr 23, 2011, at 00:36, Juergen Hannken-Illjes wrote: > Module Name: src > Committed By: hannken > Date: Sat Apr 23 07:36:02 UTC 2011 > > Modified Files: > src/sys/ufs/ffs: ffs_balloc.c > > Log Message: > Try to keep snapshot indirect blocks contiguous. > > This speeds up snapshot creation by a factor of ~3 and reduces > the file system suspension time by a factor of ~5. This definitely sounds like a pullup candidate for netbsd-5. Erik
Re: CVS commit: src/usr.bin/units
On Apr 3, 2011, at 12:44, David A. Holland wrote: > Module Name: src > Committed By: dholland > Date: Sun Apr 3 19:44:15 UTC 2011 > > Modified Files: > src/usr.bin/units: units.1 > > Log Message: > Expand user warning (adding examples) and move to CAVEATS as it's not fixable. > > > To generate a diff of this commit: > cvs rdiff -u -r1.16 -r1.17 src/usr.bin/units/units.1 > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. > Traditionally, "BUGS" was the section for that ... Erik
Re: CVS commit: src/sys/nfs
Pullup candidate? On Mar 23, 2011, at 10:42, Thor Lancelot Simon wrote: > Module Name: src > Committed By: tls > Date: Wed Mar 23 17:42:11 UTC 2011 > > Modified Files: > src/sys/nfs: nfs_socket.c > > Log Message: > As suggested by matt@: change socket buffer reservations for NFS send/receive > to 3 times max RPC size rather than 2 times. Avoids nasty TCP stalls observed > at Panix. Will require increase to sbmax via sysctl for those running really > huge NFS rsize/wsize (>64K). > > > To generate a diff of this commit: > cvs rdiff -u -r1.188 -r1.189 src/sys/nfs/nfs_socket.c > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src/sys/dev/pci
Has a pull-up request to netbsd-5 been submitted? Erik On Nov 3, 2010, at 04:46, Takeshi Nakayama wrote: > Module Name: src > Committed By: nakayama > Date: Wed Nov 3 11:46:31 UTC 2010 > > Modified Files: > src/sys/dev/pci: aceride.c > > Log Message: > Add a workaround for 48-bit LBA DMA access bug on M5229 rev. <= 0xc4. > - use DMA in 28-bit LBA addressing. > - use PIO in 48-bit LBA addressing. > > Idea and message are from FreeBSD, and tested on M5229 rev. 0xc3 in > my Sun Netra X1 with 160GB drive. > > > To generate a diff of this commit: > cvs rdiff -u -r1.27 -r1.28 src/sys/dev/pci/aceride.c > > Please note that diffs are not public domain; they are subject to the > copyright notices on the relevant files. >
Re: CVS commit: src/etc/rc.d
On Sep 8, 2009, at 01:56, Christoph Egger wrote: Module Name:src Committed By: cegger Date: Tue Sep 8 08:56:34 UTC 2009 Modified Files: src/etc/rc.d: network Log Message: Do not flush routes if root file system is nfs mounted. Fixes boot problem when the nfs server is in a different subnet. To generate a diff of this commit: cvs rdiff -u -r1.58 -r1.59 src/etc/rc.d/network Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files. This change should be pulled up to netbsd-5. Erik
Re: CVS commit: src/sys/netipsec
On Jul 30, 2009, at 07:41, Jonathan A. Kollasch wrote: Module Name:src Committed By: jakllsch Date: Thu Jul 30 14:41:59 UTC 2009 Modified Files: src/sys/netipsec: ipsec.c Log Message: As explained in kern/41701 there's a missing splx() here. To generate a diff of this commit: cvs rdiff -u -r1.45 -r1.46 src/sys/netipsec/ipsec.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files. Has a pullup request for the netbsd-5 branch been submitted? Erik
