staging: ion: WARN when the handle kmap_cnt is going to wrap around
authorMitchel Humpherys <mitchelh@codeaurora.org>
Fri, 23 May 2014 20:01:22 +0000 (13:01 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 May 2014 22:23:15 +0000 (07:23 +0900)
commit22f6b9789aef2e0b0b409b1d7708cc653540248c
treed8ce2a8e49c13bffd4c240fe4619e9550b5300a4
parent7949f30a36550b9feeedff10bb4b156b372c6e53
staging: ion: WARN when the handle kmap_cnt is going to wrap around

There are certain client bugs (double unmap, for example) that can cause
the handle->kmap_cnt (an unsigned int) to wrap around from zero. This
causes problems when the handle is destroyed because we have:

        while (handle->kmap_cnt)
                ion_handle_kmap_put(handle);

which takes a long time to complete when kmap_cnt starts at ~0 and can
result in a watchdog timeout.

WARN and bail when kmap_cnt is about to wrap around from zero.

Signed-off-by: Mitchel Humpherys <mitchelh@codeaurora.org>
Acked-by: Colin Cross <ccross@android.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/android/ion/ion.c