I've fixed the code to deal with LDI and support block (hopefully) even
block devices. The updated webrev is at the same location:
http://cr.opensolaris.org/~gdamore/schism/
I've been able to test both OSS and Sun personalities (using both
ldi_write() and ldi_putmsg()) using a small bit of kernel code I wrote
for the task. (Let me know if you want a copy.) The old code panic'd,
but the new code works flawlessly.
Its still not approved at PSARC, but if I get it approved, I'll be
filing the RTI once I get my webrevs approved.
- Garrett
Garrett D'Amore wrote:
Although the PSARC case is not yet approved for this, I've gone ahead
and tested up the changes with a custom version of the audio driver
that supports both STREAMS and character devices.) The good news is
that it all seems to work, and I can't find any problems with it. :-)
The changes to the core specfs code for this are very modest, only 18
lines of change. I'd like folks to review them. Here's the webrev:
http://cr.opensolaris.org/~gdamore/schism/
The changes to the audio framework to eliminate the STREAMS stub are
considerably larger, and will be offered up for review separately.
(I also cleaned up a bunch of legacy cruft, so the audio framework
actually sheds about 500 lines of code as part of these changes... but
again, I'll be submitting that review later.)
Please let me know what you think. Assuming that I get PSARC approval
and reviewers to verify this, then I'll probably submit the RTI on
this later this week.
- Garrett
_______________________________________________
driver-discuss mailing list
driver-disc...@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/driver-discuss
_______________________________________________
opensolaris-code mailing list
opensolaris-code@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/opensolaris-code
