Skip to content

Conversation

@jmi-reds
Copy link
Contributor

  • vlogs frontend driver
  • logs function in kernel and usr-space

- vlogs frontend driver
- logs function in kernel and usr-space

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds jmi-reds self-assigned this Sep 16, 2025
@jmi-reds jmi-reds linked an issue Sep 16, 2025 that may be closed by this pull request
Copy link
Contributor

@clemdiep clemdiep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fast review, didn't looked deeply in implementation yet. Also, correct formats errors reported by clang-format

Copy link
Contributor

@daniel-rossier daniel-rossier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess we need to understand the relevance of these logs.

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds
Copy link
Contributor Author

the goals of this PR is to add the support of logs when SO3 is used as a Capsule. It means is it only valid when SO3 is used as a capsule in SOO environment: the logs messages are sent to the agency via vlogs frontend/backend and sorted in a specific file (thanks to syslog-ng).

The following picture show an overview of the flow:
micofe-logs

I was to focus on agency side and ported/adapted code from emiso project. I missed to better check what was present in SO3 - the LOG macro is new and not present when emiso was created.

so:

  • The name logs is not great (it came from Docker container logs cmd...). It can be merged with the LOG macro or a new better name to be find, il possible the same/similar name in usr and kernel space (soo_log, capsule_log, clog`, ...)
  • In driver side, yes vlogs is similar to vuart but vuart uses, in Linux, consoles where vlogs does simple printk. It seemed simpler and future proof to create vlogs instead of modify vuart. If vlogs of vuart has to evolve, it will not impact the feature of the other driver.

I will check the new LOG macro and see if and how it can be adapted to support this feature

Side question: Does SO3 have a mechanism to start an application during boot/injection ?

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@daniel-rossier
Copy link
Contributor

Yes, SO3 launch init by default, but we can start another. Look at the config file and kernel/main.c

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Copy link
Contributor

@clemdiep clemdiep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New batch of comments

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds jmi-reds merged commit 27ffea8 into main Sep 26, 2025
11 checks passed
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.

Add logs support in SOO environment

4 participants