Re: [PATCH] slob_free:free objects to their own list

Pekka Enberg
2010-07-13T17:52:12Z

Hi Bob,

[ Please CC me on SLOB patches. You can use the 'scripts/get_maintainer.pl'
script to figure out automatically who to CC on your patches. ]

On Sat, Jul 10, 2010 at 1:05 PM, Bob Liu <lliubbo@gmail.com> wrote:
> slob has alloced smaller objects from their own list in reduce
> overall external fragmentation and increase repeatability,
> free to their own list also.
>
> Signed-off-by: Bob Liu <lliubbo@gmail.com>

The patch looks sane to me. Matt, does it look OK to you as well?

It would be nice to have some fragmentation numbers for this. One
really simple test case is to grep for MemTotal and MemFree in
/proc/meminfo. I'd expect to see some small improvement with your
patch applied. Quantifying long term fragmentation would be even
better but I don't have a good test case for that so I'm CC'ing Mel.

> ---
>  mm/slob.c |    9 ++++++++-
>  1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/mm/slob.c b/mm/slob.c
> index 3f19a34..d582171 100644
> --- a/mm/slob.c
> +++ b/mm/slob.c
> @@ -396,6 +396,7 @@ static void slob_free(void *block, int size)
>        slob_t *prev, *next, *b = (slob_t *)block;
>        slobidx_t units;
>        unsigned long flags;
> +       struct list_head *slob_list;
>
>        if (unlikely(ZERO_OR_NULL_PTR(block)))
>                return;
> @@ -424,7 +425,13 @@ static void slob_free(void *block, int size)
>                set_slob(b, units,
>                        (void *)((unsigned long)(b +
>                                        SLOB_UNITS(PAGE_SIZE)) & PAGE_MASK));
> -               set_slob_page_free(sp, &free_slob_small);
> +               if (size < SLOB_BREAK1)
> +                       slob_list = &free_slob_small;
> +               else if (size < SLOB_BREAK2)
> +                       slob_list = &free_slob_medium;
> +               else
> +                       slob_list = &free_slob_large;
> +               set_slob_page_free(sp, slob_list);
>                goto out;
>        }
>
> --
> 1.5.6.3
>
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@kvack.org.  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: email@kvack.org
>