Skip to content

Commit df44184

Browse files
johngroAndroid (Google) Code Review
authored andcommitted
Merge "common_time: Move default election config to bcast." into jb-mr1-dev
2 parents 4a81ef8 + 7a947c4 commit df44184

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

services/common_time/common_time_server.cpp

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,9 @@
5353

5454
namespace android {
5555

56-
const char* CommonTimeServer::kDefaultMasterElectionAddr = "239.195.128.88";
57-
const uint16_t CommonTimeServer::kDefaultMasterElectionPort = 8887;
58-
const uint64_t CommonTimeServer::kDefaultSyncGroupID = 0;
56+
const char* CommonTimeServer::kDefaultMasterElectionAddr = "255.255.255.255";
57+
const uint16_t CommonTimeServer::kDefaultMasterElectionPort = 8886;
58+
const uint64_t CommonTimeServer::kDefaultSyncGroupID = 1;
5959
const uint8_t CommonTimeServer::kDefaultMasterPriority = 1;
6060
const uint32_t CommonTimeServer::kDefaultMasterAnnounceIntervalMs = 10000;
6161
const uint32_t CommonTimeServer::kDefaultSyncRequestIntervalMs = 1000;
@@ -752,6 +752,9 @@ bool CommonTimeServer::handleTimeoutWaitForElection() {
752752
bool CommonTimeServer::handleWhoIsMasterRequest(
753753
const WhoIsMasterRequestPacket* request,
754754
const sockaddr_storage& srcAddr) {
755+
// Skip our own messages which come back via broadcast loopback.
756+
if (request->senderDeviceID == mDeviceID)
757+
return true;
755758

756759
char srcEPStr[64];
757760
sockaddrToString(srcAddr, true, srcEPStr, sizeof(srcEPStr));
@@ -829,6 +832,10 @@ bool CommonTimeServer::handleWhoIsMasterRequest(
829832
bool CommonTimeServer::handleWhoIsMasterResponse(
830833
const WhoIsMasterResponsePacket* response,
831834
const sockaddr_storage& srcAddr) {
835+
// Skip our own messages which come back via broadcast loopback.
836+
if (response->deviceID == mDeviceID)
837+
return true;
838+
832839
char srcEPStr[64];
833840
sockaddrToString(srcAddr, true, srcEPStr, sizeof(srcEPStr));
834841
mElectionLog.log("RXed WhoIs master response while in state %s. "
@@ -996,6 +1003,10 @@ bool CommonTimeServer::handleMasterAnnouncement(
9961003
uint8_t newDevicePrio = packet->devicePriority;
9971004
uint64_t newTimelineID = packet->timelineID;
9981005

1006+
// Skip our own messages which come back via broadcast loopback.
1007+
if (newDeviceID == mDeviceID)
1008+
return true;
1009+
9991010
char srcEPStr[64];
10001011
sockaddrToString(srcAddr, true, srcEPStr, sizeof(srcEPStr));
10011012
mElectionLog.log("RXed master announcement while in state %s. "

0 commit comments

Comments
 (0)