On Friday 21 February 2014 16:11:08 Thomas Petazzoni wrote:
> On Fri, 21 Feb 2014 16:05:16 +0100, Arnd Bergmann wrote:
>
> > > *) I don't know if the algorithm to split the BAR into multiple
> > > windows is going to be trivial.
> >
> > The easiest solution would be to special case 'size is between
> > 128MB+1 and 192MB' if that turns out to be the most interesting
> > case. It's easy enough to make the second window smaller than 64MB
> > if we want.
> >
> > If we want things to be a little fancier, we could use:
> >
> > switch (size) {
> > case (SZ_32M+1) ... (SZ_32M+SZ_16M):
> > size2 = size - SZ_32M;
> > size -= SZ_32M;
> > break;
> > case (SZ_64M+1) ... (SZ_64M+SZ_32M):
> > size2 = size - SZ_64M;
> > size -= SZ_64M;
> > break;
> > case (SZ_128M+1) ... (SZ_128M+SZ_64M):
> > size2 = size - SZ_128M;
> > size -= SZ_128M;
> > break;
> > };
>
> What if the size of your BAR is 128 MB + 64 MB + 32 MB ? Then you need
> three windows, and your algorithm doesn't work
I was hoping we could avoid using more than two windows.
With the algorithm above we would round up to 256MB and
fail if that doesn't fit, which is the same thing that
happens when you run out of space.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
