projects
/
deliverable
/
lttng-modules.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Performance: cache the backend pages pointer in context
[deliverable/lttng-modules.git]
/
lib
/
ringbuffer
/
backend.h
diff --git
a/lib/ringbuffer/backend.h
b/lib/ringbuffer/backend.h
index b362a04a436bfe64c640f694968032fb7de5aa32..9db0095638973a2e0fb6da77affde9fb30bed8f5 100644
(file)
--- a/
lib/ringbuffer/backend.h
+++ b/
lib/ringbuffer/backend.h
@@
-90,26
+90,20
@@
void lib_ring_buffer_write(const struct lib_ring_buffer_config *config,
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
- size_t
sbidx,
index, pagecpy;
+ size_t index, pagecpy;
size_t offset = ctx->buf_offset;
size_t offset = ctx->buf_offset;
- struct lib_ring_buffer_backend_pages *rpages;
- unsigned long sb_bindex, id;
+ struct lib_ring_buffer_backend_pages *backend_pages;
if (unlikely(!len))
return;
if (unlikely(!len))
return;
+ backend_pages =
+ lib_ring_buffer_get_backend_pages_from_ctx(config, ctx);
offset &= chanb->buf_size - 1;
offset &= chanb->buf_size - 1;
- sbidx = offset >> chanb->subbuf_size_order;
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
- id = bufb->buf_wsb[sbidx].id;
- sb_bindex = subbuffer_id_get_index(config, id);
- rpages = bufb->array[sb_bindex];
- CHAN_WARN_ON(ctx->chan,
- config->mode == RING_BUFFER_OVERWRITE
- && subbuffer_id_is_noref(config, id));
if (likely(pagecpy == len))
lib_ring_buffer_do_copy(config,
if (likely(pagecpy == len))
lib_ring_buffer_do_copy(config,
-
r
pages->p[index].virt
+
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
src, len);
else
+ (offset & ~PAGE_MASK),
src, len);
else
@@
-137,25
+131,19
@@
void lib_ring_buffer_memset(const struct lib_ring_buffer_config *config,
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
- size_t
sbidx,
index, pagecpy;
+ size_t index, pagecpy;
size_t offset = ctx->buf_offset;
size_t offset = ctx->buf_offset;
- struct lib_ring_buffer_backend_pages *rpages;
- unsigned long sb_bindex, id;
+ struct lib_ring_buffer_backend_pages *backend_pages;
if (unlikely(!len))
return;
if (unlikely(!len))
return;
+ backend_pages =
+ lib_ring_buffer_get_backend_pages_from_ctx(config, ctx);
offset &= chanb->buf_size - 1;
offset &= chanb->buf_size - 1;
- sbidx = offset >> chanb->subbuf_size_order;
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
- id = bufb->buf_wsb[sbidx].id;
- sb_bindex = subbuffer_id_get_index(config, id);
- rpages = bufb->array[sb_bindex];
- CHAN_WARN_ON(ctx->chan,
- config->mode == RING_BUFFER_OVERWRITE
- && subbuffer_id_is_noref(config, id));
if (likely(pagecpy == len))
if (likely(pagecpy == len))
- lib_ring_buffer_do_memset(
r
pages->p[index].virt
+ lib_ring_buffer_do_memset(
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
c, len);
else
+ (offset & ~PAGE_MASK),
c, len);
else
@@
-240,28
+228,22
@@
void lib_ring_buffer_strcpy(const struct lib_ring_buffer_config *config,
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
- size_t
sbidx,
index, pagecpy;
+ size_t index, pagecpy;
size_t offset = ctx->buf_offset;
size_t offset = ctx->buf_offset;
- struct lib_ring_buffer_backend_pages *rpages;
- unsigned long sb_bindex, id;
+ struct lib_ring_buffer_backend_pages *backend_pages;
if (unlikely(!len))
return;
if (unlikely(!len))
return;
+ backend_pages =
+ lib_ring_buffer_get_backend_pages_from_ctx(config, ctx);
offset &= chanb->buf_size - 1;
offset &= chanb->buf_size - 1;
- sbidx = offset >> chanb->subbuf_size_order;
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
- id = bufb->buf_wsb[sbidx].id;
- sb_bindex = subbuffer_id_get_index(config, id);
- rpages = bufb->array[sb_bindex];
- CHAN_WARN_ON(ctx->chan,
- config->mode == RING_BUFFER_OVERWRITE
- && subbuffer_id_is_noref(config, id));
if (likely(pagecpy == len)) {
size_t count;
count = lib_ring_buffer_do_strcpy(config,
if (likely(pagecpy == len)) {
size_t count;
count = lib_ring_buffer_do_strcpy(config,
-
r
pages->p[index].virt
+
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
src, len - 1);
offset += count;
+ (offset & ~PAGE_MASK),
src, len - 1);
offset += count;
@@
-269,13
+251,13
@@
void lib_ring_buffer_strcpy(const struct lib_ring_buffer_config *config,
if (unlikely(count < len - 1)) {
size_t pad_len = len - 1 - count;
if (unlikely(count < len - 1)) {
size_t pad_len = len - 1 - count;
- lib_ring_buffer_do_memset(
r
pages->p[index].virt
+ lib_ring_buffer_do_memset(
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
pad, pad_len);
offset += pad_len;
}
/* Ending '\0' */
+ (offset & ~PAGE_MASK),
pad, pad_len);
offset += pad_len;
}
/* Ending '\0' */
- lib_ring_buffer_do_memset(
r
pages->p[index].virt
+ lib_ring_buffer_do_memset(
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
'\0', 1);
} else {
+ (offset & ~PAGE_MASK),
'\0', 1);
} else {
@@
-304,25
+286,19
@@
void lib_ring_buffer_copy_from_user_inatomic(const struct lib_ring_buffer_config
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
- size_t
sbidx,
index, pagecpy;
+ size_t index, pagecpy;
size_t offset = ctx->buf_offset;
size_t offset = ctx->buf_offset;
- struct lib_ring_buffer_backend_pages *rpages;
- unsigned long sb_bindex, id;
+ struct lib_ring_buffer_backend_pages *backend_pages;
unsigned long ret;
mm_segment_t old_fs = get_fs();
if (unlikely(!len))
return;
unsigned long ret;
mm_segment_t old_fs = get_fs();
if (unlikely(!len))
return;
+ backend_pages =
+ lib_ring_buffer_get_backend_pages_from_ctx(config, ctx);
offset &= chanb->buf_size - 1;
offset &= chanb->buf_size - 1;
- sbidx = offset >> chanb->subbuf_size_order;
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
- id = bufb->buf_wsb[sbidx].id;
- sb_bindex = subbuffer_id_get_index(config, id);
- rpages = bufb->array[sb_bindex];
- CHAN_WARN_ON(ctx->chan,
- config->mode == RING_BUFFER_OVERWRITE
- && subbuffer_id_is_noref(config, id));
set_fs(KERNEL_DS);
pagefault_disable();
set_fs(KERNEL_DS);
pagefault_disable();
@@
-331,7
+307,7
@@
void lib_ring_buffer_copy_from_user_inatomic(const struct lib_ring_buffer_config
if (likely(pagecpy == len)) {
ret = lib_ring_buffer_do_copy_from_user_inatomic(
if (likely(pagecpy == len)) {
ret = lib_ring_buffer_do_copy_from_user_inatomic(
-
r
pages->p[index].virt + (offset & ~PAGE_MASK),
+
backend_
pages->p[index].virt + (offset & ~PAGE_MASK),
src, len);
if (unlikely(ret > 0)) {
/* Copy failed. */
src, len);
if (unlikely(ret > 0)) {
/* Copy failed. */
@@
-381,24
+357,18
@@
void lib_ring_buffer_strcpy_from_user_inatomic(const struct lib_ring_buffer_conf
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
{
struct lib_ring_buffer_backend *bufb = &ctx->buf->backend;
struct channel_backend *chanb = &ctx->chan->backend;
- size_t
sbidx,
index, pagecpy;
+ size_t index, pagecpy;
size_t offset = ctx->buf_offset;
size_t offset = ctx->buf_offset;
- struct lib_ring_buffer_backend_pages *rpages;
- unsigned long sb_bindex, id;
+ struct lib_ring_buffer_backend_pages *backend_pages;
mm_segment_t old_fs = get_fs();
if (unlikely(!len))
return;
mm_segment_t old_fs = get_fs();
if (unlikely(!len))
return;
+ backend_pages =
+ lib_ring_buffer_get_backend_pages_from_ctx(config, ctx);
offset &= chanb->buf_size - 1;
offset &= chanb->buf_size - 1;
- sbidx = offset >> chanb->subbuf_size_order;
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
index = (offset & (chanb->subbuf_size - 1)) >> PAGE_SHIFT;
pagecpy = min_t(size_t, len, (-offset) & ~PAGE_MASK);
- id = bufb->buf_wsb[sbidx].id;
- sb_bindex = subbuffer_id_get_index(config, id);
- rpages = bufb->array[sb_bindex];
- CHAN_WARN_ON(ctx->chan,
- config->mode == RING_BUFFER_OVERWRITE
- && subbuffer_id_is_noref(config, id));
set_fs(KERNEL_DS);
pagefault_disable();
set_fs(KERNEL_DS);
pagefault_disable();
@@
-409,7
+379,7
@@
void lib_ring_buffer_strcpy_from_user_inatomic(const struct lib_ring_buffer_conf
size_t count;
count = lib_ring_buffer_do_strcpy_from_user_inatomic(config,
size_t count;
count = lib_ring_buffer_do_strcpy_from_user_inatomic(config,
-
r
pages->p[index].virt
+
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
src, len - 1);
offset += count;
+ (offset & ~PAGE_MASK),
src, len - 1);
offset += count;
@@
-417,13
+387,13
@@
void lib_ring_buffer_strcpy_from_user_inatomic(const struct lib_ring_buffer_conf
if (unlikely(count < len - 1)) {
size_t pad_len = len - 1 - count;
if (unlikely(count < len - 1)) {
size_t pad_len = len - 1 - count;
- lib_ring_buffer_do_memset(
r
pages->p[index].virt
+ lib_ring_buffer_do_memset(
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
pad, pad_len);
offset += pad_len;
}
/* Ending '\0' */
+ (offset & ~PAGE_MASK),
pad, pad_len);
offset += pad_len;
}
/* Ending '\0' */
- lib_ring_buffer_do_memset(
r
pages->p[index].virt
+ lib_ring_buffer_do_memset(
backend_
pages->p[index].virt
+ (offset & ~PAGE_MASK),
'\0', 1);
} else {
+ (offset & ~PAGE_MASK),
'\0', 1);
} else {
This page took
0.028152 seconds
and
5
git commands to generate.