[ovs-git] Open vSwitch: test-csum: Avoid "cast increases required alignment of target type" warning. (master)

dev at openvswitch.org dev at openvswitch.org
Thu May 26 16:05:24 UTC 2011


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, master has been updated
       via  e7b3e606b2f6054bc20518d64100ae501b3e6ef3 (commit)
       via  98dbe2dded157a753a56df30faba3f5f51dc35f7 (commit)
       via  a75777916820892493ae797e88dafd62317c00a7 (commit)
       via  ee693ba0850c5201830f22c645f6c5a06fa4180e (commit)
       via  08c4a39ef8417de8f48874296f07ad58de46eefc (commit)
      from  48e1b7fb9b2e0f6c7cf3f54eda9679ec0130e737 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit e7b3e606b2f6054bc20518d64100ae501b3e6ef3
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e7b3e606b2f6054bc20518d64100ae501b3e6ef3
Author: Ben Pfaff <blp at nicira.com>
		
test-csum: Avoid "cast increases required alignment of target type" warning.
		
Casting a character pointer to uint16_t * or uint32_t * provokes this
warning on sparc.  There is no actual problem, because all of the accesses
to data occurs through calls to the get_unaligned_*() functions, so this
commit suppresses the warning by using "void *" as an intermediate type.


commit 98dbe2dded157a753a56df30faba3f5f51dc35f7
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=98dbe2dded157a753a56df30faba3f5f51dc35f7
Author: Ben Pfaff <blp at nicira.com>
		
bridge: Avoid "cast increases required alignment of target type" warning.
		
This pointer arithmetic caused a warning on sparc.  One alternative would
be to add a cast through "void *", but it seems at least as reasonable to
just use a few macros to do direct assignments.


commit a75777916820892493ae797e88dafd62317c00a7
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a75777916820892493ae797e88dafd62317c00a7
Author: Ben Pfaff <blp at nicira.com>
		
in-band: Avoid "cast increases required alignment of target type" warning.
		
Casting a struct ofp_action_output object's address to type union
ofp_action * provokes a warning on sparc.  It's easier to just declare
the object as having type union ofp_action, even though I don't think that
there is a real problem here.


commit ee693ba0850c5201830f22c645f6c5a06fa4180e
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=ee693ba0850c5201830f22c645f6c5a06fa4180e
Author: Ben Pfaff <blp at nicira.com>
		
sha1: Avoid unused variable warnings on big-endian systems.
		


commit 08c4a39ef8417de8f48874296f07ad58de46eefc
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=08c4a39ef8417de8f48874296f07ad58de46eefc
Author: Ben Pfaff <blp at nicira.com>
		
netlink: Avoid "cast increases required alignment of target type" on RISC.
		
This warning doesn't indicate a genuine problem, since 'nla' must be
aligned properly and NLA_ALIGN ensures that the offset is aligned properly
too.  Casting via void * suppresses the warning.


-----------------------------------------------------------------------

Summary of changes:
 lib/netlink.h     |    2 +-
 lib/sha1.c        |    5 ++-
 ofproto/in-band.c |   13 +++++------
 tests/test-csum.c |    5 ++-
 vswitchd/bridge.c |   61 +++++++++++++++++++++++-----------------------------
 5 files changed, 40 insertions(+), 46 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list