> 2015-02-19 18:26, Michal Jastrzebski: > > From: Maciej Gajdzica <maciejx.t.gajdzica at intel.com> > > > > Changed MAC address type from uint8_t[6] to struct ether_addr and IP > > address type from uint8_t[4] to uint32_t. Also removed union from > > arp_hdr struct. Updated test-pmd to match new arp_hdr version. > > > > Signed-off-by: Maciej Gajdzica <maciejx.t.gajdzica at intel.com> > > Why? > "Changed A to B" is not a sufficient explanation.
Hi Thomas I changed commit message to this: Changed MAC address type from uint8_t[6] to struct ether_addr and IP address type from uint8_t[4] to uint32_t to make it consistent with other DPDK code using MAC and IP addresses. It allows us to use is_same_ether_addr and ether_addr_copy functions on MAC addresses in ARP header. Also removed union from arp_hdr struct to make calls to arp_data items shorter. Updated test-pmd to match new arp_hdr version. Is that sufficient? Best regards, Maciek -------------------------------------------------------------- Intel Shannon Limited Registered in Ireland Registered Office: Collinstown Industrial Park, Leixlip, County Kildare Registered Number: 308263 Business address: Dromore House, East Park, Shannon, Co. Clare This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.