Commit 7d2da603 authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller
Browse files

nfp: fix nfp_cpp_read()/nfp_cpp_write() error paths



When acquiring an area fails we can't call function doing both
release and free.

Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 1bb665e3
Loading
Loading
Loading
Loading
+9 −6
Original line number Original line Diff line number Diff line
@@ -951,12 +951,14 @@ int nfp_cpp_read(struct nfp_cpp *cpp, u32 destination,
			return -ENOMEM;
			return -ENOMEM;


		err = nfp_cpp_area_acquire(area);
		err = nfp_cpp_area_acquire(area);
		if (err)
		if (err) {
			goto out;
			nfp_cpp_area_free(area);
			return err;
		}
	}
	}


	err = nfp_cpp_area_read(area, offset, kernel_vaddr, length);
	err = nfp_cpp_area_read(area, offset, kernel_vaddr, length);
out:

	if (cache)
	if (cache)
		area_cache_put(cpp, cache);
		area_cache_put(cpp, cache);
	else
	else
@@ -993,13 +995,14 @@ int nfp_cpp_write(struct nfp_cpp *cpp, u32 destination,
			return -ENOMEM;
			return -ENOMEM;


		err = nfp_cpp_area_acquire(area);
		err = nfp_cpp_area_acquire(area);
		if (err)
		if (err) {
			goto out;
			nfp_cpp_area_free(area);
			return err;
		}
	}
	}


	err = nfp_cpp_area_write(area, offset, kernel_vaddr, length);
	err = nfp_cpp_area_write(area, offset, kernel_vaddr, length);


out:
	if (cache)
	if (cache)
		area_cache_put(cpp, cache);
		area_cache_put(cpp, cache);
	else
	else