On 08/06/2013 05:02 PM, Graeme Gregory wrote:
This will eventually need to be fixed in kernel but for now to allow work to progress alter mab so it will not attempt to place a table across a page boundary.
If people could give this patch some proper testing it would be good as there may be some corner cases I missed constructing this. I have boot tested the result on Arndale.
While working on this I discovered that write_table was allocating buffers which were never used and never had free() called on them.
Whups. My bad. I do have to admit I didn't worry about malloc/free pairs -- I reckoned this was an incredibly short-lived process and the chances of using > 100K were pretty slim. Nice catch.
Thanks
Graeme
Everything else looks good.
Acked-by: Al Stone al.stone@linaro.org