d7a1d629c3
The slab initialization code includes the calculation of the slab 'elem_size', which is in turn used to determine the size (capacity) of the slab. Each element of the slab represents an array, of length 'stride', of 'elemtype'. (Note that it may be clearer if the define_commit_slab macro parameter was called 'basetype' rather than 'elemtype'). However, the 'elem_size' calculation incorrectly uses 'sizeof(struct slabname)' in the expression, rather than 'sizeof(elemtype)'. Within the slab access routine, <slabname>_at(), the given commit 'index' is transformed into an (slab#, slot#) pair used to address the required element (a pointer to the first element of the array of 'elemtype' associated with that commit). The current code to calculate these address coordinates multiplies the commit index by the 'stride' which, at least for the slab#, produces the wrong result. Using the commit index directly, without scaling by the 'stride', produces the correct 'logical' address. Also, when allocating a new slab, the size of the allocation only allows for a slab containing elements of single element arrays of 'elemtype'. This should allow for elements of an array of length 'stride' of 'elemtype'. In order to fix this, we need to change the element size parameter to xcalloc() by multiplying the current element size (sizeof(**s->slab)) by the s->stride. Having changed the calculation of the slot#, we now need to convert the logical 'nth_slot', by scaling with s->stride, into the correct physical address. Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk> Signed-off-by: Junio C Hamano <gitster@pobox.com>
98 lines
2.9 KiB
C
98 lines
2.9 KiB
C
#ifndef COMMIT_SLAB_H
|
|
#define COMMIT_SLAB_H
|
|
|
|
/*
|
|
* define_commit_slab(slabname, elemtype) creates boilerplate code to define
|
|
* a new struct (struct slabname) that is used to associate a piece of data
|
|
* of elemtype to commits, and a few functions to use that struct.
|
|
*
|
|
* After including this header file, using:
|
|
*
|
|
* define_commit_slab(indegee, int);
|
|
*
|
|
* will let you call the following functions:
|
|
*
|
|
* - int *indegree_at(struct indegree *, struct commit *);
|
|
*
|
|
* This function locates the data associated with the given commit in
|
|
* the indegree slab, and returns the pointer to it.
|
|
*
|
|
* - void init_indegree(struct indegree *);
|
|
* void init_indegree_with_stride(struct indegree *, int);
|
|
*
|
|
* Initializes the indegree slab that associates an array of integers
|
|
* to each commit. 'stride' specifies how big each array is. The slab
|
|
* that id initialied by the variant without "_with_stride" associates
|
|
* each commit with an array of one integer.
|
|
*/
|
|
|
|
/* allocate ~512kB at once, allowing for malloc overhead */
|
|
#ifndef COMMIT_SLAB_SIZE
|
|
#define COMMIT_SLAB_SIZE (512*1024-32)
|
|
#endif
|
|
|
|
#define define_commit_slab(slabname, elemtype) \
|
|
\
|
|
struct slabname { \
|
|
unsigned slab_size; \
|
|
unsigned stride; \
|
|
unsigned slab_count; \
|
|
elemtype **slab; \
|
|
}; \
|
|
static int stat_ ##slabname## realloc; \
|
|
\
|
|
static void init_ ##slabname## _with_stride(struct slabname *s, \
|
|
unsigned stride) \
|
|
{ \
|
|
unsigned int elem_size; \
|
|
if (!stride) \
|
|
stride = 1; \
|
|
s->stride = stride; \
|
|
elem_size = sizeof(elemtype) * stride; \
|
|
s->slab_size = COMMIT_SLAB_SIZE / elem_size; \
|
|
s->slab_count = 0; \
|
|
s->slab = NULL; \
|
|
} \
|
|
\
|
|
static void init_ ##slabname(struct slabname *s) \
|
|
{ \
|
|
init_ ##slabname## _with_stride(s, 1); \
|
|
} \
|
|
\
|
|
static void clear_ ##slabname(struct slabname *s) \
|
|
{ \
|
|
int i; \
|
|
for (i = 0; i < s->slab_count; i++) \
|
|
free(s->slab[i]); \
|
|
s->slab_count = 0; \
|
|
free(s->slab); \
|
|
s->slab = NULL; \
|
|
} \
|
|
\
|
|
static elemtype *slabname## _at(struct slabname *s, \
|
|
const struct commit *c) \
|
|
{ \
|
|
int nth_slab, nth_slot; \
|
|
\
|
|
nth_slab = c->index / s->slab_size; \
|
|
nth_slot = c->index % s->slab_size; \
|
|
\
|
|
if (s->slab_count <= nth_slab) { \
|
|
int i; \
|
|
s->slab = xrealloc(s->slab, \
|
|
(nth_slab + 1) * sizeof(s->slab)); \
|
|
stat_ ##slabname## realloc++; \
|
|
for (i = s->slab_count; i <= nth_slab; i++) \
|
|
s->slab[i] = NULL; \
|
|
s->slab_count = nth_slab + 1; \
|
|
} \
|
|
if (!s->slab[nth_slab]) \
|
|
s->slab[nth_slab] = xcalloc(s->slab_size, \
|
|
sizeof(**s->slab) * s->stride); \
|
|
return &s->slab[nth_slab][nth_slot * s->stride]; \
|
|
} \
|
|
\
|
|
static int stat_ ##slabname## realloc
|
|
|
|
#endif /* COMMIT_SLAB_H */
|