On Mon, Aug 08, 2016 at 12:12:53PM -0500, Ryan Moats wrote: > "dev" <[email protected]> wrote on 08/08/2016 11:14:21 AM: > > > From: Ben Pfaff <[email protected]> > > To: [email protected] > > Cc: Ben Pfaff <[email protected]> > > Date: 08/08/2016 11:16 AM > > Subject: [ovs-dev] [PATCH v2 10/21] lflow: Correct register > > definitions to use subfields for overlaps. > > Sent by: "dev" <[email protected]> > > > > OVN expressions need to know what fields overlap or alias one another. > > This is supposed to be done via subfields: if two fields overlap, then > the > > smaller one should be defined as a subfield of the larger one. For > > example, reg0 should be defined as xxreg0[96..127]. The symbol table in > > lflow didn't do this, so it's possible for confusion to result. (I don't > > have evidence of this actually happening, because it would only occur > > in a case where the same bits of a field were referred to with different > > names.) > > > > This commit fixes the problem. It deserves a test, but that's somewhat > > difficult at this point, so it will actually happen in a future commit. > > > > Signed-off-by: Ben Pfaff <[email protected]> > > --- > > Yes, we do really need a test for this - add it to the pile of debt... > > Acked-by: Ryan Moats <[email protected]>
Oh, the test is added one or two commits later, so it's only a very short-term debt. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
