Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 955dc7ff authored by Linux Build Service Account's avatar Linux Build Service Account Committed by Gerrit - the friendly Code Review server
Browse files

Merge "USB: f_fs: Add memory barrier before atomic operations"

parents 80e46a9e c6c0a866
Loading
Loading
Loading
Loading
+7 −2
Original line number Diff line number Diff line
@@ -699,6 +699,7 @@ static int ffs_ep0_open(struct inode *inode, struct file *file)
	if (unlikely(ffs->state == FFS_CLOSING))
		return -EBUSY;

	smp_mb__before_atomic();
	if (atomic_read(&ffs->opened))
		return -EBUSY;

@@ -1344,6 +1345,7 @@ static void ffs_data_get(struct ffs_data *ffs)
{
	ENTER();

	smp_mb__before_atomic();
	atomic_inc(&ffs->ref);
}

@@ -1351,6 +1353,7 @@ static void ffs_data_opened(struct ffs_data *ffs)
{
	ENTER();

	smp_mb__before_atomic();
	atomic_inc(&ffs->ref);
	atomic_inc(&ffs->opened);
}
@@ -1359,6 +1362,7 @@ static void ffs_data_put(struct ffs_data *ffs)
{
	ENTER();

	smp_mb__before_atomic();
	if (unlikely(atomic_dec_and_test(&ffs->ref))) {
		pr_info("%s(): freeing\n", __func__);
		ffs_data_clear(ffs);
@@ -1373,6 +1377,7 @@ static void ffs_data_closed(struct ffs_data *ffs)
{
	ENTER();

	smp_mb__before_atomic();
	if (atomic_dec_and_test(&ffs->opened)) {
		ffs->state = FFS_CLOSING;
		ffs_data_reset(ffs);
@@ -1416,8 +1421,8 @@ static void ffs_data_clear(struct ffs_data *ffs)

	/* Dump ffs->gadget and ffs->flags */
	if (ffs->gadget)
		pr_err("%s: ffs->gadget= %p, ffs->flags= %lu\n", __func__,
						ffs->gadget, ffs->flags);
		pr_err("%s: ffs:%p ffs->gadget= %p, ffs->flags= %lu\n",
				__func__, ffs, ffs->gadget, ffs->flags);
	BUG_ON(ffs->gadget);

	if (ffs->epfiles)