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

Module configuration/installation issues #448

Open
marcbonnici opened this issue Nov 28, 2019 · 1 comment
Open

Module configuration/installation issues #448

marcbonnici opened this issue Nov 28, 2019 · 1 comment
Labels

Comments

@marcbonnici
Copy link
Collaborator

We currently have no method of specifying dependencies for modules, this can result in them failing to initialise due to an incorrect ordering. Therefore we need to introduce a dependency mechanism to determine if a required module is going to be installed at a later time or whether an error needs to be thrown.

Another issue is there are currently multiple ways of supplying configuration for a module which can conflict. For example a flashing module can be partially configured via the Platform and via user entered module configuration and the priority of which configuration is not clear.

@douglas-raillard-arm
Copy link
Contributor

FWIW the problem is somewhat solved in LISA by loading them on demand whenever the attribute is accessed:
https://github.com/ARM-software/lisa/blob/master/lisa/target.py#L340

This could also be easily exposed on a dedicated proxy object like target.lazy_module.sched in case you want to preserve target.sched behavior.

Note that this won't solve problems across module types, in case they need to be initialized in more than one batch (like if a regular module that wants to depend on a setup module, since the setup module would need to be initialized way earlier).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants