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

Add BOOT and DHCP event structure #120

Merged
merged 2 commits into from
Jun 20, 2024
Merged

Add BOOT and DHCP event structure #120

merged 2 commits into from
Jun 20, 2024

Conversation

jknpark
Copy link
Collaborator

@jknpark jknpark commented Jun 19, 2024

다른 이벤트 구조와 동일한 방식으로 정의했는데

  • 다른 채널로 전송해야 하는지 확인
  • 필요한 정보 필드 확인
    바랍니다.

@jknpark jknpark requested a review from syncpark June 19, 2024 06:45
@syncpark
Copy link
Contributor

@jknpark
다른 채널로 전송이라는 의미는 Piglet이 한 코어에서 DHCP 메시지를 받은 후 다른 코어의 쓰레드에게 정보를 전달한다는 의미일까요? FTP처럼?

아니면 DHCP 메시지들을 Message type 단위로 이벤트를 다른 구조체로 분리한다는 의미일까요? 만약 이쪽이라면 굳이 분리할 필요는 없을 것 같습니다.

DHCP나 BOOTP 패킷의 request/response를 하나로 묶어서 이벤트를 생성하지 않고, 각각 개별적으로 DHCP/BOOTP 메시지를 생성하는게 목적입니다. 모든 패킷이 독립적으로 이벤트를 생성, 즉 Request 이벤트와 Response 이벤트가 각각 생성되어야 합니다.

메시지 구조체는 지금처럼 하나로 통일하는게 좋을 듯 합니다.

@kimhanbeom kimhanbeom merged commit 3aedc75 into main Jun 20, 2024
8 checks passed
@kimhanbeom kimhanbeom deleted the bootp branch June 20, 2024 02:09
@jknpark
Copy link
Collaborator Author

jknpark commented Jun 21, 2024

@syncpark Statistics 처럼 별도로 보내야 하는지 궁금한 것이옸습니다.

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.

3 participants