Commit 713c4ff8 authored by Borislav Petkov's avatar Borislav Petkov
Browse files

EDAC/mc: Get rid of edac_align_ptr() 



Get rid of it now that it is unused.

Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
Link: https://lore.kernel.org/r/20220310095254.1510-6-bp@alien8.de
parent 0d24a49e

drivers/edac/edac_mc.c

+0 −55
Original line number Diff line number Diff line
@@ -170,61 +170,6 @@ const char * const edac_mem_types[] = { 
};

EXPORT_SYMBOL_GPL(edac_mem_types);



/**

 * edac_align_ptr - Prepares the pointer offsets for a single-shot allocation

 * @p:		pointer to a pointer with the memory offset to be used. At

 *		return, this will be incremented to point to the next offset

 * @size:	Size of the data structure to be reserved

 * @n_elems:	Number of elements that should be reserved

 *

 * If 'size' is a constant, the compiler will optimize this whole function

 * down to either a no-op or the addition of a constant to the value of '*p'.

 *

 * The 'p' pointer is absolutely needed to keep the proper advancing

 * further in memory to the proper offsets when allocating the struct along

 * with its embedded structs, as edac_device_alloc_ctl_info() does it

 * above, for example.

 *

 * At return, the pointer 'p' will be incremented to be used on a next call

 * to this function.

 */

void *edac_align_ptr(void **p, unsigned int size, int n_elems)

{

	unsigned int align, r;

	void *ptr = *p;



	*p += size * n_elems;



	/*

	 * 'p' can possibly be an unaligned item X such that sizeof(X) is

	 * 'size'.  Adjust 'p' so that its alignment is at least as

	 * stringent as what the compiler would provide for X and return

	 * the aligned result.

	 * Here we assume that the alignment of a "long long" is the most

	 * stringent alignment that the compiler will ever provide by default.

	 * As far as I know, this is a reasonable assumption.

	 */

	if (size > sizeof(long))

		align = sizeof(long long);

	else if (size > sizeof(int))

		align = sizeof(long);

	else if (size > sizeof(short))

		align = sizeof(int);

	else if (size > sizeof(char))

		align = sizeof(short);

	else

		return ptr;



	r = (unsigned long)ptr % align;



	if (r == 0)

		return ptr;



	*p += align - r;



	return (void *)(((unsigned long)ptr) + align - r);

}



static void _edac_mc_free(struct mem_ctl_info *mci)

{

	put_device(&mci->dev);

drivers/edac/edac_module.h

+0 −2
Original line number Diff line number Diff line
@@ -59,8 +59,6 @@ extern void edac_device_reset_delay_period(struct edac_device_ctl_info 
					   *edac_dev, unsigned long value);

extern void edac_mc_reset_delay_period(unsigned long value);



extern void *edac_align_ptr(void **p, unsigned size, int n_elems);



/*

 * EDAC debugfs functions

 */