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

Commit 1cc3f4a4 authored by Nucca Chen's avatar Nucca Chen Committed by Android (Google) Code Review
Browse files

Merge "A minor followup change for BPF offload device option stuff" into rvc-dev

parents 85f2c794 61d59a74
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -59,7 +59,7 @@

    <!-- Use the BPF offload for tethering when the kernel has support. True by default.
         If the device doesn't want to support tether BPF offload, this should be false.
         Note that this setting could be override by device config.
         Note that this setting could be overridden by device config.
    -->
    <bool translatable="false" name="config_tether_enable_bpf_offload">true</bool>

+1 −1
Original line number Diff line number Diff line
@@ -25,7 +25,7 @@
            <item type="array" name="config_tether_dhcp_range"/>
            <!-- Use the BPF offload for tethering when the kernel has support. True by default.
                 If the device doesn't want to support tether BPF offload, this should be false.
                 Note that this setting could be override by device config.
                 Note that this setting could be overridden by device config.
            -->
            <item type="bool" name="config_tether_enable_bpf_offload"/>
            <item type="bool" name="config_tether_enable_legacy_dhcp_server"/>
+6 −6
Original line number Diff line number Diff line
@@ -303,6 +303,8 @@ public class IpServer extends StateMachine {

    private final IpNeighborMonitor mIpNeighborMonitor;

    // TODO: Add a dependency object to pass the data members or variables from the tethering
    // object. It helps to reduce the arguments of the constructor.
    public IpServer(
            String ifaceName, Looper looper, int interfaceType, SharedLog log,
            INetd netd, Callback callback, boolean usingLegacyDhcp, boolean usingBpfOffload,
@@ -325,15 +327,13 @@ public class IpServer extends StateMachine {
        mIpNeighborMonitor = mDeps.getIpNeighborMonitor(getHandler(), mLog,
                new MyNeighborEventConsumer());

        // IP neighbor monitor monitors the neighbor event for adding/removing offload
        // IP neighbor monitor monitors the neighbor events for adding/removing offload
        // forwarding rules per client. If BPF offload is not supported, don't start listening
        // neighbor events. See updateIpv6ForwardingRules, addIpv6ForwardingRule,
        // for neighbor events. See updateIpv6ForwardingRules, addIpv6ForwardingRule,
        // removeIpv6ForwardingRule.
        if (mUsingBpfOffload) {
            if (!mIpNeighborMonitor.start()) {
        if (mUsingBpfOffload && !mIpNeighborMonitor.start()) {
            mLog.e("Failed to create IpNeighborMonitor on " + mIfaceName);
        }
        }

        mInitialState = new InitialState();
        mLocalHotspotState = new LocalHotspotState();