[Hardy-xen] SRU: Fix OMG how did this ever work (32bit)

Andy Whitcroft apw at canonical.com
Wed Jun 22 08:35:57 UTC 2011


On Tue, Jun 21, 2011 at 05:55:36PM +0200, Stefan Bader wrote:

> Fix: This is a bit more than needed to fix the bug itself, but it will bring our
> code more into a shape that resembles upstream (factually there is only a 2.6.18
> upstream but that code did not do the double list access).

The first patch looks fine, the second patch is just enormous.  I am not
against making the code match upstream better as that vastly improves its
maintainability going forward.  So perhaps we could split the second one
out into one "Reorder the code to match upstream order -- no functional
change" which I can then 'ignore' and one which fixes the actual bug.

Obviously even with the larger change we are going to need to test,
test, test but I believe you have this in hand and have a test case.

-apw




More information about the kernel-team mailing list