Hi Ranjit.
I’m opening up this conversation to everyone by cc’ing the spdk mail list.
Yes, the upcoming changes we have will complete the separation of dpdk and spdk.
They will also abstract all of the posix dependencies (e.g. pthread_create becomes
spdk_thread_create).
The question about how many components to refactor needs to be agreed upon.
At this point the feedback I’ve received from Ben and Daniel has been that they don’t want
to convert all components at once. So I’ve been taking my pull requests apart into
smaller pieces. Here is one of my older pull requests for reference.
https://github.com/spdk/spdk/pull/132/commits
<
https://github.com/spdk/spdk/pull/132/commits>
This can be compared with my current pull request:
SPDK EAL Abstractions <
https://github.com/spdk/spdk/pull/150>
and with yours.
Abstract and move rte_ functions into spdk functions that act ad pass-through functions in
env_dpdk <
https://github.com/spdk/spdk/pull/142>
In terms of timeline, I’d like to complete these changes ASAP. If we can’t merge
everything we have into master relatively soon (like this week) then I’d like to propose
that a separate SPDK "platform_abstraction" branch be created (not a Github
fork) so all of the environmental changes can be submitted there. This will allow
contributors to develop these changes away from master more quickly, and the process of
merging SHAs from the “platform_abstraction” into master can go more slowly.
As was discussed at the Intel Builders conference last week, we should set up a periodic
conference call to others in the community can discuss this and other subjects. I’ll
work with Jim, Ben, and Daniel to set that up.
John Meneghini
john.a.meneghini(a)gmail.com
On Apr 26, 2017, at 2:30 AM, Ranjit Noronha
<notifications(a)github.com> wrote:
Hi John,
Sure, lets work together on this.
Will your upcoming changes complete the separation of dpdk and spdk? Or
will there be components still left to refactor?
Do you have a ballpark timeline on when these commits will complete?
thanks,
--ranjit
On 4/26/2017 3:39 AM, John Meneghini wrote:
>
> *@johnmeneghini* commented on this pull request.
>
> Hi Ranjit.
>
> Some of these changes conflict with the changes in my pull requests
> at: #136
>
<
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_spdk_spdk...
> #148
>
<
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_spdk_spdk...
> #150
>
<
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_spdk_spdk...
>
> As you can see, I've refactored my pull request several times, and I
> have many more commits waiting to go in.
>
> I spoke to Ben about this last week and I think our changes
> percipitated Ben's email on the reflector. Let's see if we can work
> together to get these changes in.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
>
<
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_spdk_spdk...;,
> or mute the thread
>
<
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notificat...;.
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/spdk/spdk/pull/142#issuecomment-297252727>, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AMVL3Rosso6EsrPuJojLNae...;.