Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6fbc9c45 authored by Chalard Jean's avatar Chalard Jean
Browse files

[NS A38] Fill the initial reassignment 

This is more expensive for now but it will allow subsequent patches
to read relevant state from the reassignment instead of from the
global state, which will stop the computation of the reassignment
from reading state that is mutated by the same loop. Eventually
this lets us completely split the computation from the side effects.
The ugly parts of this patch will be cleaned up later as a result,
namely in patches [NS B04] and [NS B05].

Test: ConnectivityServiceTest
Change-Id: I271e7f4d4bc81493c1ea212025b7130619592a8a
parent 25be037b

services/core/java/com/android/server/ConnectivityService.java

+13 −11
Original line number Diff line number Diff line
@@ -6500,6 +6500,16 @@ public class ConnectivityService extends IConnectivityManager.Stub 
        }

    }



    // TODO : remove this when it's useless

    @NonNull private NetworkReassignment computeInitialReassignment() {

        final NetworkReassignment change = new NetworkReassignment();

        for (NetworkRequestInfo nri : mNetworkRequests.values()) {

            change.addRequestReassignment(new NetworkReassignment.RequestReassignment(nri,

                    nri.mSatisfier, nri.mSatisfier));

        }

        return change;

    }



    private ArrayMap<NetworkRequestInfo, NetworkAgentInfo> computeRequestReassignmentForNetwork(

            @NonNull final NetworkAgentInfo newNetwork) {

        final int score = newNetwork.getCurrentScore();
@@ -6541,19 +6551,9 @@ public class ConnectivityService extends IConnectivityManager.Stub 
    //   satisfied by newNetwork, and reassigns to newNetwork

    //   any such requests for which newNetwork is the best.

    //

    // - Lingers any validated Networks that as a result are no longer

    //   needed. A network is needed if it is the best network for

    //   one or more NetworkRequests, or if it is a VPN.

    //

    // - Writes into the passed reassignment object all changes that should be done for

    //   rematching this network with all requests, to be applied later.

    //

    // NOTE: This function only adds NetworkRequests that "newNetwork" could satisfy,

    // it does not remove NetworkRequests that other Networks could better satisfy.

    // If you need to handle decreases in score, use {@link rematchAllNetworksAndRequests}.

    // This function should be used when possible instead of {@code rematchAllNetworksAndRequests}

    // as it performs better by a factor of the number of Networks.

    //

    // TODO : stop writing to the passed reassignment. This is temporarily more useful, but

    // it's unidiomatic Java and it's hard to read.

    //
@@ -6634,7 +6634,7 @@ public class ConnectivityService extends IConnectivityManager.Stub 
        // scoring network and then a higher scoring network, which could produce multiple

        // callbacks.

        Arrays.sort(nais);

        final NetworkReassignment changes = new NetworkReassignment();

        final NetworkReassignment changes = computeInitialReassignment();

        for (final NetworkAgentInfo nai : nais) {

            rematchNetworkAndRequests(changes, nai, now);

        }
@@ -6663,6 +6663,8 @@ public class ConnectivityService extends IConnectivityManager.Stub 
        // before LegacyTypeTracker sends legacy broadcasts

        for (final NetworkReassignment.RequestReassignment event :

                changes.getRequestReassignments()) {

            if (event.mOldNetwork == event.mNewNetwork) continue;



            // Tell NetworkProviders about the new score, so they can stop

            // trying to connect if they know they cannot match it.

            // TODO - this could get expensive if there are a lot of outstanding requests for this