This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: rc: get start time just before calling driver tx
Author:  Sean Young <[email protected]>
Date:    Mon Feb 12 09:00:28 2018 -0500

The current code gets the start time before copying the IR from
userspace (could cause page faults) and encoding IR. This means
that the gap calculation could be off.

Signed-off-by: Sean Young <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/rc/lirc_dev.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

---

diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c
index fd3860d50034..da3b5c095a59 100644
--- a/drivers/media/rc/lirc_dev.c
+++ b/drivers/media/rc/lirc_dev.c
@@ -249,8 +249,6 @@ static ssize_t ir_lirc_transmit_ir(struct file *file, const 
char __user *buf,
                goto out_unlock;
        }
 
-       start = ktime_get();
-
        if (!dev->tx_ir) {
                ret = -EINVAL;
                goto out_unlock;
@@ -343,6 +341,8 @@ static ssize_t ir_lirc_transmit_ir(struct file *file, const 
char __user *buf,
                duration += txbuf[i];
        }
 
+       start = ktime_get();
+
        ret = dev->tx_ir(dev, txbuf, count);
        if (ret < 0)
                goto out_kfree;

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to