-
-
Notifications
You must be signed in to change notification settings - Fork 266
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARM 64 support #690
Comments
It compiles just fine on ARM64 and runs. @Spiritreader can you compile the releases for ARM64 and release them? I see significant perf improvements when running my ARM64 compiled version of the app vs the x86_64 version |
As ARM support improves we will of course look at building for that arch. Once we know for certain that the entirety of the System.Management tools are compatible, including WMI and #660, we can move ahead in looking at this. So, the deal is this: I don't have an ARM system, so I can't test for it. The most useful thing people can do to move this along would be to have testers on ARM actively compile that app and debug it to check whether the ManagementEventWatcher calls are working:
They are used here:
and here:
As well as in the UI here:
and here:
If that doesn't throw an exception, and works then we can move ahead.
Unfortunately, I don't have a lot of time at the moment and new features / arch changes will remain on the backburner for the forseeable future. However: The more preparation steps are done, the faster things move along in the future. |
I did a search and found dotnet/runtime#81400 . Updating the dependency System.Management to 8.0.0 of AutoDarkModeApp and AutoDarkModeSvc seems to have fixed the UI error as #812 . |
@xgdgsc if you can verify that everything works here we can move along with this. The next challenge at that point would be modifying the update system to detect which arch windows runs on and modify it accordingly. We could then also look into this again #658 |
Yes. It works. I have a group chat of people with arm devices testing this and no issue have been reported so far. |
I also have an arm device Surface Pro 9 if you need another tester. |
- support arm on publish profiles - arch display on info
Please check if this works And verify that it says "Arch: ARM64" or similar on the "About page" Example: This should say ARM64 instead of X64 Once I know that this build works I can start integrating arm builds into the update system (where the idea is that ADM will automatically upgrade to an ARM version if one is available, no user input required) |
Do you guys feel any improvements with the native ARM64 version compared to the previous x86 version? |
yep, feels way more responsive. |
Alright, then the only thing we have to resolve is that the updater (which is still x86 due to technical reasons again) cannot seem to write to disk in x86 emulated mode. Once we figured that out I'll update again. Atm they're stil unsigned, as soon as they are integrated into the update system I will include them into our signing system. |
Again needing a ARM user to verify that: it seems like the updater doesn't have write access. Please download the newest STABLE Auto Dark Mode release version. Then open Auto Dark Mode App and navigate to settings. Enter the Beta channel and search for an update. Please try to install the update. Does it work or do you see an update error? We would much appreciate if you would try this. This is a major issue. |
You would need an older version than current to trigger the updater. So make sure to download stable release. |
I installed the latest stable X64 version and checked for updates within the application. Although I successfully updated to the Beta version, it remains an X64 application instead of being ARM64 as expected. |
The arm64 version has been used on Surface Pro X. Compared with the X64 version, the page opening speed is 4 seconds faster (x64 takes more than 8 seconds). The page cannot slide normally when the screen is touched. When selecting the BETA channel to update, it will prompt that there is no update content.
|
I am running Windows on a Parallels virtual machine on my M1 Pro MacBook, and the update program is functioning normally. |
Updating with the built-in updater should now upgrade all ARM users to the ARM64 version of Auto Dark Mode Once you're on beta, you will first be offered a non ARM upgrade. Keep in mind that this upgrade path is completely untested, please report issues with the upgrade process immediately. |
The upgrade failed, still in 10.4.1.1 version ,attach with the log: |
Downloading from https://github.com/AutoDarkMode/Windows-Auto-Night-Mode/releases/download/10.4.1.1/AutoDarkModeX_10.4.1.1.exe and update to beta version of 10.4.2.10 (x64) works for me on arm. |
@HuaDeity Yep, you've run into a known issue where for some reason administrative privileges are required to run the updater. We're not yet sure what causes this. @xgdgsc Can you now update to 10.4.2.10 ARM? Or do you run into the same issue? |
No new update prompt on 10.4.2.10 (x64) version. While using https://github.com/AutoDarkMode/AutoDarkModeVersion/releases/download/10.4.2.10/ADM_ARM64.zip everytime I start I can see an update with same version number available and a download can be started and progress finished then nothing changes. |
Please check if it works now. |
I successfully upgraded to the Arm64 version 10.4.2.11 while running as an administrator. The upgrade process first installed the x64 version of 10.4.2.11 before proceeding with the Arm64 version. However, even after upgrading to the Arm64 version, I continued to receive prompts for further upgrades which resulted in an infinite loop of launching and upgrading attempts. |
Should be fixed with 10.4.2.12. The new update should be offered now and once you're on that version the loop should stop. However I have no idea why administrator privileges are requires still? Is it not possible to launch x86-64 apps from an ARM context? EDIT: I've pushed 10.4.2.13 which includes an ARM64 build for the updater. Please try updating and check if you still need administrator privileges to perform the update. |
Updating from 10.4.2.10 to 10.4.2.13 works for me. |
The update was successful without requiring administrator privileges. |
How's everyone's experience with the ARM build compared to the previous x86 version? Did you encounter any issues so far? |
No issue here. |
I am sorry but I didn't see any win arm64 version of ADM in releases, and the previous links have expired. Can anyone give me the latest version of ADM on win arm 64? |
|
Thanks a lot! |
We are currently preparing the stable release for ADMs ARM build. But there will be an installer that ships the ARM version soon |
Thanks a lot! Expecting the new release! |
@Spiritreader what's the status on the ARM installer? |
You can expect it in late November or maybe even December. |
Can an arm64 version be produced? X64 freezes when opening in Windows On Arm, thanks to the developer
The text was updated successfully, but these errors were encountered: