Commit b3c6de49 authored by Julia Lawall's avatar Julia Lawall Committed by Konrad Rzeszutek Wilk
Browse files

cleancache: constify cleancache_ops structure 



The cleancache_ops structure is never modified, so declare it as const.

Done with the help of Coccinelle.

Signed-off-by: default avatarJulia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: default avatarKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
parent afd2ff9b

drivers/xen/tmem.c

+1 −1
Original line number Diff line number Diff line
@@ -242,7 +242,7 @@ static int tmem_cleancache_init_shared_fs(char *uuid, size_t pagesize) 
	return xen_tmem_new_pool(shared_uuid, TMEM_POOL_SHARED, pagesize);

}



static struct cleancache_ops tmem_cleancache_ops = {

static const struct cleancache_ops tmem_cleancache_ops = {

	.put_page = tmem_cleancache_put_page,

	.get_page = tmem_cleancache_get_page,

	.invalidate_page = tmem_cleancache_flush_page,

include/linux/cleancache.h

+1 −1
Original line number Diff line number Diff line
@@ -37,7 +37,7 @@ struct cleancache_ops { 
	void (*invalidate_fs)(int);

};



extern int cleancache_register_ops(struct cleancache_ops *ops);

extern int cleancache_register_ops(const struct cleancache_ops *ops);

extern void __cleancache_init_fs(struct super_block *);

extern void __cleancache_init_shared_fs(struct super_block *);

extern int  __cleancache_get_page(struct page *);

mm/cleancache.c

+2 −2
Original line number Diff line number Diff line
@@ -22,7 +22,7 @@ 
 * cleancache_ops is set by cleancache_register_ops to contain the pointers

 * to the cleancache "backend" implementation functions.

 */

static struct cleancache_ops *cleancache_ops __read_mostly;

static const struct cleancache_ops *cleancache_ops __read_mostly;



/*

 * Counters available via /sys/kernel/debug/cleancache (if debugfs is
@@ -49,7 +49,7 @@ static void cleancache_register_ops_sb(struct super_block *sb, void *unused) 
/*

 * Register operations for cleancache. Returns 0 on success.

 */

int cleancache_register_ops(struct cleancache_ops *ops)

int cleancache_register_ops(const struct cleancache_ops *ops)

{

	if (cmpxchg(&cleancache_ops, NULL, ops))

		return -EBUSY;