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 ord

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 f

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 f

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 k

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 hop