홍길동 / onos

Go to a project
Toggle navigation Toggle navigation pinning
Thomas Vachuska
50c7bb71 2 parents 2ac1b82f 2befc661

Merge remote-tracking branch 'origin/master'

Expand all
Inline Side-by-side
Showing 16 changed files with 104 additions and 86 deletions
...@@ -36,7 +36,6 @@ import org.onlab.onos.net.host.InterfaceIpAddress; ...@@ -36,7 +36,6 @@ import org.onlab.onos.net.host.InterfaceIpAddress;
36 import org.onlab.onos.net.host.PortAddresses; 36 import org.onlab.onos.net.host.PortAddresses;
37 import org.onlab.packet.IpAddress; 37 import org.onlab.packet.IpAddress;
38 import org.onlab.packet.IpPrefix; 38 import org.onlab.packet.IpPrefix;
39 -import org.onlab.packet.Ip4Prefix;
40 import org.onlab.packet.MacAddress; 39 import org.onlab.packet.MacAddress;
41 import org.slf4j.Logger; 40 import org.slf4j.Logger;
42 41
...@@ -81,12 +80,8 @@ public class NetworkConfigReader { ...@@ -81,12 +80,8 @@ public class NetworkConfigReader {
81 if (splits.length != 2) { 80 if (splits.length != 2) {
82 throw new IllegalArgumentException("Invalid IP address and prefix length format"); 81 throw new IllegalArgumentException("Invalid IP address and prefix length format");
83 } 82 }
84 - // 83 + // NOTE: IpPrefix will mask-out the bits after the prefix length.
85 - // TODO: For now we need Ip4Prefix to mask-out the 84 + IpPrefix subnet = IpPrefix.valueOf(strIp);
86 - // subnet address.
87 - //
88 - Ip4Prefix subnet4 = new Ip4Prefix(strIp);
89 - IpPrefix subnet = IpPrefix.valueOf(subnet4.toString());
90 IpAddress addr = IpAddress.valueOf(splits[0]); 85 IpAddress addr = IpAddress.valueOf(splits[0]);
91 InterfaceIpAddress ia = 86 InterfaceIpAddress ia =
92 new InterfaceIpAddress(addr, subnet); 87 new InterfaceIpAddress(addr, subnet);
......
...@@ -73,7 +73,7 @@ public class RouteEntry { ...@@ -73,7 +73,7 @@ public class RouteEntry {
73 } 73 }
74 74
75 StringBuilder result = new StringBuilder(ip4Prefix.prefixLength()); 75 StringBuilder result = new StringBuilder(ip4Prefix.prefixLength());
76 - long value = ip4Prefix.toInt(); 76 + long value = ip4Prefix.address().toInt() & 0xffffffffL;
77 for (int i = 0; i < ip4Prefix.prefixLength(); i++) { 77 for (int i = 0; i < ip4Prefix.prefixLength(); i++) {
78 long mask = 1 << (IpPrefix.MAX_INET_MASK_LENGTH - 1 - i); 78 long mask = 1 << (IpPrefix.MAX_INET_MASK_LENGTH - 1 - i);
79 result.append(((value & mask) == 0) ? "0" : "1"); 79 result.append(((value & mask) == 0) ? "0" : "1");
......
...@@ -188,7 +188,7 @@ class TestBgpPeerChannelHandler extends SimpleChannelHandler { ...@@ -188,7 +188,7 @@ class TestBgpPeerChannelHandler extends SimpleChannelHandler {
188 int prefixBytelen = (prefixBitlen + 7) / 8; // Round-up 188 int prefixBytelen = (prefixBitlen + 7) / 8; // Round-up
189 message.writeByte(prefixBitlen); 189 message.writeByte(prefixBitlen);
190 190
191 - IpAddress address = prefix.toIpAddress(); 191 + IpAddress address = prefix.address();
192 long value = address.toInt() & 0xffffffffL; 192 long value = address.toInt() & 0xffffffffL;
193 for (int i = 0; i < IpAddress.INET_BYTE_LENGTH; i++) { 193 for (int i = 0; i < IpAddress.INET_BYTE_LENGTH; i++) {
194 if (prefixBytelen-- == 0) { 194 if (prefixBytelen-- == 0) {
......
...@@ -22,7 +22,7 @@ import java.util.Objects; ...@@ -22,7 +22,7 @@ import java.util.Objects;
22 import org.onlab.onos.net.PortNumber; 22 import org.onlab.onos.net.PortNumber;
23 import org.onlab.onos.net.flow.instructions.Instruction; 23 import org.onlab.onos.net.flow.instructions.Instruction;
24 import org.onlab.onos.net.flow.instructions.Instructions; 24 import org.onlab.onos.net.flow.instructions.Instructions;
25 -import org.onlab.packet.IpPrefix; 25 +import org.onlab.packet.IpAddress;
26 import org.onlab.packet.MacAddress; 26 import org.onlab.packet.MacAddress;
27 import org.onlab.packet.VlanId; 27 import org.onlab.packet.VlanId;
28 28
...@@ -181,12 +181,12 @@ public final class DefaultTrafficTreatment implements TrafficTreatment { ...@@ -181,12 +181,12 @@ public final class DefaultTrafficTreatment implements TrafficTreatment {
181 } 181 }
182 182
183 @Override 183 @Override
184 - public Builder setIpSrc(IpPrefix addr) { 184 + public Builder setIpSrc(IpAddress addr) {
185 return add(Instructions.modL3Src(addr)); 185 return add(Instructions.modL3Src(addr));
186 } 186 }
187 187
188 @Override 188 @Override
189 - public Builder setIpDst(IpPrefix addr) { 189 + public Builder setIpDst(IpAddress addr) {
190 return add(Instructions.modL3Dst(addr)); 190 return add(Instructions.modL3Dst(addr));
191 } 191 }
192 192
......
...@@ -19,7 +19,7 @@ import java.util.List; ...@@ -19,7 +19,7 @@ import java.util.List;
19 19
20 import org.onlab.onos.net.PortNumber; 20 import org.onlab.onos.net.PortNumber;
21 import org.onlab.onos.net.flow.instructions.Instruction; 21 import org.onlab.onos.net.flow.instructions.Instruction;
22 -import org.onlab.packet.IpPrefix; 22 +import org.onlab.packet.IpAddress;
23 import org.onlab.packet.MacAddress; 23 import org.onlab.packet.MacAddress;
24 import org.onlab.packet.VlanId; 24 import org.onlab.packet.VlanId;
25 25
...@@ -92,14 +92,14 @@ public interface TrafficTreatment { ...@@ -92,14 +92,14 @@ public interface TrafficTreatment {
92 * @param addr an ip 92 * @param addr an ip
93 * @return a treatment builder 93 * @return a treatment builder
94 */ 94 */
95 - public Builder setIpSrc(IpPrefix addr); 95 + public Builder setIpSrc(IpAddress addr);
96 96
97 /** 97 /**
98 * Sets the dst l3 address. 98 * Sets the dst l3 address.
99 * @param addr an ip 99 * @param addr an ip
100 * @return a treatment builder 100 * @return a treatment builder
101 */ 101 */
102 - public Builder setIpDst(IpPrefix addr); 102 + public Builder setIpDst(IpAddress addr);
103 103
104 /** 104 /**
105 * Sets the optical channel ID or lambda. 105 * Sets the optical channel ID or lambda.
......
...@@ -27,7 +27,7 @@ import org.onlab.onos.net.flow.instructions.L2ModificationInstruction.L2SubType; ...@@ -27,7 +27,7 @@ import org.onlab.onos.net.flow.instructions.L2ModificationInstruction.L2SubType;
27 import org.onlab.onos.net.flow.instructions.L2ModificationInstruction.ModEtherInstruction; 27 import org.onlab.onos.net.flow.instructions.L2ModificationInstruction.ModEtherInstruction;
28 import org.onlab.onos.net.flow.instructions.L3ModificationInstruction.L3SubType; 28 import org.onlab.onos.net.flow.instructions.L3ModificationInstruction.L3SubType;
29 import org.onlab.onos.net.flow.instructions.L3ModificationInstruction.ModIPInstruction; 29 import org.onlab.onos.net.flow.instructions.L3ModificationInstruction.ModIPInstruction;
30 -import org.onlab.packet.IpPrefix; 30 +import org.onlab.packet.IpAddress;
31 import org.onlab.packet.MacAddress; 31 import org.onlab.packet.MacAddress;
32 import org.onlab.packet.VlanId; 32 import org.onlab.packet.VlanId;
33 33
...@@ -115,7 +115,7 @@ public final class Instructions { ...@@ -115,7 +115,7 @@ public final class Instructions {
115 * @param addr the ip address to modify to. 115 * @param addr the ip address to modify to.
116 * @return a L3 modification 116 * @return a L3 modification
117 */ 117 */
118 - public static L3ModificationInstruction modL3Src(IpPrefix addr) { 118 + public static L3ModificationInstruction modL3Src(IpAddress addr) {
119 checkNotNull(addr, "Src l3 address cannot be null"); 119 checkNotNull(addr, "Src l3 address cannot be null");
120 return new ModIPInstruction(L3SubType.IP_SRC, addr); 120 return new ModIPInstruction(L3SubType.IP_SRC, addr);
121 } 121 }
...@@ -125,7 +125,7 @@ public final class Instructions { ...@@ -125,7 +125,7 @@ public final class Instructions {
125 * @param addr the ip address to modify to. 125 * @param addr the ip address to modify to.
126 * @return a L3 modification 126 * @return a L3 modification
127 */ 127 */
128 - public static L3ModificationInstruction modL3Dst(IpPrefix addr) { 128 + public static L3ModificationInstruction modL3Dst(IpAddress addr) {
129 checkNotNull(addr, "Dst l3 address cannot be null"); 129 checkNotNull(addr, "Dst l3 address cannot be null");
130 return new ModIPInstruction(L3SubType.IP_DST, addr); 130 return new ModIPInstruction(L3SubType.IP_DST, addr);
131 } 131 }
......
...@@ -19,7 +19,7 @@ import static com.google.common.base.MoreObjects.toStringHelper; ...@@ -19,7 +19,7 @@ import static com.google.common.base.MoreObjects.toStringHelper;
19 19
20 import java.util.Objects; 20 import java.util.Objects;
21 21
22 -import org.onlab.packet.IpPrefix; 22 +import org.onlab.packet.IpAddress;
23 23
24 /** 24 /**
25 * Abstraction of a single traffic treatment step. 25 * Abstraction of a single traffic treatment step.
...@@ -60,9 +60,9 @@ public abstract class L3ModificationInstruction implements Instruction { ...@@ -60,9 +60,9 @@ public abstract class L3ModificationInstruction implements Instruction {
60 public static final class ModIPInstruction extends L3ModificationInstruction { 60 public static final class ModIPInstruction extends L3ModificationInstruction {
61 61
62 private final L3SubType subtype; 62 private final L3SubType subtype;
63 - private final IpPrefix ip; 63 + private final IpAddress ip;
64 64
65 - public ModIPInstruction(L3SubType subType, IpPrefix addr) { 65 + public ModIPInstruction(L3SubType subType, IpAddress addr) {
66 66
67 this.subtype = subType; 67 this.subtype = subType;
68 this.ip = addr; 68 this.ip = addr;
...@@ -73,7 +73,7 @@ public abstract class L3ModificationInstruction implements Instruction { ...@@ -73,7 +73,7 @@ public abstract class L3ModificationInstruction implements Instruction {
73 return this.subtype; 73 return this.subtype;
74 } 74 }
75 75
76 - public IpPrefix ip() { 76 + public IpAddress ip() {
77 return this.ip; 77 return this.ip;
78 } 78 }
79 79
......
...@@ -254,8 +254,7 @@ public class DeviceManager ...@@ -254,8 +254,7 @@ public class DeviceManager
254 // } 254 // }
255 return; 255 return;
256 } 256 }
257 - MastershipTerm term = mastershipService.requestTermService() 257 + MastershipTerm term = termService.getMastershipTerm(deviceId);
258 - .getMastershipTerm(deviceId);
259 258
260 final NodeId myNodeId = clusterService.getLocalNode().id(); 259 final NodeId myNodeId = clusterService.getLocalNode().id();
261 if (!myNodeId.equals(term.master())) { 260 if (!myNodeId.equals(term.master())) {
......
...@@ -30,6 +30,9 @@ import org.onlab.onos.net.intent.IntentStore; ...@@ -30,6 +30,9 @@ import org.onlab.onos.net.intent.IntentStore;
30 import org.onlab.onos.net.intent.IntentStoreDelegate; 30 import org.onlab.onos.net.intent.IntentStoreDelegate;
31 import org.onlab.onos.store.hz.AbstractHazelcastStore; 31 import org.onlab.onos.store.hz.AbstractHazelcastStore;
32 import org.onlab.onos.store.hz.SMap; 32 import org.onlab.onos.store.hz.SMap;
33 +import org.onlab.onos.store.serializers.KryoNamespaces;
34 +import org.onlab.onos.store.serializers.KryoSerializer;
35 +import org.onlab.util.KryoNamespace;
33 import org.slf4j.Logger; 36 import org.slf4j.Logger;
34 37
35 import java.util.List; 38 import java.util.List;
...@@ -61,9 +64,21 @@ public class DistributedIntentStore ...@@ -61,9 +64,21 @@ public class DistributedIntentStore
61 @Activate 64 @Activate
62 public void activate() { 65 public void activate() {
63 // FIXME: We need a way to add serializer for intents which has been plugged-in. 66 // FIXME: We need a way to add serializer for intents which has been plugged-in.
64 - // TODO: As a short term workaround, relax Kryo config to 67 + // As a short term workaround, relax Kryo config to
65 - // registrationRequired=false? 68 + // registrationRequired=false
66 super.activate(); 69 super.activate();
70 + super.serializer = new KryoSerializer() {
71 +
72 + @Override
73 + protected void setupKryoPool() {
74 + serializerPool = KryoNamespace.newBuilder()
75 + .setRegistrationRequired(false)
76 + .register(KryoNamespaces.API)
77 + .build()
78 + .populate(1);
79 + }
80 +
81 + };
67 82
68 // TODO: enable near cache, allow read from backup for this IMap 83 // TODO: enable near cache, allow read from backup for this IMap
69 IMap<byte[], byte[]> rawIntents = super.theInstance.getMap("intents"); 84 IMap<byte[], byte[]> rawIntents = super.theInstance.getMap("intents");
......
...@@ -38,7 +38,7 @@ public final class IpPrefixSerializer extends Serializer<IpPrefix> { ...@@ -38,7 +38,7 @@ public final class IpPrefixSerializer extends Serializer<IpPrefix> {
38 @Override 38 @Override
39 public void write(Kryo kryo, Output output, 39 public void write(Kryo kryo, Output output,
40 IpPrefix object) { 40 IpPrefix object) {
41 - byte[] octs = object.toOctets(); 41 + byte[] octs = object.address().toOctets();
42 output.writeInt(octs.length); 42 output.writeInt(octs.length);
43 output.writeBytes(octs); 43 output.writeBytes(octs);
44 output.writeInt(object.prefixLength()); 44 output.writeInt(object.prefixLength());
......
...@@ -31,6 +31,7 @@ import org.onlab.onos.net.flow.FlowRule; ...@@ -31,6 +31,7 @@ import org.onlab.onos.net.flow.FlowRule;
31 import org.onlab.onos.net.flow.TrafficSelector; 31 import org.onlab.onos.net.flow.TrafficSelector;
32 import org.onlab.onos.net.flow.TrafficTreatment; 32 import org.onlab.onos.net.flow.TrafficTreatment;
33 import org.onlab.onos.openflow.controller.Dpid; 33 import org.onlab.onos.openflow.controller.Dpid;
34 +import org.onlab.packet.IpAddress;
34 import org.onlab.packet.IpPrefix; 35 import org.onlab.packet.IpPrefix;
35 import org.onlab.packet.MacAddress; 36 import org.onlab.packet.MacAddress;
36 import org.onlab.packet.VlanId; 37 import org.onlab.packet.VlanId;
...@@ -167,24 +168,12 @@ public class FlowEntryBuilder { ...@@ -167,24 +168,12 @@ public class FlowEntryBuilder {
167 case SET_NW_DST: 168 case SET_NW_DST:
168 OFActionSetNwDst nwdst = (OFActionSetNwDst) act; 169 OFActionSetNwDst nwdst = (OFActionSetNwDst) act;
169 IPv4Address di = nwdst.getNwAddr(); 170 IPv4Address di = nwdst.getNwAddr();
170 - if (di.isCidrMask()) { 171 + builder.setIpDst(IpAddress.valueOf(di.getInt()));
171 - builder.setIpDst(IpPrefix.valueOf(di.getInt(),
172 - di.asCidrMaskLength()));
173 - } else {
174 - builder.setIpDst(IpPrefix.valueOf(di.getInt(),
175 - IpPrefix.MAX_INET_MASK_LENGTH));
176 - }
177 break; 172 break;
178 case SET_NW_SRC: 173 case SET_NW_SRC:
179 OFActionSetNwSrc nwsrc = (OFActionSetNwSrc) act; 174 OFActionSetNwSrc nwsrc = (OFActionSetNwSrc) act;
180 IPv4Address si = nwsrc.getNwAddr(); 175 IPv4Address si = nwsrc.getNwAddr();
181 - if (si.isCidrMask()) { 176 + builder.setIpSrc(IpAddress.valueOf(si.getInt()));
182 - builder.setIpSrc(IpPrefix.valueOf(si.getInt(),
183 - si.asCidrMaskLength()));
184 - } else {
185 - builder.setIpSrc(IpPrefix.valueOf(si.getInt(),
186 - IpPrefix.MAX_INET_MASK_LENGTH));
187 - }
188 break; 177 break;
189 case EXPERIMENTER: 178 case EXPERIMENTER:
190 OFActionExperimenter exp = (OFActionExperimenter) act; 179 OFActionExperimenter exp = (OFActionExperimenter) act;
......
...@@ -29,6 +29,8 @@ import org.onlab.onos.net.flow.criteria.Criteria.TcpPortCriterion; ...@@ -29,6 +29,8 @@ import org.onlab.onos.net.flow.criteria.Criteria.TcpPortCriterion;
29 import org.onlab.onos.net.flow.criteria.Criteria.VlanIdCriterion; 29 import org.onlab.onos.net.flow.criteria.Criteria.VlanIdCriterion;
30 import org.onlab.onos.net.flow.criteria.Criteria.VlanPcpCriterion; 30 import org.onlab.onos.net.flow.criteria.Criteria.VlanPcpCriterion;
31 import org.onlab.onos.net.flow.criteria.Criterion; 31 import org.onlab.onos.net.flow.criteria.Criterion;
32 +import org.onlab.packet.IpAddress;
33 +import org.onlab.packet.IpPrefix;
32 import org.projectfloodlight.openflow.protocol.OFFactory; 34 import org.projectfloodlight.openflow.protocol.OFFactory;
33 import org.projectfloodlight.openflow.protocol.OFFlowAdd; 35 import org.projectfloodlight.openflow.protocol.OFFlowAdd;
34 import org.projectfloodlight.openflow.protocol.OFFlowDelete; 36 import org.projectfloodlight.openflow.protocol.OFFlowDelete;
...@@ -141,22 +143,30 @@ public abstract class FlowModBuilder { ...@@ -141,22 +143,30 @@ public abstract class FlowModBuilder {
141 break; 143 break;
142 case IPV4_DST: 144 case IPV4_DST:
143 ip = (IPCriterion) c; 145 ip = (IPCriterion) c;
144 - if (ip.ip().isMasked()) { 146 + if (ip.ip().prefixLength() != IpPrefix.MAX_INET_MASK_LENGTH) {
145 - Masked<IPv4Address> maskedIp = Masked.of(IPv4Address.of(ip.ip().toInt()), 147 + IpAddress maskAddr =
146 - IPv4Address.of(ip.ip().netmask().toInt())); 148 + IpAddress.makeMaskPrefix(ip.ip().prefixLength());
149 + Masked<IPv4Address> maskedIp =
150 + Masked.of(IPv4Address.of(ip.ip().address().toInt()),
151 + IPv4Address.of(maskAddr.toInt()));
147 mBuilder.setMasked(MatchField.IPV4_DST, maskedIp); 152 mBuilder.setMasked(MatchField.IPV4_DST, maskedIp);
148 } else { 153 } else {
149 - mBuilder.setExact(MatchField.IPV4_DST, IPv4Address.of(ip.ip().toInt())); 154 + mBuilder.setExact(MatchField.IPV4_DST,
155 + IPv4Address.of(ip.ip().address().toInt()));
150 } 156 }
151 break; 157 break;
152 case IPV4_SRC: 158 case IPV4_SRC:
153 ip = (IPCriterion) c; 159 ip = (IPCriterion) c;
154 - if (ip.ip().isMasked()) { 160 + if (ip.ip().prefixLength() != IpPrefix.MAX_INET_MASK_LENGTH) {
155 - Masked<IPv4Address> maskedIp = Masked.of(IPv4Address.of(ip.ip().toInt()), 161 + IpAddress maskAddr =
156 - IPv4Address.of(ip.ip().netmask().toInt())); 162 + IpAddress.makeMaskPrefix(ip.ip().prefixLength());
163 + Masked<IPv4Address> maskedIp =
164 + Masked.of(IPv4Address.of(ip.ip().address().toInt()),
165 + IPv4Address.of(maskAddr.toInt()));
157 mBuilder.setMasked(MatchField.IPV4_SRC, maskedIp); 166 mBuilder.setMasked(MatchField.IPV4_SRC, maskedIp);
158 } else { 167 } else {
159 - mBuilder.setExact(MatchField.IPV4_SRC, IPv4Address.of(ip.ip().toInt())); 168 + mBuilder.setExact(MatchField.IPV4_SRC,
169 + IPv4Address.of(ip.ip().address().toInt()));
160 } 170 }
161 break; 171 break;
162 case IP_PROTO: 172 case IP_PROTO:
......
...@@ -21,7 +21,8 @@ import java.util.Objects; ...@@ -21,7 +21,8 @@ import java.util.Objects;
21 import static com.google.common.base.Preconditions.checkNotNull; 21 import static com.google.common.base.Preconditions.checkNotNull;
22 22
23 /** 23 /**
24 - * A class representing an IPv4 address. 24 + * A class representing an IP address.
25 + * TODO: Add support for IPv6 as well.
25 */ 26 */
26 public final class IpAddress implements Comparable<IpAddress> { 27 public final class IpAddress implements Comparable<IpAddress> {
27 // IP Versions 28 // IP Versions
...@@ -44,8 +45,6 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -44,8 +45,6 @@ public final class IpAddress implements Comparable<IpAddress> {
44 * @param value the IP address value 45 * @param value the IP address value
45 */ 46 */
46 private IpAddress(Version version, byte[] value) { 47 private IpAddress(Version version, byte[] value) {
47 - checkNotNull(value);
48 -
49 this.version = version; 48 this.version = version;
50 this.octets = Arrays.copyOf(value, INET_BYTE_LENGTH); 49 this.octets = Arrays.copyOf(value, INET_BYTE_LENGTH);
51 } 50 }
...@@ -53,7 +52,7 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -53,7 +52,7 @@ public final class IpAddress implements Comparable<IpAddress> {
53 /** 52 /**
54 * Converts an integer into an IPv4 address. 53 * Converts an integer into an IPv4 address.
55 * 54 *
56 - * @param value an integer representing an IPv4 value 55 + * @param value an integer representing an IPv4 address value
57 * @return an IP address 56 * @return an IP address
58 */ 57 */
59 public static IpAddress valueOf(int value) { 58 public static IpAddress valueOf(int value) {
...@@ -70,6 +69,7 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -70,6 +69,7 @@ public final class IpAddress implements Comparable<IpAddress> {
70 * @return an IP address 69 * @return an IP address
71 */ 70 */
72 public static IpAddress valueOf(byte[] value) { 71 public static IpAddress valueOf(byte[] value) {
72 + checkNotNull(value);
73 return new IpAddress(Version.INET, value); 73 return new IpAddress(Version.INET, value);
74 } 74 }
75 75
...@@ -106,13 +106,13 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -106,13 +106,13 @@ public final class IpAddress implements Comparable<IpAddress> {
106 /** 106 /**
107 * Converts a dotted-decimal string (x.x.x.x) into an IPv4 address. 107 * Converts a dotted-decimal string (x.x.x.x) into an IPv4 address.
108 * 108 *
109 - * @param address a IP address in string form, e.g. "10.0.0.1". 109 + * @param address an IP address in string form, e.g. "10.0.0.1"
110 * @return an IP address 110 * @return an IP address
111 */ 111 */
112 public static IpAddress valueOf(String address) { 112 public static IpAddress valueOf(String address) {
113 final String[] net = address.split("\\."); 113 final String[] net = address.split("\\.");
114 if (net.length != INET_BYTE_LENGTH) { 114 if (net.length != INET_BYTE_LENGTH) {
115 - String msg = "Malformed IPv4 address string; " + 115 + String msg = "Malformed IPv4 address string: " + address + "." +
116 "Address must have four decimal values separated by dots (.)"; 116 "Address must have four decimal values separated by dots (.)";
117 throw new IllegalArgumentException(msg); 117 throw new IllegalArgumentException(msg);
118 } 118 }
...@@ -154,20 +154,21 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -154,20 +154,21 @@ public final class IpAddress implements Comparable<IpAddress> {
154 /** 154 /**
155 * Creates an IP network mask prefix. 155 * Creates an IP network mask prefix.
156 * 156 *
157 - * @param prefixLen the length of the mask prefix. Must be in the interval 157 + * @param prefixLength the length of the mask prefix. Must be in the
158 - * [0, 32] for IPv4 158 + * interval [0, 32] for IPv4
159 * @return a new IP address that contains a mask prefix of the 159 * @return a new IP address that contains a mask prefix of the
160 * specified length 160 * specified length
161 */ 161 */
162 - public static IpAddress makeMaskPrefix(int prefixLen) { 162 + public static IpAddress makeMaskPrefix(int prefixLength) {
163 // Verify the prefix length 163 // Verify the prefix length
164 - if ((prefixLen < 0) || (prefixLen > INET_BIT_LENGTH)) { 164 + if ((prefixLength < 0) || (prefixLength > INET_BIT_LENGTH)) {
165 - final String msg = "Invalid IPv4 prefix length: " + prefixLen + 165 + final String msg = "Invalid IPv4 prefix length: " + prefixLength +
166 ". Must be in the interval [0, 32]."; 166 ". Must be in the interval [0, 32].";
167 throw new IllegalArgumentException(msg); 167 throw new IllegalArgumentException(msg);
168 } 168 }
169 169
170 - long v = (0xffffffffL << (INET_BIT_LENGTH - prefixLen)) & 0xffffffffL; 170 + long v =
171 + (0xffffffffL << (INET_BIT_LENGTH - prefixLength)) & 0xffffffffL;
171 return IpAddress.valueOf((int) v); 172 return IpAddress.valueOf((int) v);
172 } 173 }
173 174
...@@ -176,14 +177,14 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -176,14 +177,14 @@ public final class IpAddress implements Comparable<IpAddress> {
176 * mask length. 177 * mask length.
177 * 178 *
178 * @param addr the address to mask 179 * @param addr the address to mask
179 - * @param prefixLen the length of the mask prefix. Must be in the interval 180 + * @param prefixLength the length of the mask prefix. Must be in the
180 - * [0, 32] for IPv4 181 + * interval [0, 32] for IPv4
181 * @return a new IP address that is masked with a mask prefix of the 182 * @return a new IP address that is masked with a mask prefix of the
182 * specified length 183 * specified length
183 */ 184 */
184 public static IpAddress makeMaskedAddress(final IpAddress addr, 185 public static IpAddress makeMaskedAddress(final IpAddress addr,
185 - int prefixLen) { 186 + int prefixLength) {
186 - IpAddress mask = IpAddress.makeMaskPrefix(prefixLen); 187 + IpAddress mask = IpAddress.makeMaskPrefix(prefixLength);
187 byte[] net = new byte[INET_BYTE_LENGTH]; 188 byte[] net = new byte[INET_BYTE_LENGTH];
188 189
189 // Mask each byte 190 // Mask each byte
...@@ -207,7 +208,7 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -207,7 +208,7 @@ public final class IpAddress implements Comparable<IpAddress> {
207 208
208 @Override 209 @Override
209 public boolean equals(Object obj) { 210 public boolean equals(Object obj) {
210 - if (obj == this) { 211 + if (this == obj) {
211 return true; 212 return true;
212 } 213 }
213 if ((obj == null) || (getClass() != obj.getClass())) { 214 if ((obj == null) || (getClass() != obj.getClass())) {
...@@ -221,7 +222,7 @@ public final class IpAddress implements Comparable<IpAddress> { ...@@ -221,7 +222,7 @@ public final class IpAddress implements Comparable<IpAddress> {
221 @Override 222 @Override
222 /* 223 /*
223 * (non-Javadoc) 224 * (non-Javadoc)
224 - * format is "x.x.x.x" for IPv4 addresses. 225 + * The format is "x.x.x.x" for IPv4 addresses.
225 * 226 *
226 * @see java.lang.Object#toString() 227 * @see java.lang.Object#toString()
227 */ 228 */
......
...@@ -56,6 +56,7 @@ public final class KryoNamespace implements KryoFactory { ...@@ -56,6 +56,7 @@ public final class KryoNamespace implements KryoFactory {
56 public static final class Builder { 56 public static final class Builder {
57 57
58 private final List<Pair<Class<?>, Serializer<?>>> types = new ArrayList<>(); 58 private final List<Pair<Class<?>, Serializer<?>>> types = new ArrayList<>();
59 + private boolean registrationRequired = true;
59 60
60 /** 61 /**
61 * Builds a {@link KryoNamespace} instance. 62 * Builds a {@link KryoNamespace} instance.
...@@ -63,7 +64,7 @@ public final class KryoNamespace implements KryoFactory { ...@@ -63,7 +64,7 @@ public final class KryoNamespace implements KryoFactory {
63 * @return KryoNamespace 64 * @return KryoNamespace
64 */ 65 */
65 public KryoNamespace build() { 66 public KryoNamespace build() {
66 - return new KryoNamespace(types); 67 + return new KryoNamespace(types, registrationRequired);
67 } 68 }
68 69
69 /** 70 /**
...@@ -101,6 +102,11 @@ public final class KryoNamespace implements KryoFactory { ...@@ -101,6 +102,11 @@ public final class KryoNamespace implements KryoFactory {
101 types.addAll(pool.registeredTypes); 102 types.addAll(pool.registeredTypes);
102 return this; 103 return this;
103 } 104 }
105 +
106 + public Builder setRegistrationRequired(boolean registrationRequired) {
107 + this.registrationRequired = registrationRequired;
108 + return this;
109 + }
104 } 110 }
105 111
106 /** 112 /**
...@@ -116,11 +122,11 @@ public final class KryoNamespace implements KryoFactory { ...@@ -116,11 +122,11 @@ public final class KryoNamespace implements KryoFactory {
116 * Creates a Kryo instance pool. 122 * Creates a Kryo instance pool.
117 * 123 *
118 * @param registeredTypes types to register 124 * @param registeredTypes types to register
125 + * @param registrationRequired
119 */ 126 */
120 - private KryoNamespace(final List<Pair<Class<?>, Serializer<?>>> registeredTypes) { 127 + private KryoNamespace(final List<Pair<Class<?>, Serializer<?>>> registeredTypes, boolean registrationRequired) {
121 this.registeredTypes = ImmutableList.copyOf(registeredTypes); 128 this.registeredTypes = ImmutableList.copyOf(registeredTypes);
122 - // always true for now 129 + this.registrationRequired = registrationRequired;
123 - this.registrationRequired = true;
124 } 130 }
125 131
126 /** 132 /**
......
...@@ -22,7 +22,7 @@ import static org.junit.Assert.assertTrue; ...@@ -22,7 +22,7 @@ import static org.junit.Assert.assertTrue;
22 import java.util.Arrays; 22 import java.util.Arrays;
23 23
24 import org.junit.Test; 24 import org.junit.Test;
25 -import org.onlab.packet.IpPrefix.Version; 25 +import org.onlab.packet.IpAddress.Version;
26 26
27 import com.google.common.testing.EqualsTester; 27 import com.google.common.testing.EqualsTester;
28 28
...@@ -30,8 +30,9 @@ public class IpPrefixTest { ...@@ -30,8 +30,9 @@ public class IpPrefixTest {
30 30
31 private static final byte [] BYTES1 = new byte [] {0xa, 0x0, 0x0, 0xa}; 31 private static final byte [] BYTES1 = new byte [] {0xa, 0x0, 0x0, 0xa};
32 private static final byte [] BYTES2 = new byte [] {0xa, 0x0, 0x0, 0xb}; 32 private static final byte [] BYTES2 = new byte [] {0xa, 0x0, 0x0, 0xb};
33 - private static final int INTVAL1 = 167772170; 33 + private static final int INTVAL0 = 0x0a000000;
34 - private static final int INTVAL2 = 167772171; 34 + private static final int INTVAL1 = 0x0a00000a;
35 + private static final int INTVAL2 = 0x0a00000b;
35 private static final String STRVAL = "10.0.0.12/16"; 36 private static final String STRVAL = "10.0.0.12/16";
36 private static final int MASK_LENGTH = 16; 37 private static final int MASK_LENGTH = 16;
37 38
...@@ -59,27 +60,29 @@ public class IpPrefixTest { ...@@ -59,27 +60,29 @@ public class IpPrefixTest {
59 @Test 60 @Test
60 public void basics() { 61 public void basics() {
61 IpPrefix ip1 = IpPrefix.valueOf(BYTES1, MASK_LENGTH); 62 IpPrefix ip1 = IpPrefix.valueOf(BYTES1, MASK_LENGTH);
62 - final byte [] bytes = new byte [] {0xa, 0x0, 0x0, 0xa}; 63 + final byte [] bytes = new byte [] {0xa, 0x0, 0x0, 0x0};
63 64
64 - //check fields 65 + // check fields
65 assertEquals("incorrect IP Version", Version.INET, ip1.version()); 66 assertEquals("incorrect IP Version", Version.INET, ip1.version());
66 assertEquals("incorrect netmask", 16, ip1.prefixLength()); 67 assertEquals("incorrect netmask", 16, ip1.prefixLength());
67 - assertTrue("faulty toOctets()", Arrays.equals(bytes, ip1.toOctets())); 68 + assertTrue("faulty toOctets()",
68 - assertEquals("faulty toInt()", INTVAL1, ip1.toInt()); 69 + Arrays.equals(bytes, ip1.address().toOctets()));
69 - assertEquals("faulty toString()", "10.0.0.10/16", ip1.toString()); 70 + assertEquals("faulty toInt()", INTVAL0, ip1.address().toInt());
71 + assertEquals("faulty toString()", "10.0.0.0/16", ip1.toString());
70 } 72 }
71 73
72 @Test 74 @Test
73 public void netmasks() { 75 public void netmasks() {
74 // masked 76 // masked
75 IpPrefix ip1 = IpPrefix.valueOf(BYTES1, MASK_LENGTH); 77 IpPrefix ip1 = IpPrefix.valueOf(BYTES1, MASK_LENGTH);
76 - 78 + IpPrefix ip2 = IpPrefix.valueOf("10.0.0.10/16");
77 - IpPrefix host = IpPrefix.valueOf("0.0.0.10/16"); 79 + IpPrefix ip3 = IpPrefix.valueOf("10.0.0.0/16");
78 - IpPrefix network = IpPrefix.valueOf("10.0.0.0/16"); 80 + assertEquals("incorrect binary masked address",
79 - assertEquals("incorrect host address", host, ip1.host()); 81 + ip1.toString(), "10.0.0.0/16");
80 - assertEquals("incorrect network address", network, ip1.network()); 82 + assertEquals("incorrect string masked address",
81 - assertEquals("incorrect netmask", "255.255.0.0", ip1.netmask().toString()); 83 + ip2.toString(), "10.0.0.0/16");
82 - 84 + assertEquals("incorrect network address",
85 + ip2.toString(), "10.0.0.0/16");
83 } 86 }
84 87
85 @Test 88 @Test
......