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

Migration towards new ASN compiler #15317

Closed
panyogesh opened this issue Oct 10, 2023 · 8 comments
Closed

Migration towards new ASN compiler #15317

panyogesh opened this issue Oct 10, 2023 · 8 comments
Labels
type: proposal Proposals and design documents

Comments

@panyogesh
Copy link
Contributor

Problem

The current compiler used in Magma has several shortcomings and is occasionally unable to generate code capable of parsing messages sent by gNB #14125 and #12846. Additionally, there are several other messages that failed to parse due to the compiler.

Solution

In order to address the existing limitations and to overcome future obstacles, we are proposing a transition to a new compiler that includes pre-generated files for both 4G (S1-MME) and 5G (NGAP).

Non-goals

Move towards new compiler

Proposal link

@panyogesh panyogesh added the type: proposal Proposals and design documents label Oct 10, 2023
@panyogesh
Copy link
Contributor Author

Current 1.9 release will be using the existing compiler.

@lucasgonze
Copy link
Contributor

lucasgonze commented Oct 25, 2023

I agree that we need to migrate away from the old compiler, which isn't well maintained.

@jorgevargasmor Feel like a taking a look at the security of Mouse07410?

@panyogesh
Copy link
Contributor Author

panyogesh commented Oct 30, 2023

@lucasgonze @jorgevargasmor: We have checked Mouse07410. As such there is no security specific framework that's present.
Few advantage we can see is :

  • Actively Maintained
  • Supported by other open sources
  • Support till release 17.

I guess we need to conclude on how to proceed.

@lucasgonze
Copy link
Contributor

lucasgonze commented Nov 8, 2023

Problem: there are a number of unfixed issues in the proposed new compiler. These are severe enough to matter. The question is whether these are any worse than known issues in the old compiler. We need to find the issue queue for it.

Update: the community around the new compiler is actively working on a fix.

@lucasgonze
Copy link
Contributor

Updates:

  1. OAI has also switched to the mouse compiler
  2. The mouse community has pushed a fix to their buffer overflow

@rdefosse
Copy link
Contributor

rdefosse commented Nov 21, 2023

Updates:

1. OAI has also switched to the mouse compiler

2. The mouse community has pushed a fix to their buffer overflow

Confirmed.

And we also considered the Nokia version.

@rdefosse
Copy link
Contributor

I agree to the move.

1 similar comment
@lucasgonze
Copy link
Contributor

I agree to the move.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: proposal Proposals and design documents
Projects
None yet
Development

No branches or pull requests

3 participants