IRC logs for #cip for Thursday, 2018-09-27

*** rajm has joined #cip07:10
*** patersonc has joined #cip07:11
* rajm runs the daily health on the BBB with an email to the list. 07:20
* rajm runs the daily health check on the iwg20m with an email to the list07:33
patersonc:)07:39
patersoncI haven't received the iwg20m email though, just the bbb one (#192).07:40
*** toscalix has joined #cip07:45
rajmhmm - it definitely came via cip-testing-results did you get the one yesterday patersonc ?07:46
patersoncAh, it's just arrived!07:46
patersoncAnd yes I did get the one yesterday07:47
patersoncSorry for the fuss07:47
rajmok no problem!07:54
*** sangorrin has joined #cip07:58
*** mungaip has joined #cip07:59
*** wagi has joined #cip07:59
szlinwagi: wave07:59
szlin#startmeeting08:00
szlin#roll call08:00
wagiszlin: hi :)08:00
szlinplease say hi if you're around08:00
sangorrin你好08:00
szlinhi ._./08:00
patersonchi08:00
rajmhi08:00
szlinsangorrin: !08:00
bwhhi08:01
szlin#topic kernel maintenance updates08:01
szlinThe following patch was merged into 4.4-stable.08:01
szlind733f7542ad4 ("drivers: net: cpsw: fix segfault in case of bad phy-handle")08:01
szlinhttps://www.spinics.net/lists/stable/msg258336.html08:01
szlin=== 4.4.155 reviewing ===08:01
szlinI have reviewed patches below without any issue08:01
szlinf7f501c753f3 ("getxattr: use correct xattr length")08:01
szlinc2b736ff27b4 ("spi: davinci: fix a NULL pointer dereference")08:01
szlinWhilst reviewing the patch c2b736ff27b4 ("spi: davinci: fix a NULL pointer dereference"), I think below patches should be also backported.08:02
szlin7f3ac71ac3b0 ("spi: davinci: fix spurious i/o error")08:02
szlin35fc3b9ff66b ("spi: davinci: Handle return value of clk_prepare_enable")08:02
*** hungtran has joined #cip08:02
sangorrinszlin, thank you.08:03
wagiCan't really say anything usefull about those parts of the kernel08:03
szlinThe reivew progress is quite slow, but keep going.08:04
szlinI am thinking about to create a wiki page for listing reviewing progress?08:05
sangorrinit would be nice to distribute the review among the kernel team members08:06
sangorrindo you think this patches could be accepted in cip? https://lore.kernel.org/patchwork/cover/622352/08:07
sangorrinthey touch some critical parts08:08
sangorrinsuch as drivers/base/property.c08:08
szlinif no one objects, I will try to create a wiki page for it08:08
sangorrinbut they are necessary to support non-ACPI platforms such as Intel Skylake08:08
patersoncszlin: Good idea08:08
*** iwamatsu__ has joined #cip08:08
szlinIMHO, stable won't accept this kind of patch but only bug-fixed08:09
sangorrinszlin: you can also send us homework. I can probably review a few patches per week08:10
sangorrinthe important thing is that we dont review the same patches08:10
*** iwamatsu__ has quit IRC08:10
bwhszlin: Right, it's definitely not OK for stable. Might be OK for CIP (I didn't look very far).08:10
sangorrinI said CIP not stable08:10
*** iwamatsu__ has joined #cip08:11
szlinsangorrin: it depends on maintainer ?08:11
sangorrinyes, I'm afraid of causing regressions..08:11
szlinsangorrin: yes, this patch contains lots of files and changes...08:13
szlinFurthermore,  I will send the v2 dts patch to upstream for our board this week08:14
szlinany other point?08:14
patersonco/08:15
*** iwamatsu__ has joined #cip08:15
szlinwagi: Im so happy to see you, do you have any updates in rt-stable?08:15
wagiyes, there is a problem with spinlocks08:15
*** iwamatsu__ has quit IRC08:16
wagiThis was I was haunting now for 1 year. As it turns out the qspinlock code is not working correctly08:16
wagijust a sec I fetch an url08:16
szlinwagi: do you have related link?08:16
szlinpatersonc: sorry, please go ahead08:17
patersoncSpeaking of patchwork - is there any value in setting this up (or something else) for CIP? Especially if we're hoping to start getting more maintainers.08:17
wagihttps://lore.kernel.org/lkml/20180921120226.6xjgr4oiho22ex75@linutronix.de/08:17
wagiso for 4.4. I'll drop back to the ticket spinlock implementation08:17
wagiI am testing this right now on the hardware I was observing the problem08:18
wagiso far 12h without an issue08:18
bwhpatersonc: I haven't used it before as a maintainer, but I think it could be useful08:19
szlinwagi: thanks, please keep us posted08:19
patersoncbwh: The renesas-soc mailing list uses it and it seems to work. Helps to organise reviewers and I assume it helps the maintainers keep an eye on everything.08:20
szlinpatersonc: I know some subsystem use patchwork but some don't use it.08:20
wagiszlin: sure08:20
szlinpatersonc: I think it could be useful, too08:21
patersoncShould we ask TSC? Or can it just be the maintainers call? I don't know if it would cost for LF to set it up for us08:21
szlin#agreed create a wiki page for stable patches review08:22
patersonc(unless there are other suggestions)08:22
bwhI would hope that we could use an existing instance (patchwork.kernel.org or patchwork.ozlabs.org)08:22
szlin#action SZ Lin create a wiki page for collaboration08:22
patersoncbwh: good point08:23
szlinbwh: do you know the  differences between patchwork.kernel.org and patchwork.ozlabs.org?08:24
bwhszlin: AFAIK they are running the same software but different people administer them (and might have different rules for which lists they include)08:25
szlinbwh: do you have idea that which one is more suitable for CIP?08:25
bwhI don't08:26
szlinI see, so we need to do some survey if we want to use patchwork08:27
szlinpatersonc: I think we can discuss this in TSC meeting08:28
patersoncOkay08:29
*** hungtran has quit IRC08:30
szlin#action Discuss patchwork item in TSC meeting08:30
szlinany other point?08:30
*** hungtran has joined #cip08:31
szlin#topic kernel testing08:31
rajmI tried Nguyen's suggestion on #186 (the iwg20m) and so far it works reliably, looking how to alter the config so that a uImage doesn't need building manually08:31
rajmbut I'm hoping to resolve this today08:31
szlinrajm: thanks08:31
rajmI'm now using lavacli within b@d for non health check testing and have updated the wiki08:32
szlinI saw the slide from Nguyen yesterday08:32
szlinhttps://events.linuxfoundation.org/wp-content/uploads/2017/11/A-Flexible-Test-Automation-System-for-Various-Embedded-Linux-Usecases-Khiem-Nguyen-Renesas-Design-Vietnam.pdf08:32
rajmMore testing of lava-docker but I'm still getting uboot timeouts - looking into the settings within lava-docker on this08:33
rajmthanks szlin I'll take a look08:33
patersoncFYI, that is a different Nguyen08:33
sangorrinThat's a very common name in Vietnam ;)08:34
rajmok08:34
szlinoops, I see08:34
patersoncBut is one who is very experienced with testing - he works heavily on AGL's setup08:34
patersoncRenesas update: We have a lava lab instance set up, and now have access to the CIP lava master. We're just waiting on LF to exchange keys and then we can start getting the communication working between master and lab.08:34
szlinthis slide mentioned B@D and CIP08:35
patersoncSo it does!08:35
szlinpatersonc: thanks for update08:36
sangorrinKhiem Nguyen?08:36
szlinsangorrin: yes08:36
sangorrinhe is coming to ELCE/ATS08:36
sangorrinpatersonc: great, I already have admin permissions on https://lava.ciplatform.org/08:37
szlinhe mentioned it at page 22 in the slide08:37
szlinany other points?08:38
sangorrinpatersonc: are you using the "polling" setup for the lava slave?08:38
szlin308:38
patersoncsangorrin: I guess so - assuming that means the slave will be polling the master for jobs08:38
sangorrinin the past, Lava server connected with a direct ssh connection, which is almost impossible if you have a firewall08:38
patersoncAs far as I understand it, it's the other way around now.08:39
sangorrinyes, that's correct08:39
patersoncGood.08:39
sangorrinok, good.08:39
szlin208:40
szlin108:40
szlin#topic AOB08:40
szlin308:41
szlin208:41
szlin108:41
szlin#endmeeting08:41
sangorrinAOB?08:41
sangorrinwhat's that?08:41
szlinany other business08:41
sangorrinI will be in ELCE/ATS08:42
sangorrinsee you there if you are going :)08:42
patersoncsangorrin: Great. Tim wouldn't let me join ATS ;)08:42
szlinthank you all08:43
sangorrinsorry, I will let you know what we discuss08:43
patersoncSure08:43
patersoncThanks szlin08:43
rajmthanks szlin08:44
wagipatersonc: do you have any preferences which day we should go for a breakfest together?08:45
sangorrin謝謝08:45
szlinsangorrin: de nada08:45
sangorrin:D08:45
patersoncwagi: Nope! Maybe Sunday if the TSC meeting isn't until lunch?08:46
wagipatersonc: I'll arrive on Sunday night, so Monday morning would be the first opertunity08:46
sangorrinbreakfest? is it like the oktoberfest? :P08:47
patersoncwagi: That's fine08:48
wagisangorrin: hehe :)08:48
wagipatersonc: Monday morning, works for me08:49
wagipatersonc: Do you know the area? Where should we meet?08:49
patersoncwagi: I haven't been there for a few years - so no ;)08:50
patersoncWhere are you staying?08:50
wagiLeonardo Royal Hotel08:51
wagiI think it is very near08:51
wagito the confernce hotel (around the corner)08:51
patersoncOkay. I'm not too far away08:51
patersoncI'll scope scope out a venue and let you know08:51
wagiExcellent. Maybe someone else wants to join as well.08:52
wagiPlease speak up :)08:52
* wagi shutsup now08:52
patersonc:)08:54
*** hungtran has quit IRC09:00
*** tpollard has joined #cip09:07
*** sangorrin has left #cip09:12
toscalixwagi: I will09:59
*** CTtpollard has joined #cip10:02
*** tpollard has quit IRC10:05
* rajm sees lots of possible settings in https://code.fe80.eu/lede/lava-dispatcher-config/commit/daada02d54efe05e942af31793533be8b7255a04 looking to see if there's something I can borrow10:17
wagitoscalix: nice :)11:42
*** mungaip has quit IRC11:59
*** rajm has quit IRC12:15
*** rajm has joined #cip12:26
rajmapologies there's going to be a cip-testing-results for iwg20m again - one of my tests failed badly and it decided it needed another health check :(13:09
*** patersonc has quit IRC13:14
toscalixdo not worry, we love test results13:15
rajmmaybe the email ought to appear then on all results rather than just with status complete (ie the test worked)13:19
* rajm sees mentions of b@d on https://events.linuxfoundation.org/wp-content/uploads/2017/11/A-Flexible-Test-Automation-System-for-Various-Embedded-Linux-Usecases-Khiem-Nguyen-Renesas-Design-Vietnam.pdf and diagrams but fails to see a link to the website13:30
rajm(and multiple copies of that picture of my laptop in Ducie house ;-) )13:30
*** rajm has quit IRC14:26
*** rajm has joined #cip14:41
*** patersonc has joined #cip14:44
rajmaha - gets iwg20m to pass using a zImage - tries again...15:35
rajmlooking good :)15:39
rajmyes 4 working tests  with zImage I'll set up a MR for this tomorrow - the solution being to tell LAVA to use the zImage in the first place15:45
toscalixgreat15:56
toscalixcongrats15:56
rajmthank you - heads off to a meal and concert..15:58
*** rajm has quit IRC16:03
*** toscalix has quit IRC16:33
*** patersonc has quit IRC17:29
*** rajm has joined #cip18:14
*** toscalix has joined #cip19:14

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!