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

docs: add owner of ApicEmul item #371

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

vijaydhanraj
Copy link
Contributor

@vijaydhanraj vijaydhanraj commented May 29, 2024

  • Add myself as the owner of the ApicEmul item, to add vXAPIC and vX2APIC support for TD partitioned guest.

@msft-jlange Please let me know if you plan to work on this item and we can co-own this item.

@msft-jlange
Copy link
Collaborator

  • Add myself as the owner of the ApicEmul item, to add vXAPIC and vX2APIC support for guest OS.
    @msft-jlange Please let me know if you plan to work on this item and we can co-own this item.

I don't know what "owner" is supposed to mean, but I am doing active development in this area and will continue to do so. We should coordinate if you want to be involved too.

@vijaydhanraj
Copy link
Contributor Author

Owners are people assigned for a particular task/sub-task in the development plan.
Yes, sure I would like to be involved as well. Please let me know how we can get started.

Currently I am following your PRs on this and seeing if I can apply/extend it to TD partitioning.

@joergroedel
Copy link
Member

Hi Vijay, some of the CI checks failed, please add a message body to all commits to make the tests pass.

@joergroedel joergroedel added the wait-for-update PR is waiting to be updated to address review comments label Jun 5, 2024
Add myself as the owner of the ApicEmul item, to implement
vXAPIC and vX2APIC support for TD partitioned guest.

Signed-off-by: Vijay Dhanraj <[email protected]>
@vijaydhanraj
Copy link
Contributor Author

Hi Joerg, I have added the body and also updated the commit to add @msft-jlange as owner. Please let me know if anything else is needed. Thx.

@joergroedel joergroedel merged commit 8442d27 into coconut-svsm:main Jun 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wait-for-update PR is waiting to be updated to address review comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants