Skip to content

[Hexagon] Adjust Hexagon pytest port range#13172

Merged
mehrdadh merged 1 commit intoapache:mainfrom
supersat:fix-hexagon-pytest-ports
Oct 21, 2022
Merged

[Hexagon] Adjust Hexagon pytest port range#13172
mehrdadh merged 1 commit intoapache:mainfrom
supersat:fix-hexagon-pytest-ports

Conversation

@supersat
Copy link
Contributor

ADB, the Android Debug Bridge, tries to determine if an emulator is present by looking for port 5555 being open. Unfortunately, once adb thinks there's an emulator running, it will continously try to connect to it, adding an error to its log on every failed attempt. This log file quickly grows to hundreds of gigabytes before developers notice they are completely out of disk space. This problem is further compounded by the Hexagon Launcher forwarding 10 contiguous ports from the randomly-chosen RPC base port, causing this failure to happen more often than you'd otherwise suspect.

See https://stackoverflow.com/questions/3152681/android-emulator-5554-offline/10356594#10356594

cc: @csullivan @mehrdadh @cconvey

ADB, the Android Debug Bridge, tries to determine if an emulator is present by
looking for port 5555 being open. Unfortunately, once adb thinks there's an
emulator running, it will continously try to connect to it, adding an error
to its log on every failed attempt. This log file quickly grows to hundreds
of gigabytes before developers notice they are completely out of disk space.
This problem is further compounded by the Hexagon Launcher forwarding 10
contiguous ports from the randomly-chosen RPC base port, causing this failure
to happen more often than you'd otherwise suspect.

See https://stackoverflow.com/questions/3152681/android-emulator-5554-offline/10356594#10356594
@tvm-bot
Copy link
Collaborator

tvm-bot commented Oct 21, 2022

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

Copy link
Member

@mehrdadh mehrdadh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great catch! thanks for the fix!

@mehrdadh mehrdadh merged commit e42e4bb into apache:main Oct 21, 2022
@supersat supersat deleted the fix-hexagon-pytest-ports branch October 21, 2022 23:23
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 10, 2022
Adjust Hexagon pytest port range

ADB, the Android Debug Bridge, tries to determine if an emulator is present by
looking for port 5555 being open. Unfortunately, once adb thinks there's an
emulator running, it will continously try to connect to it, adding an error
to its log on every failed attempt. This log file quickly grows to hundreds
of gigabytes before developers notice they are completely out of disk space.
This problem is further compounded by the Hexagon Launcher forwarding 10
contiguous ports from the randomly-chosen RPC base port, causing this failure
to happen more often than you'd otherwise suspect.

See https://stackoverflow.com/questions/3152681/android-emulator-5554-offline/10356594#10356594
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
Adjust Hexagon pytest port range

ADB, the Android Debug Bridge, tries to determine if an emulator is present by
looking for port 5555 being open. Unfortunately, once adb thinks there's an
emulator running, it will continously try to connect to it, adding an error
to its log on every failed attempt. This log file quickly grows to hundreds
of gigabytes before developers notice they are completely out of disk space.
This problem is further compounded by the Hexagon Launcher forwarding 10
contiguous ports from the randomly-chosen RPC base port, causing this failure
to happen more often than you'd otherwise suspect.

See https://stackoverflow.com/questions/3152681/android-emulator-5554-offline/10356594#10356594
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants