From 824840d16de731860efe80df981ec10ae78bb97e Mon Sep 17 00:00:00 2001 From: Xiao Ma Date: Tue, 28 Dec 2021 06:33:23 +0000 Subject: [PATCH] Move Ethernet related files to f/b/packages/ConnectivityT. ethernet-service is going to be moved into Connectivity mainline module. Notice that below sources are also only used in the ethernet framework during compiling the filegroup: framework-connectivity-ethernet-sources. Move them from f/b to f/b/packages/ConnectivityT as well. Ethernet framework only related files: - IInternalNetworkManagementListener.aidl - InternalNetworkManagementException.java - InternalNetworkManagementException.aidl - InternalNetworkUpdateRequest.java - InternalNetworkUpdateRequest.aidl Ethernet service only related files: - DelayedDiskWrite.java(IpConfigStore imports this class) Bug: 210586283 Test: build pass atest FrameworksNetTests atest EthernetServiceTests Change-Id: I1ec2d1d182c04f3f2acc9b757d5061ca749a4a3c --- packages/ConnectivityT/framework-t/Android.bp | 5 +++++ .../IInternalNetworkManagementListener.aidl | 0 .../InternalNetworkManagementException.aidl | 0 .../InternalNetworkManagementException.java | 0 .../net/InternalNetworkUpdateRequest.aidl | 0 .../net/InternalNetworkUpdateRequest.java | 0 packages/ConnectivityT/service/Android.bp | 1 + .../android/server/net/DelayedDiskWrite.java | 22 ++++++++++++++++--- 8 files changed, 25 insertions(+), 3 deletions(-) rename {core/java => packages/ConnectivityT/framework-t/src}/android/net/IInternalNetworkManagementListener.aidl (100%) rename {core/java => packages/ConnectivityT/framework-t/src}/android/net/InternalNetworkManagementException.aidl (100%) rename {core/java => packages/ConnectivityT/framework-t/src}/android/net/InternalNetworkManagementException.java (100%) rename {core/java => packages/ConnectivityT/framework-t/src}/android/net/InternalNetworkUpdateRequest.aidl (100%) rename {core/java => packages/ConnectivityT/framework-t/src}/android/net/InternalNetworkUpdateRequest.java (100%) rename {services/core/java => packages/ConnectivityT/service/src}/com/android/server/net/DelayedDiskWrite.java (82%) diff --git a/packages/ConnectivityT/framework-t/Android.bp b/packages/ConnectivityT/framework-t/Android.bp index d3d8bba16c7c4..223bdcdd9c955 100644 --- a/packages/ConnectivityT/framework-t/Android.bp +++ b/packages/ConnectivityT/framework-t/Android.bp @@ -129,6 +129,11 @@ filegroup { "src/android/net/EthernetNetworkSpecifier.java", "src/android/net/IEthernetManager.aidl", "src/android/net/IEthernetServiceListener.aidl", + "src/android/net/IInternalNetworkManagementListener.aidl", + "src/android/net/InternalNetworkUpdateRequest.java", + "src/android/net/InternalNetworkUpdateRequest.aidl", + "src/android/net/InternalNetworkManagementException.java", + "src/android/net/InternalNetworkManagementException.aidl", "src/android/net/ITetheredInterfaceCallback.aidl", ], path: "src", diff --git a/core/java/android/net/IInternalNetworkManagementListener.aidl b/packages/ConnectivityT/framework-t/src/android/net/IInternalNetworkManagementListener.aidl similarity index 100% rename from core/java/android/net/IInternalNetworkManagementListener.aidl rename to packages/ConnectivityT/framework-t/src/android/net/IInternalNetworkManagementListener.aidl diff --git a/core/java/android/net/InternalNetworkManagementException.aidl b/packages/ConnectivityT/framework-t/src/android/net/InternalNetworkManagementException.aidl similarity index 100% rename from core/java/android/net/InternalNetworkManagementException.aidl rename to packages/ConnectivityT/framework-t/src/android/net/InternalNetworkManagementException.aidl diff --git a/core/java/android/net/InternalNetworkManagementException.java b/packages/ConnectivityT/framework-t/src/android/net/InternalNetworkManagementException.java similarity index 100% rename from core/java/android/net/InternalNetworkManagementException.java rename to packages/ConnectivityT/framework-t/src/android/net/InternalNetworkManagementException.java diff --git a/core/java/android/net/InternalNetworkUpdateRequest.aidl b/packages/ConnectivityT/framework-t/src/android/net/InternalNetworkUpdateRequest.aidl similarity index 100% rename from core/java/android/net/InternalNetworkUpdateRequest.aidl rename to packages/ConnectivityT/framework-t/src/android/net/InternalNetworkUpdateRequest.aidl diff --git a/core/java/android/net/InternalNetworkUpdateRequest.java b/packages/ConnectivityT/framework-t/src/android/net/InternalNetworkUpdateRequest.java similarity index 100% rename from core/java/android/net/InternalNetworkUpdateRequest.java rename to packages/ConnectivityT/framework-t/src/android/net/InternalNetworkUpdateRequest.java diff --git a/packages/ConnectivityT/service/Android.bp b/packages/ConnectivityT/service/Android.bp index b261e165a112d..36dd20054c5c5 100644 --- a/packages/ConnectivityT/service/Android.bp +++ b/packages/ConnectivityT/service/Android.bp @@ -66,6 +66,7 @@ filegroup { filegroup { name: "services.connectivity-ethernet-sources", srcs: [ + "src/com/android/server/net/DelayedDiskWrite.java", "src/com/android/server/net/IpConfigStore.java", ], path: "src", diff --git a/services/core/java/com/android/server/net/DelayedDiskWrite.java b/packages/ConnectivityT/service/src/com/android/server/net/DelayedDiskWrite.java similarity index 82% rename from services/core/java/com/android/server/net/DelayedDiskWrite.java rename to packages/ConnectivityT/service/src/com/android/server/net/DelayedDiskWrite.java index 8f09eb7c19ab6..35dc4557252cd 100644 --- a/services/core/java/com/android/server/net/DelayedDiskWrite.java +++ b/packages/ConnectivityT/service/src/com/android/server/net/DelayedDiskWrite.java @@ -26,21 +26,37 @@ import java.io.DataOutputStream; import java.io.FileOutputStream; import java.io.IOException; +/** + * This class provides APIs to do a delayed data write to a given {@link OutputStream}. + */ public class DelayedDiskWrite { + private static final String TAG = "DelayedDiskWrite"; + private HandlerThread mDiskWriteHandlerThread; private Handler mDiskWriteHandler; /* Tracks multiple writes on the same thread */ private int mWriteSequence = 0; - private final String TAG = "DelayedDiskWrite"; + /** + * Used to do a delayed data write to a given {@link OutputStream}. + */ public interface Writer { - public void onWriteCalled(DataOutputStream out) throws IOException; + /** + * write data to a given {@link OutputStream}. + */ + void onWriteCalled(DataOutputStream out) throws IOException; } + /** + * Do a delayed data write to a given output stream opened from filePath. + */ public void write(final String filePath, final Writer w) { write(filePath, w, true); } + /** + * Do a delayed data write to a given output stream opened from filePath. + */ public void write(final String filePath, final Writer w, final boolean open) { if (TextUtils.isEmpty(filePath)) { throw new IllegalArgumentException("empty file path"); @@ -77,7 +93,7 @@ public class DelayedDiskWrite { if (out != null) { try { out.close(); - } catch (Exception e) {} + } catch (Exception e) { } } // Quit if no more writes sent