HID: i2c-hid: remove I2C_HID_READ_PENDING flag to prevent lock-up

mainline inclusion
from mainline-v6.9-rc6
commit 9c0f59e47a90c54d0153f8ddc0f80d7a36207d0e
category: bugfix
bugzilla: https://gitee.com/src-openeuler/kernel/issues/I9QRN6
CVE: CVE-2024-35997

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=9c0f59e47a90c54d0153f8ddc0f80d7a36207d0e



--------------------------------

The flag I2C_HID_READ_PENDING is used to serialize I2C operations.
However, this is not necessary, because I2C core already has its own
locking for that.

More importantly, this flag can cause a lock-up: if the flag is set in
i2c_hid_xfer() and an interrupt happens, the interrupt handler
(i2c_hid_irq) will check this flag and return immediately without doing
anything, then the interrupt handler will be invoked again in an
infinite loop.

Since interrupt handler is an RT task, it takes over the CPU and the
flag-clearing task never gets scheduled, thus we have a lock-up.

Delete this unnecessary flag.

Reported-and-tested-by: Eva Kurchatova <nyandarknessgirl@gmail.com>
Closes: https://lore.kernel.org/r/CA+eeCSPUDpUg76ZO8dszSbAGn+UHjcyv8F1J-CUPVARAzEtW9w@mail.gmail.com


Fixes: 4a200c3b ("HID: i2c-hid: introduce HID over i2c specification implementation")
Cc: <stable@vger.kernel.org>
Signed-off-by: Nam Cao <namcao@linutronix.de>
Signed-off-by: Jiri Kosina <jkosina@suse.com>
Conflicts:
	drivers/hid/i2c-hid/i2c-hid-core.c
[yyl: adjust context]
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>