fixes #18103 · Pull Request #18104 · EFForg/https-everywhere · GitHub
Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

fixes #18103#18104

Merged
Hainish merged 2 commits into
masterfrom
unknown repository
Jun 26, 2019
Merged

fixes #18103#18104
Hainish merged 2 commits into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Jun 19, 2019

Copy link
Copy Markdown

Killing adb server(aka daemon) prevents seeing groups of these messages
on dmesg every second:

[ 1460.763742] pcieport 0000:00:1c.4: restoring config space at offset 0x2c (was 0x0, writing 0x0)
[ 1460.763748] pcieport 0000:00:1c.4: restoring config space at offset 0x28 (was 0x0, writing 0x0)
[ 1460.763752] pcieport 0000:00:1c.4: restoring config space at offset 0x24 (was 0x1fff1, writing 0x1fff1)
[ 1460.763813] pcieport 0000:00:1c.4: PME# disabled
[ 1460.763920] xhci_hcd 0000:03:00.0: PME# disabled
[ 1460.763928] xhci_hcd 0000:03:00.0: enabling bus mastering
[ 1460.770239] xhci_hcd 0000:03:00.0: PME# enabled
[ 1460.770377] xhci_hcd 0000:03:00.0: PME# disabled
[ 1460.770383] xhci_hcd 0000:03:00.0: enabling bus mastering
[ 1460.884813] xhci_hcd 0000:03:00.0: PME# enabled
[ 1460.884925] pcieport 0000:00:1c.4: PME# enabled

safety bonus: we only kill it if we started it.

issue is here: #18103

Killing `adb` server(aka daemon) prevents seeing groups of these messages
on dmesg every second:
[ 1460.763742] pcieport 0000:00:1c.4: restoring config space at offset 0x2c (was 0x0, writing 0x0)
[ 1460.763748] pcieport 0000:00:1c.4: restoring config space at offset 0x28 (was 0x0, writing 0x0)
[ 1460.763752] pcieport 0000:00:1c.4: restoring config space at offset 0x24 (was 0x1fff1, writing 0x1fff1)
[ 1460.763813] pcieport 0000:00:1c.4: PME# disabled
[ 1460.763920] xhci_hcd 0000:03:00.0: PME# disabled
[ 1460.763928] xhci_hcd 0000:03:00.0: enabling bus mastering
[ 1460.770239] xhci_hcd 0000:03:00.0: PME# enabled
[ 1460.770377] xhci_hcd 0000:03:00.0: PME# disabled
[ 1460.770383] xhci_hcd 0000:03:00.0: enabling bus mastering
[ 1460.884813] xhci_hcd 0000:03:00.0: PME# enabled
[ 1460.884925] pcieport 0000:00:1c.4: PME# enabled

safety bonus: we only kill it if we started it.
@ghost

ghost commented Jun 19, 2019

Copy link
Copy Markdown
Author

Comment thread utils/android-push.sh Outdated
no need to futureproof an 'if' case (causing confusion) for when the "=="
would be changed to an "-eq" and $we_started_adb_daemon would be empty.

@pipboy96 pipboy96 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Hainish Hainish merged commit 158dd36 into EFForg:master Jun 26, 2019
@Hainish

Hainish commented Jun 26, 2019

Copy link
Copy Markdown
Member

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants