| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | ChanServ sets mode: +v mchehab | [00:09] |
| ................ (idle for 1h18mn) | ||
| ChanServ sets mode: +v mchehab | [01:27] | |
| ............................................................................................................................... (idle for 10h32mn) | ||
| mkrufky | ``` (06:58:18 AM) Sigyn: Invitation denied, there are only 61 users in #linuxtv: contact staffers if needed. ```
?!? | [11:59] |
| *** | ChanServ sets mode: +v snawrocki | [11:59] |
| mkrufky | meh, looks like im going to have to get staff involved | [11:59] |
| hverkuil | hi guys! | [12:00] |
| mkrufky | hi hans :-) | [12:00] |
| syoung | Hello | [12:01] |
| sailus | Hello!
Just pinged pinchartl... | [12:02] |
| hverkuil | mchehab: ping | [12:05] |
| sailus | hverkuil: Wasn't mchehab travelling these days?
Did he say whether he'd join? | [12:07] |
| hverkuil | good question
He's spending two weeks at the brazil office. but should be able to join the meeting AFAICT. | [12:07] |
| syoung | During last week's meeting: 12:03 <+mchehab> not sure yet if I'll be able to join for the meetings but I'll likely find a way
I'm guessing he did not find a way :/ | [12:09] |
| sailus | It's ten past. I guess we could start. | [12:10] |
| hverkuil | I don't have much to report. I've been working on this: http://people.canonical.com/~ubuntu-security/cve/2017/CVE-2017-13166.html
Expect a bunch of patches soon. Besides fixing that CVE I also found a ton of bugs in the compat-ioctl32 code. | [12:11] |
| sailus | Gosh.
I should expect to see a flow of patches to stable kernels then? | [12:12] |
| hverkuil | We'll see. That's a lot of work. | [12:13] |
| sailus | That seems pretty bad, doesn't it? | [12:14] |
| hverkuil | I think it's a real security risk that is exploitable. The patch Daniel Mentz came up with has been there for almost half a year, but nobody could be bothered to inform upstream. That really pisses me off. | [12:16] |
| mchehab | sorry, I had some issues to connect today | [12:16] |
| sailus | hverkuil: Agreed. | [12:17] |
| mchehab | yep, they should have pinged us
hverkuil: the better would be to apply those patchesets for 4.16, of course c/c stable from my side, I'm still stuck with the softirq issue trying to setup a way to test it at the office | [12:18] |
| hverkuil | right now they only apply to 4.15 and up. | [12:19] |
| mchehab | the previous patchset didn't work. A new one was submitted today
hverkuil: well, let's solve 4.15+ first then work on backporting it | [12:20] |
| hverkuil | yup | [12:20] |
| mchehab | at least the CVE ones should be send to earlier Kernels
we should try to simplify the compat32 on V4L i have the feeling that it is more complex than it should be as even compatible ioctls should be listed there | [12:21] |
| hverkuil | the problem is that v4l2-compat-ioctl32.c changes a lot, so for almost every kernel you'd need to do some manual work. | [12:22] |
| mchehab | that's due to that: all IOCTLS are listed there | [12:22] |
| hverkuil | No, only ioctls that need conversion are listed.
It used to be that all ioctls were listed, but that was changed some time ago. | [12:23] |
| mchehab | ah, OK! | [12:23] |
| hverkuil | Anyway, because of this I didn't have time to do any patch handling. | [12:23] |
| mchehab | ok
yeah, this has priority over normal patch handling sailus, mkrufky, pinchartl, snawrocki, syoung: anything from your side? | [12:24] |
| hverkuil | vivid + v4l2-compliance have been incredibly useful to test this. I get good coverage that way.
(ioctl coverage) | [12:26] |
| mchehab | good! | [12:26] |
| sailus | mchehab: Thanks for pulling the patches. I think that was it. I'll postpone all non-fix patches until we have rc1. | [12:26] |
| mchehab | ok
my guess is that the merge window will start next week | [12:27] |
| sailus | On media device refcounting --- I've done some more work on it but lately I've been stuck debugging a crash.
I've converted some existing drivers for refcounting and I've done testing with a rtl28xxu stick. | [12:27] |
| mchehab | ok | [12:29] |
| sailus | That's media device refcounting only; the DVB framework appears to have similar issues but also makes an attept to work with them by waiting until all users are gone, it seems.
I'll review Alexandre's request patches in the near future. | [12:29] |
| mchehab | dvb refcounting used to work fine, until 2016
it is more complex than V4L, as, on DVB, there is a thread that starts running when frontend is opened | [12:30] |
| sailus | Some of that seems fine whereas elsewhere there's an apparent assumption of a use case such as a stateless video codec. | [12:31] |
| mchehab | the refcount logic there has to wait for all users to go | [12:31] |
| sailus | It's not refcounting, but waiting. :-)
That was another approach considered for media device btw., and still needed in some cases, for example to shut down hardware. | [12:32] |
| hverkuil | pinchartl: can you comment on https://www.mail-archive.com/linux-media@vger.kernel.org/msg125078.html? | [12:33] |
| mchehab | sailus: v4l2 has a waiting logic too for some structs | [12:33] |
| hverkuil | omap_vout.c is badly broken. Perhaps we should retire that driver? | [12:33] |
| mchehab | anything else? | [12:36] |
| hverkuil | not from me | [12:38] |
| mchehab | (with regards to omap_vout, if it is broken, I'm OK to retire it)
send a RFC if none complains, it is probably safe to retire it | [12:38] |
| hverkuil | will do | [12:40] |
| .................................................. (idle for 4h7mn) | ||
| I finished my last tests for the CVE fix. I'll post the patch series tomorrow morning. | [16:47] | |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |