On Mon, Oct 7, 2013 at 10:34 AM, Thierry Reding <thierry.reding@xxxxxxxxx> wrote: > Rework the address table code for the host1x firewall. The previous > implementation allocated a bitfield but didn't check for a valid pointer > so it could potentially crash. I don't think it could crash. The bitmaps was allocated as a 256-bit field, and the register offset gets AND'ed with 0xFF before being looked up. What am I missing? > Furthermore setting up a bitfield makes > the code more complex than it needs to be. Doesn't this perform worse than the current implementation? Going from 1 to 13 checks per write sounds less than ideal to me... _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
