-
Notifications
You must be signed in to change notification settings - Fork 24
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
Badicsalex s3 sleep support #13
base: master
Are you sure you want to change the base?
Badicsalex s3 sleep support #13
Conversation
Useful for users, who want to use different utilities with their SED drives, and also in combination with "-x -n", meaning "no hash", "hex", so no plaintext passwords are saved to scripts or shell history.
The new command is --prepareForS3Sleep, and it should be called every new boot, as it stores the drive key (password hash) in kernel memory.
This is to allow proper '-h' (no password hashing) usage, where the user might have saved the hash itself, or used a different hashing mechanism altogether, and ends up with 0 bytes or control characters in the resulting string.
Please comment if you have tested this PR, including system info on test system (OS, brand of motherboard or laptop, etc). |
I have tested this on a Lenovo X1 Extreme (Gen 1) with Manjaro having 2 NVME disks with one of them being enabled for SED. Sleep works. |
I merged your commit to my local branch and compiled it. But on my Desktop with an SATA Samsung 850 EVO I was not able to get it working (It works with sedutil 1.15.1 from DTA). I followed the instauctions:
but when I want to test it with
After a full power down I can boot the drive (not the rescue image) and can revert (see picture) without any data loss. Can you provide the RESCUE64.img and sedutil-cli binary to let me test? |
Frankly, i never used the |
But I booted the drive and the debug password did not work (is OPAL Failed). This should work not only with a real custom password, shouldn't it? |
@mabachel yeah, it should. The things coming to my mind are an improperly followed procedure, or mismatched version (SHA1 pw hash vs SHA256), missing kernel params for SATA ( |
I followed the instructions for SATA devices and enabled that |
works. Manjaro 20 & Ryzen & KC2500 NVME. opening dmesg -w to monitor last messages of death, but with this port finally resumes seamlessly. funny something similar has to be done with NIC (igc driver) on all distros except Manjaro. Manjaro is the only OS that actually installs on Ryzen+Nvidia platform without need of safe mode.
note: so each fork is not working with another. would love devs cooperate #forkfest |
Tested on Arch Linux on Lenovo T15g with two NVME drives: original WDC PC SN730 SDBQNTY-512G-1001 and additional Crucial CT1000P5SSD8). Worked, the computer was able to wake up from the sleep and continue. Thanks!!! I actually ended up here because I have made a mistake (?) by using the rescue image and instructions from sedutil.com and, as result, ended up with the version that used the hashing algorithm different from the original DriveTrustAlliance's sedutil. Then I was unable to use the default sedutil-cli or sedutil-sleep-git package from AUR - as I later understood, because of the hashing method, not command worked. Two comments:
|
code ported to work with @ChubbyAnt 's master branch.
Tested on Manjaro Linux with a 5.6.12 kernel on a Thinkpad with a Samsung 970 EVO Plus.