Skip to content

Add test and docs for many encrypted fs#18

Open
mattludwigs wants to merge 1 commit into
mainfrom
config-many-encrypted-devices
Open

Add test and docs for many encrypted fs#18
mattludwigs wants to merge 1 commit into
mainfrom
config-many-encrypted-devices

Conversation

@mattludwigs
Copy link
Copy Markdown
Contributor

Added a little stub code around to mounting dm_crypt to feel out what implementation might work like since we don't really have arrays in the language runtime. I am sure that stub will all change and we will have a better solution. I figured implementing a proper array in the runtime will be a fair amount of extra work that we might not really need.

Another way I thought this could be implemented it fetching each dm_crypt item (until there are no more) and appending that to an array and then for looping that array to do the mounting.

I am not sure what the most C way of doing this would be, so I look forward to seeing what the solution is.

@mattludwigs mattludwigs requested a review from fhunleth November 27, 2019 20:46
@fhunleth fhunleth force-pushed the config-many-encrypted-devices branch 3 times, most recently from ead5f1b to 671d0ca Compare December 14, 2019 22:10
@fhunleth fhunleth force-pushed the config-many-encrypted-devices branch 2 times, most recently from ee2c08a to 1c71287 Compare January 8, 2020 20:16
@fhunleth fhunleth force-pushed the config-many-encrypted-devices branch 3 times, most recently from 78e6a04 to d365db9 Compare January 20, 2020 03:17
@fhunleth fhunleth force-pushed the config-many-encrypted-devices branch from d365db9 to af176ed Compare January 22, 2020 23:07
@fhunleth fhunleth force-pushed the config-many-encrypted-devices branch from af176ed to 6055e53 Compare March 31, 2020 17:51
@fhunleth fhunleth changed the base branch from master to main June 17, 2020 22:36
@fhunleth
Copy link
Copy Markdown
Member

fhunleth commented Aug 9, 2021

I had forgotten about this PR, but if anyone comes back to it, the loop device parts need to be removed.

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.

2 participants