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

[Pekka Enberg]

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@xxxxxxxxx> 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@xxxxxxxxx>

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@xxxxxxxxxx  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/