On Tuesday 08 December 2009 09:19:27 David Howells wrote:
> Jie Zhang <jie.zh...@analog.com> wrote:
> > I agree on using 0 instead of NULL. But for !MMU, I think, vaddr is
> > always as same as addr. So we don't need to pass it?
> 
> FRV flushes the vaddr because in MMU mode the cache flush instructions take
> virtual addresses, so if we pass addr as vaddr, I can use the same cache
>  flush code for both modes.  I suspect it makes little difference to the
>  amount of code if we pass that rather than 0, as the value is already
>  computed, and either way, it's going to take one instruction to set up the
>  argument.
> 
> Note that Blackfin assumes that it may use the dst address for flushing -
>  an assumption that isn't valid in MMU mode with a VIVT cache (which I
>  presume Blackfin isn't, but other CPUs are).

the Blackfin cpu currently does not have virtual memory support at all, so i 
imagine there's a bunch of things that'll need updating when that day comes

the new patch is of course fine; thanks
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev

Reply via email to