Skip to content
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

boot: cancelable kernel loads #1698

Closed
wants to merge 4 commits into from
Closed

Conversation

hugelgupf
Copy link
Member

This implements Option 1 from #1672.

Despite me liking that option less when I wrote #1672, I had started working on version 2 as well, and it involved basically a second copy of every structure we have in pkg/boot and weird translations between them.

Putting this out as an RFC.

@hugelgupf hugelgupf force-pushed the lazy-files branch 3 times, most recently from 6b0ef14 to 9e5a202 Compare June 18, 2020 05:10
@hugelgupf hugelgupf force-pushed the lazy-files branch 2 times, most recently from ac21738 to 9cb2938 Compare June 26, 2020 03:24
@hugelgupf hugelgupf marked this pull request as ready for review June 26, 2020 03:24
@hugelgupf hugelgupf force-pushed the lazy-files branch 2 times, most recently from 4c43d9b to bb006f4 Compare July 5, 2020 07:49
@hugelgupf
Copy link
Member Author

I just added the ability to interrupt a kernel from being downloaded if it's already been chosen in the boot menu. So you can do something like choose option 1, but still cancel it as long as the actual kexec hasn't happened yet. Useful if the download is taking a long time and you want to explore other options (or drop to shell and debug why it's taking so long).

Signed-off-by: Chris Koch <chrisko@google.com>
We'll be able to test interruptions with this.

Signed-off-by: Chris Koch <chrisko@google.com>
Signed-off-by: Chris Koch <chrisko@google.com>
Signed-off-by: Chris Koch <chrisko@google.com>
@hugelgupf hugelgupf changed the title boot: cancelable kernel downloads boot: cancelable kernel loads Jul 13, 2020
@rminnich
Copy link
Member

do we still want this? It seems useful?

@rminnich rminnich added the Awaiting author Waiting for new changes or feedback for author. label May 25, 2021
@rjoleary rjoleary removed their request for review September 21, 2021 08:17
@rminnich
Copy link
Member

Cleaning out PRs > 1yo

@rminnich rminnich closed this Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting author Waiting for new changes or feedback for author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants