firewire: core: fix upper bound of possible CSR allocations
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Fri, 23 Jul 2010 11:02:54 +0000 (13:02 +0200)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Fri, 23 Jul 2010 11:36:28 +0000 (13:36 +0200)
commit0c9ae701ae1caf657326db22d61074b40a747c9d
tree97eccc9b9941e71c471b5b3f32450c89f476093f
parentcc550216ae9a2993ef3973464714dc1a39ab1f86
firewire: core: fix upper bound of possible CSR allocations

region->end is defined as an upper bound of the requested address range,
exclusive --- i.e. as an address outside of the range in which the
requested CSR is to be placed.

Hence 0x0001,0000,0000,0000 is the biggest valid region->end, not
0x0000,ffff,ffff,fffc like the current check asserted.

For simplicity, the fix drops the region->end & 3 test because there is
no actual problem with these bits set in region->end.  The allocated
address range will be quadlet aligned and of a size of multiple quadlets
due to the checks for region->start & 3 and handler->length & 3 alone.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
drivers/firewire/core-transaction.c