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 <s...@mess.org>
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 <s...@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>

 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
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to