[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