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

2010-03-23 Thread Matthias Drochner
[reposting, was sent to s-c first] Module Name:src Committed By: dyoung Date: Mon Mar 22 22:30:58 UTC 2010 Modified Files: src/sys/dev/isa: isadma.c pcdisplay.c Log Message: [...] pcdisplay.c: #if 0 some code that compares two bus_space_tag_t's in order to

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

2010-03-23 Thread Matthias Drochner
dyo...@pobox.com said: How many pcdisplay at isa can there be one machine? Iirc, this code is originating from alpha where you can have multiple independant PCI/ISA/EISA hierarchies and thus multiple pcdisplay/ega/vga devices. Can the two tags that pcdisplay_is_console() compared ever come

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

2010-03-23 Thread Matthias Drochner
dyo...@pobox.com said: How many pcdisplay at isa can there be one machine? Iirc, this code is originating from alpha where you can have multiple independant PCI/ISA/EISA hierarchies and thus multiple pcdisplay/ega/vga devices. Can the two tags that pcdisplay_is_console() compared ever come

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

2010-03-23 Thread David Young
On Tue, Mar 23, 2010 at 11:39:54AM +0100, Matthias Drochner wrote: dyo...@pobox.com said: Can the two tags that pcdisplay_is_console() compared ever come from different spaces? Yes, if there are independant ISA buses they are distinguished by their tags. Good to know. I did not know

Re: CVS commit: src/sys

2010-03-23 Thread David Holland
On Sat, Mar 20, 2010 at 11:31:31PM +, Chuck Silvers wrote: fix copy{in,out}{,str}() to return the error returned by uvm_fault(). fixes PR 41813. Do you know if/how this will affect PR 11904? What happens now on EIO in a memory-mapped file? -- David A. Holland dholl...@netbsd.org

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

2010-03-23 Thread Matthias Drochner
dyo...@pobox.com said: Let us add a bus_space_is_equal(bus_space_tag_t t1, bus_space_tag_t t2) for implementation in MD code That's generally OK. (We have this in pthread_equal().) return memcpy(t1, t2, sizeof(t1)) == 0; That will work even if bus_space_tag_t is a struct. I hope you