On Fri, 2023-01-20 at 23:46 +0000, Teres Alexis, Alan Previn wrote: > Thanks for the review comment... > Replying here with the summary of our offline chat: So we concluded that for the next rev, lets move the buffer/vma allocations from patch #2 into this patch #5-send-message. But for now we shall keep patch #6-create-session as separate because when future terminate-session joins this series, it will join #6. That said, #5 and #6 togther would then appear rather large. However, we both agreed that we can always make the final call to squash all of those together (alloc/free + send-message + create-session + terminate-session) at the end when we are merging if we see fit. That would be easy and painless (as opposed to squashing now and then getting a request to re-split later). NOTE: all numbering above will appear as n-minus-1 on next rev because Patch-#1 is going away (the stubs). thanks again...alan
