[PATCH] update sf_buf and uio for n64

Jayachandran C. c.jayachandran at gmail.com
Wed Jan 26 11:52:59 UTC 2011


On Tue, Jan 25, 2011 at 11:10 PM, Warner Losh <imp at bsdimp.com> wrote:
> On 01/25/2011 06:48, Jayachandran C. wrote:
>>
>> This is one of the remaining pieces in n64 work from Juli's octeon
>> branch. The attached patch updates the sf_buf code and uio_machdep.c
>> in n64 compilation to use direct mapping.
>>
>> Planning to check this in later this week, if there are no objections.
>>
>> JC.
>
> I really like the uio_machdep.c.  I think it is good and can go in without
> further objection.
>
> I like the idea of the n64 direct mapping of the sbufs.  That's a good
> optimization.  I get nervous when I see some code implemented in one place
> on one ifdef branch, and in a completely different file for another.  Is the
> gain in efficiency sufficient to justify this odd split?  If not, then
> having all the code in vm_machdep.c might make more sense.  If the gain is
> enough, then having a comment in vm_machdep.c pointing to sf_buf.h would be
> good to have as well... (that's assuming the simplifications suggested by
> Andrew Duane don't change the code distribution).

Not sure what you mean here, the implementation of sf_buf_kva() and
sf_buf_page() are in sf_buf.h, and the sf_buf_{init,alloc,free} are in
vm_machdep.c as before.

JC.


More information about the freebsd-mips mailing list