[PATCH] ring-buffer: fix free page - Kernel

This is a discussion on [PATCH] ring-buffer: fix free page - Kernel ; The pages of a buffer was originally pointing to the page struct, it now points to the page address. The freeing of the page still uses the page frame free "__free_page" instead of the correct free_page to the address. Signed-off-by: ...

+ Reply to Thread
Results 1 to 2 of 2

Thread: [PATCH] ring-buffer: fix free page

  1. [PATCH] ring-buffer: fix free page


    The pages of a buffer was originally pointing to the page struct, it
    now points to the page address. The freeing of the page still uses
    the page frame free "__free_page" instead of the correct free_page to
    the address.

    Signed-off-by: Steven Rostedt
    ---
    kernel/trace/ring_buffer.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    Index: linux-tip.git/kernel/trace/ring_buffer.c
    ================================================== =================
    --- linux-tip.git.orig/kernel/trace/ring_buffer.c 2008-10-13 10:15:16.000000000 -0400
    +++ linux-tip.git/kernel/trace/ring_buffer.c 2008-10-13 10:17:03.000000000 -0400
    @@ -130,7 +130,7 @@ struct buffer_page {
    static inline void free_buffer_page(struct buffer_page *bpage)
    {
    if (bpage->page)
    - __free_page(bpage->page);
    + free_page((unsigned long)bpage->page);
    kfree(bpage);
    }


    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  2. Re: [PATCH] ring-buffer: fix free page


    * Steven Rostedt wrote:

    >
    > The pages of a buffer was originally pointing to the page struct, it
    > now points to the page address. The freeing of the page still uses
    > the page frame free "__free_page" instead of the correct free_page to
    > the address.
    >
    > Signed-off-by: Steven Rostedt
    > ---
    > kernel/trace/ring_buffer.c | 2 +-
    > 1 file changed, 1 insertion(+), 1 deletion(-)


    applied to tip/tracing/urgent, thanks Steve!

    Ingo
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

+ Reply to Thread