On Wed, Nov 19, 2008 at 03:07:04PM +1100, Paul Mackerras wrote:
> Sonny Rao writes:
> 
> > -   if (io_remap_pfn_range(vma, vma->vm_start, dev->bsr_addr >> PAGE_SHIFT,
> > -                          size, vma->vm_page_prot))
> > +   /* check for the case of a small BSR device and map one 4k page for it*/
> > +   if (dev->bsr_len < PAGE_SIZE && size == PAGE_SIZE)
> > +           ret = remap_4k_pfn(vma, vma->vm_start, dev->bsr_addr >> 12,
> > +                              vma->vm_page_prot);
> 
> I think we should be checking that dev->bsr_len == 4096 here.
> 
> Paul.

Well, dev->bsr_len could be 4096 or 8192

-- 
Sonny Rao, LTC OzLabs, BML team
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to