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

[WIP] Implement bs_load. #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP] Implement bs_load. #7

wants to merge 1 commit into from

Conversation

jkozlowski
Copy link
Owner

  • Also trying to figure out how to test it but currently running into the fact that dpdk can only be initialised once per process.
  • Therefore I'm experimenting with a fork per test, but this is likely going to fail miserably...

@gila
Copy link

gila commented Nov 23, 2018

Out of interest -- why are you trying to create multiple instances, to begin with? Would it not be possible to allocate an event, and dispatch that event to the reactor (the SPDK reactor that is). Each event could exist out of a chain of functions/futures (that should be small to prevent blocking other events)

Not much unlike the initial callback passed to spdk_app_start() spdk_event_allocate() can be used to kick off work as wanted. Your rust skills greatly surpass mine but maybe something like:

event.alloc( || {
 // whatever I want to run on the reactor and in the context of the reactor threads
}

@jkozlowski
Copy link
Owner Author

Yeah, that wasn't my problem. I wanted to be able to have a clean slate before each test so I can read-write whatever I want. But now that I think about it, I guess I could just generate unique backends for each test and that could work.

Anyway, something to try at some point.

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.

None yet

2 participants