Commit c30b20cd authored by Kees Cook's avatar Kees Cook
Browse files

pstore: Rename "pstore_lock" to "psinfo_lock" 

The name "pstore_lock" sounds very global, but it is only supposed to be
used for managing changes to "psinfo", so rename it accordingly.

Link: https://lore.kernel.org/lkml/20200506152114.50375-3-keescook@chromium.org/


Signed-off-by: default avatarKees Cook <keescook@chromium.org>
parent e7c1c00c

fs/pstore/platform.c

+5 −5
Original line number Diff line number Diff line
@@ -69,10 +69,10 @@ static void pstore_dowork(struct work_struct *); 
static DECLARE_WORK(pstore_work, pstore_dowork);



/*

 * pstore_lock just protects "psinfo" during

 * psinfo_lock just protects "psinfo" during

 * calls to pstore_register()

 */

static DEFINE_SPINLOCK(pstore_lock);

static DEFINE_SPINLOCK(psinfo_lock);

struct pstore_info *psinfo;



static char *backend;
@@ -574,11 +574,11 @@ int pstore_register(struct pstore_info *psi) 
		return -EINVAL;

	}



	spin_lock(&pstore_lock);

	spin_lock(&psinfo_lock);

	if (psinfo) {

		pr_warn("backend '%s' already loaded: ignoring '%s'\n",

			psinfo->name, psi->name);

		spin_unlock(&pstore_lock);

		spin_unlock(&psinfo_lock);

		return -EBUSY;

	}
@@ -587,7 +587,7 @@ int pstore_register(struct pstore_info *psi) 
	psinfo = psi;

	mutex_init(&psinfo->read_mutex);

	sema_init(&psinfo->buf_lock, 1);

	spin_unlock(&pstore_lock);

	spin_unlock(&psinfo_lock);





	if (psi->flags & PSTORE_FLAGS_DMESG)

CRA Git | Maintained and supported by SUSTech CRA and CCSE