On Mon, 18 Dec 2023 15:46:18 +0000 Vincent Donnefort <vdonnef...@google.com> wrote:
> Previously was introduced the ability to change the ring-buffer page > size. It also introduced the concept of sub-buffer that is, a contiguous > virtual memory space which can now be bigger than the system page size > (4K on most systems). But behind the scene this is really just a page > with an order > 0 and a struct buffer_page (often refered as "bpage") > already exists. We have then an unnecessary duplicate subbuffer == > bpage. > > Remove all references to sub-buffer and replace them with either bpage > or ring_buffer_page. No! I think you misunderstood me before. I want it the other way around. In Linux, "page" usually means the architecture page (4096 on x86). If we are going to remove the duplicate of subbuffer == bpage, remove the "page" part, *not* the subbuffer part. The "subbuf" part is part of the design, not an implementation detail. The page part, was just an implementation detail that we do not want to expose! When you say "page" you mean a single page. If you say buffer_page_size, one will think this is an architecture page, not the subbuffer size. The ring buffer is broken up into subbuffers, The page is just an implementation detail. Let's not confuse that with the design. Thanks, -- Steve
