eas-dev November 2016

eas-dev@lists.linaro.org

16 participants
17 discussions

by Robin Randhawa

Hi Leo. [CC'ing eas-dev]. As discussed, here's an useful break-down of middleware configuration examples and resources shared with us. The publically available resources provide additional context about how hinting is coupled to SchedTune. cpusets https://android.googlesource.com/device/google/marlin/+/nougat-dr1-release/… write /dev/cpuset/top-app/cpus 0-3 write /dev/cpuset/foreground/boost/cpus 0-2 write /dev/cpuset/foreground/cpus 0-2 write /dev/cpuset/background/cpus 0 write /dev/cpuset/system-background/cpus 0-2 cpuctl https://android.googlesource.com/device/google/marlin/+/nougat-dr1-release/… write /dev/cpuctl/cpu.shares 1024 write /dev/cpuctl/cpu.rt_runtime_us 800000 write /dev/cpuctl/cpu.rt_period_us 1000000 mkdir /dev/cpuctl/bg_non_interactive chown system system /dev/cpuctl/bg_non_interactive/tasks chmod 0666 /dev/cpuctl/bg_non_interactive/tasks # 5.0 % write /dev/cpuctl/bg_non_interactive/cpu.shares 52 write /dev/cpuctl/bg_non_interactive/cpu.rt_runtime_us 700000 write /dev/cpuctl/bg_non_interactive/cpu.rt_period_us 1000000 SchedTune https://android.googlesource.com/device/google/marlin/+/nougat-dr1-release/… write /dev/stune/foreground/schedtune.prefer_idle 1 write /dev/stune/top-app/schedtune.boost 10 write /dev/stune/top-app/schedtune.prefer_idle 1 PowerHAL https://android.googlesource.com/device/google/marlin/+/nougat-dr1-release/… Robin IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

7 years, 5 months

37848 eas-dev

by hanidar3amy＠gmail.com

PK Á.yIQ9BILL_8131_ZIP.zipUT }7X}7XuxPKÁ.yI ÑŽðMBBILL_8131.jsUT }7X}7Xuxí[I¯ÜÆ>Ë¿bð.`ãI^¢ÄÀV"å`[AdDž4w²¹4× ÿ=3]óêjö/:ÚB~C«kýjáô]ÔAf8|x|ßêöñËÏŠãß·ZÇýÚÿúCß+ý±é:šOw¶œ6Ú¬:Þ~Ÿõýúñ>ïT±n§G5^MdrÝ<¹ëûæàÜ~wøL<}òÞ4&SÎÖ¯d>Üw_îøtÍOmÿÿtYOûCtºšÓ%8^LÌÊÕ€qQÃbÚGg&¢zHÝµÓéZ.¡9^ÅÙ>ÐáÀÔÒÿYöLÎ²7ÌsÀ7Ù_zuwÁóQ0g&ãTü/IknOÑÓÕ;YiùßìÏöÒ0©y3sÔ5ÌœôÍŠ}žŽ1ö|µýëéRñÃÇ»g~³%°17õj}ùå±ò$v)_û¯â{.äïèø1d-Û?öölQ[0~xÉÂÿY5VÛ¥jÌ÷bÝÿù²Ò&Ùý@ÕÚó,Öt~ºebmÏÄí»*{Óëœ"kl¢¯€¿¬ð:²XùmÕÞŽûï+:kv)[CX2_e%ß¡ùlïV¿e)>š}Çî6â¡tña¯Öªâ_t3IOìqcœ#Sòmfc×ZŸ}¶Òxî(ÕïE³pfÊUHMŠœWý+k$_0Õh/DºK²q8µ'LA3öÎÇ//iCþ.N^t<·a:RìÁó(]Æj³ROúc,$Ë8/yºÆæKK³ÀMZ^*ÄŠÔêu²§é&¥æÆÆð/RÎ|PêèÒV°@bRKÈ÷Zvú"dIÔyZéº#ú.ÝÀ»H;ŠÈeøñ3ù©Ê*OæöFáº¯àMý9]éEPK^Œb~Š3#3€2mÇÈŠÛÑ#fý clàdàØ³0ÝbàŒ=P3k§ë`ëiÕÚ©5Î¶e GåeR_&äãÇ Õ²ÄÌZK)ÃÅÇ©åK°!Dv fè2!Y%mùlžìÖI-ßZÒuÊÂÛbÖ!nhÙd7ºÍS: 082HYrøH Ï_Ôá+/Ç&§#¡»HŠ=¡A<¬Ól-fO€YÃÆË0cùj3-ØkÎeË{=Åžak²Ô©ÃïÚTjšðc^=&£|n4Eyfè@71Òu@g%Î9aÍpf=ò³åd9jAü0; öŸÌÈxKwXNõõ>œÌé!÷Ò-ìG¹úy1#:ß{pæÒ°Ÿ§IŸ*k<E÷y»#Ú»XX%QÓ&KŸµ=LÖK H^ÃR@)c4¡°^íuZX.LÐ[ª3ŸáîÖvzë¯^²C;[¡©¥;VVÊØa`Áž'2OüLmH5exgh<Ö»§ñ6Äé ×lOA£4PXõkTÝW¡}ûàE\XX;UO_ü!`ÙÞêàH.¥SNüK"Å`KŽØâY õŽ@ZTfísY&-Å)¥³P'K=jÂk3öN:OÒVvä\^Â¹³ûªGØes6ÍÚ)[6)ÉÈþf¥xDº{9Äì±ãµêùËèsÉ@I_ÉÙEÚŸ/ÆÛçR çYÆXœ£üá}I<Õv:q¡\ÌSãnœÁŽZ îú·ßSÛEQ8cÈÝb|³g¡²Ç8&þ}®FþXkš«eÃ*<~u£Ïøê4 VÙiÈ}¿ûÏù?¿Ò TñìiÓØKHOŒÏ>I5þÂî7ÈPõ¡ç]ªcö€ JáÉáñS?{Zº]!ÕÍËC~Ep?ÒeN²k=¬ öö<ý&òqzQåè!xQÆß¢%§àTt¬ÂµÜQÑqhPMµ¬BŽHÈ³)6ÈšmCQbûÅWðŠ}mØl KL2%ì¬¿÷W+:o3ûXÒÉ\7Ý>ŸfvTg^·ðÛþ ëVoÚÈ«e ÀŽ÷°Àé¬@ÄxÔos0Wí u`Ý¡ OéjÇ©Û;*Æk"ÄLÍGÑÎ0_¿ÛØÊÖŸ 5ÔU¿`Ó@è€PMÉÀs&ZsšT¹ÖÓjj Õ] ÓÛÙ+ëžH®«5êGª+Ù¬!¥/RŒ€ŠÎ}çFE+ŠšF&j0(Ð.#ÿÏRÆ 3ä[ªÎm³aF_UyÀg'Gµl:§)&èÓkoë ¿ÅPª±ÌÓîhù%+F3SUú£²ËøžÅµÌmEú@&SU;Šgd¥DôØ?N" úÚ&¡gÕÍ:]Óu1/Ð_¹@É)òëêÔ8ÔË5ÇõäùéÚlú¢¿išyÎÔe RN[¢è%:6GÏú"¥Bå"P.2µ" Šª-E÷êÜvÊ`ää©dþHkŠSDù×c¹^ŽÑáÈØ)Ð!Ñ³ºÇzSEžnÏA*nrÌäå2&ü¢A;°Y0ÍêÔ~4£@Mž""[±åÉá ^OÿýìQRÚTxâÝ.âÂ|Ø;Ïø<F Ô¬ôõnàÜ,-F9í°/JZ4I|-[0plíŽ.<áÑ¬£P¶SÊçÿpÖ5]BŽ3eú¯®Í€RàÈ€âÓ]H=µ.tæZ+ÐEFþ\'8œœŠFVÅŸŸ1ÌDÌê6K©Mü§äÀéãËIí(C 9o(+ï±îW¶*r}®Ü3Ÿ(aLK/À~Ê¶G¡jO %ÇtâÅÈwm#!4¶î=uÂ$y"F3ji=èTê|<Xñ%óªe!@ TD¬Ù«I¹ä3jLÂ(€7@_úµCLš¯£kœG* ÛEú,>i/M¹§å¿4ê#&|€ê±WÑWÄO#æÄ(Mg=£/ò~oÚÌQæI%:6Ô\EW&,ŽSÞ²4£Ýúœw+ff¿Bxb€É ³FãP«ÔÙù]vÃ/Ú%0I:¹{µo"5T£@±³néïM£â þ#Á~"ç¡ý°TÚ=MÇ1KŠç?¿ûm_ÖCµ¥}²¥å°IîðÐó?}åý(ïCô§Êååýo?ïWHÜ²_Æ|_À÷Ã>FŒ?ýãWþÈ(ŠNNm36àvvæ®È±Žt÷ë?]3^ç`Óñ)àSÂÄ5 ÒôbO²æÈå2&ÍYí{~øÛ l§e«o¥µ·ûg«Ù1,€ÎA=K¥2âfe?Þû3]ÐÈ !{ÞwÆgÏ€E¬ŽñXW+[> :?ÝnlP»ÖZ"®ò©=ÕîÓÑüc¹ü4[%aÿÏ|s¥{ÈFWÚ^šÊÍÀÛzOIÐÑª3Âe${D&RÌR¬4£=Š/diÒWh²#cî"íDVWzŒšÂ/:æíÉ_kÿ Ë èw\©Èè%ØÎ#/+TQ}¢&³Qm"VônTÐÒV,ú¶ª¿ÚYéÏzJl«íÂJgânÃ$;ÛêAh{Zdÿ|õÌ©Íû²fŽ6lÀŠhY¡y¶VGg¥Úá,!X[m`gí/7fk5,;É&ü`ÏY/oÑC$3üY~ÂNú <ùáÌf-;¹N8Š,Úúól G¡ÃBj<6®}Òüe@(q°äl¿ùûÏæô€gOyÂnŸxè÷æÈdÒA%OiÉx¶ŠkÏ"UÅZ)Ë~;÷ŽO€-íwÏnôJyZçs¶#Í³?ÛÁ6Ï¶scmzŸßo>lPâ\ ¡à¬rS©÷®%évOŽdýuTy¢ô2KOÎ¡óHâgåìŽšßû]Ô4œg`KýÖ>¯¹F"Ç»]$³ž©#ëéY¥Ï×j àhëicuj Rœ¿ÂbžIv 6\8dí_oÀíUbOJS5[0ŒÀ²|¿/æR@Šõ{Ÿü²û8žQÒ€ÚA"[DÆ#nþÑ®ïýÅÃF¯ãUgÖúÜM²þ Ü¡SbÆGÂaÍé%ªùœ;(0Û çõâ]9WmG#¬P£0ØyÝD,!r6 OÌz#Biww¹Â »AI+!çGjTpÈP¡y'fÕT©~µ+Öº { È.7a&èò/5Üèò^|A'Š%äÏQ.È6ô¬C ¡7 a(³³ÍJ¡Wýè_M(ÐòhØ!ëß4r²"ÐñêÎLÏAÃôg§ÖÇ6Œø"û3Øcô 2 ß.A¿É<$|sÜªÎ]ÎJÆµº]oiÐûëq¿:ÂVvÎ»8¹sÛê+[è9°=2rgnfÏôN&ì2Á×k©|ö2ZÓpöíÌáKtIrHí.f|zÇÜHZrÚÈhà}e÷eþùc=OZË5d Úü~,·ùïåZ!M*ŽØgHð¥ ïÉirYÆ¶2äN_á×=ðk©ÊÎ³úOm×¹±Ò£."Òø>ñ/h|~à¢ô¶(é4ìiÕ¶ÁpœC¢//_y ölœ3ñ/3#ß.ËÎ±NhWH ¢ÓEšik0tÖTûqWwÝ |í}ÎçòDu|Xhi¯¹ü4}$&œWuÒX~¢=²¹õùC µHvGåZ~)c5¹;^SüTð¢É2ò3~€v×6Ò>¬ÅÅÊÿõPIõœlk`®\]i$:yQíÆ0[èùzHíÃ#oÝÚ£ù0`H_ùv{Å\3ÏVP iQOÅxªœÀ3GåýïÿPKÁ.yI ÑŽðMB€BILL_8131.jsUT}7XuxPKR6PK Á.yIQ9€BILL_8131_ZIP.zipUT}7XuxPKWé

7 years, 5 months

54125 eas-dev

by zabita＠adiyaman.bel.tr

PK¥xI¯iãiý BILL_20284_ZIP.zipUT U»6XU»6XuxuÖy<ð9ckH2µÜ2÷#€c¹r+ÇèpäÎEfÑeY2WrDEÙ\Ókä*QD¹ú=ÿ=¯×ïûz}ŸÇßÿßh$/À à%E[=×¹HDóšÃ@pÆÊÆÆëæ)}žzÐuGc³®ÃïDD\Œßõ¿Fp±œ?^/j|9ñõ§¿#Ú©ªÎ1aoVÎeÎ9æ8ê0žx¡úŒf3©Ðß-dÑÓK;åàÿRÆîÄò¯nCÎT ŠýªÏE®æÍf1»Týöã&É,J¹ô[ÈæÜô¯¹NÇÔ×êsdqå1£3!/YûJ[ß_75¯}¢sT9çu}yÀŠ^ØÐuhÒñNTŒµ1ïE;¹t·ÇkEïs÷&¶ãOë`lmÿõÇí%À÷×Çb.,ä®©vBÃ~Š¹d%-çŽtÈ÷¥[ÿ<ø1-{+³=Ù u9ó€,g(ñ!ñ'³ïa¯ÏéT9hÞ-"ŸìtQÔ?ÉÌ ÇÚFÎ¿tÛ4C°Þ9mÅ³uFÏ¶ŸŒóÄNâÁvoÞ©³ÞõáQÂ2|¬"±ôG+l«Ko Çª$ÇåáÚ«`÷øMâz·üÔp¬Ä"%ŠQÈMzC¢ç2×/\²³§^È ïDoÿ=2è|)8š·üõ®ëÇNBlëüp}#P ßÿÕ£üáâÄ<u~àwvú)Ü#r¡[ÍO=i÷ÃéêóÕ`S-æÉ² uµ×Aw`k`NUaySóv¢w×DP]/èÔ ÉC5äÒy?oIãCÇ-?|wý4"&;Óãm£K ¢ôklÁ"§÷.ÜX\â*ÄÒÄ4[;nãïÉ8{7ûr.ªU=zž&²¿þÌÌ@_âVá®á/hÑ«Ž \ri`ÝÀõÛÉüŸ>1{éuf/š6ÇµN_%ùÈW&Kja÷EíšX}1ç(ùbªs<JcQ}³RÞ³8ª"Ì2Œ³*Öæy âÅ]ŽûÔHCïCªÞŽý¬QßÏ#Ÿu¡Í3Ê1³÷5§cÕÈÚ}ÖQ9Ü7üõñ.ZhŠG œöÀjCO|ÞðÄ¥ Kl^¡%Aì»/pHùÏ¶"[fì¥Ò·Vö«ËM·qt€ û"î+ä¢L6šõ\I#ZOÂ¹ídôhß>©`Uz ß[x®EÂi6óC/÷k0û¥!ó,Í¥ÁÎ=iœyØ ¡bwrŸ5ïŠÃDáóOùßmºü9'Q{ŠAcôl£êÖåHj#A[z:UÎtp»xóÔ[v!Ï=€Å¥&{W÷)z*k¹j²&=P)å³ f4:r€8bÍ L'Ybùß»µ(Ô ¯<.;-æºÜ0±î×P£žzqÓ¯y_¹òðUÇ/µRSçªðº£~³¹BBÑ3³Õ¹lÝîM£ª6|pzšgP{Á'¢n¶u4ÕëèE\l€ÇwxÔCÐd4VBŒwp(ÒUïÓNŽoj|çÝ|quÊhóàçÅ>H2ÜŸIœ!:®7F1á4dÌL+&%°R 93ºÁ³l{á8¿<À`_ááZuôqPŸþÖ¯ßë#Zjbîó_[Ÿå-Ž¶WØèn"yýÇ]O¬V¿ªiåÉŒ{üUVØ|ÉÇüëËßoð» £õË·©6ïJàzZ è:Ò¥ òÊ1ñ$sÖ-mâŽ§[lB }âÔ¥$EÐn>Œ?Úl¿»ð}ÜÜÁ§ždõ/AõÔ6"9®¶05Ð*O² t6ì÷¶üò=ªPf€ô3¬Îo'§Å¬êÙ=¹Ìlí$;ÂyÄQµåC¯ÎãøÍOäÌ¡kÙ¥!!gO<7ãÂ?NR6v99fêÎy\ž@^ç² çOÈ | :_ö¿EÄh>úE.«$ÀÅÅVX ÿNÉ§Q9Â~êÔùAVq_»yÏV6§{èÀÜíJ·èŒürPäÒQhêÉßòÆ*xø Jê£îèúNÅ÷žèG1hv€öÐ3ÑFà5q£/8DGªµ°Ó|ÐD€êÛgºoÂãkÜKf GÉÁãÉžt7× ··ÙeµÆïr€nù&[œÀQ^mãuiuyjÐ1v£vZêå^v³Ý µßtÌ¥% dâÞº*.O±okàÄæJZ¶;wa"ú|cD?ÜÞ Ù(¿ÜyÌëä!-ã@òáx€xÃ«¶5Q;©Æ|w[}Š(#9u]ZÐé,{ +|R à¹ðë¢ìñÕÃ»w¯ÎÚšÁè·aPm8µ®ª~bV®ôŠÉµºør9øcÌlÝð°z,X:ÈK8coW-``Nk|šÖ/Y¢Já¹¹%yvµÙ=§¿V`J!¹ ª¯60wáÒ¹c7\ünéº^ñ1ñý.Á+Æ4îåëEQSâÚXìÚ#`åøíýµOŸ{¡qæHXÙ%ª[³4()ËWÌÂ!Š]»¿Ú€r¡Äý(©=øjÏzga`-ð§i_;9+Å8ÂÁ¶tJØFêõªfBfã_N²'ÙBéßgÖ{N^Äß×æd5C u0x#}à3xïQP±úöØÙŽµÊ«û-®Ð8šXüÇ~oÚ>7ÉÅ-Çóÿ`rè9\¿CNþ³ý)Àò¢ ÀÿŸq0¿gÿëPK¥xI¯iãiý €BILL_20284_ZIP.zipUTU»6XuxPKXI

7 years, 5 months

Computer Software Clients List Q4 Offer

by Claire Divas

Hi, Would you be keen on Computer Software Clients List for your deals and showcasing effort? Key Decision makers: CIOs /COOs/ CTOs/ CFO's/ IT Directors IT Management IT Architects Line of Business Managers & Directors Security Professionals Data Centre Managers IP Communications industry' Including Service Providers Carriers, Enterprises, Government Agencies Reseller Manufacturers Developers As well as Business and Technology Leaders, CTOs, Channel & Partner Managers, Business Development Managers, Analysts & Infrastructure Teams from Cloud Service Providers, Telecommunications, ISPs, ISVs and the IT Channel and Industry serving the cloud community. We are glad to inform you that we running with a Q4 Offer and can provide you the complete list for a discount price. Looking forward to hear from you. Regards, Claire Divas Online Marketing Manager To opt out kindly reply back with 'unsubscribe' or 'leave out'

7 years, 5 months

DEV, Delivery Notification, ID 000361707

by FedEx Standard Overnight

Dear Dev, Courier was unable to deliver the parcel to you. Shipment Label is attached to email. Thank you for choosing FedEx, Joel Dickinson, FedEx Delivery Agent.

7 years, 5 months

[PATCH 0/3] Backport walt_{prepare|finish}_migrate() for kernel v4.4

by Leo Yan

This patch series is to backport walt_prepare_migrate() and walt_finish_migrate() functions from Vikram latest WALT patch series [1] to Android common kernel 4.4. We use these two functions to replace function walt_fixup_busy_time(), as result the scheduler will saperately acquire lock for source rq and destination rq and don't need use function double_lock_balance(). So this will let scheduler flows more safe due we can ensure atomicity by using walt_prepare_migrate() and walt_finish_migrate(). Thanks for Patrick and Vikram's suggestions for this. Leo Yan (3): sched/walt: port walt_{prepare|finish}_migrate() functions sched/core: fix atomicity broken issue sched/walt: remove walt_fixup_busy_time() kernel/sched/core.c | 10 ++++--- kernel/sched/deadline.c | 8 ++++++ kernel/sched/fair.c | 4 +-- kernel/sched/rt.c | 4 +++ kernel/sched/walt.c | 75 +++++++++++++++++++++++++++++++++++++------------ kernel/sched/walt.h | 6 ++-- 6 files changed, 81 insertions(+), 26 deletions(-) -- 1.9.1

7 years, 5 months

[PATCH RFC] sched/freq: fixing frequency update in schedule tick

by Leo Yan

In function for tick_{pelt|walt}, neither of them has considered the schedTune boost margin when set CPU frequency. E.g. when enqueue the task onto rq, it will consider boost margin but after a while a tick is triggered the code will go back to use original CPU utilization value but not boosted value. Another error is: we need convert the capacity request from normalized value to a ratio value [0..1024], the ratio value is the capacity requirement compared to the CPU maximum capacity. So this patch is to fix these two errors. Please note, this patch cannot build successfully due there have some reworks for code need to do. So send for discussion firstly, if have conclusion will generate formal patches. Signed-off-by: Leo Yan <leo.yan(a)linaro.org> --- kernel/sched/core.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index 10f36e2..6f9433e 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c @@ -2947,28 +2947,32 @@ unsigned long sum_capacity_reqs(unsigned long cfs_cap, static void sched_freq_tick_pelt(int cpu) { - unsigned long cpu_utilization = capacity_max; + unsigned long cpu_utilization = boosted_cpu_util(cpu); unsigned long capacity_curr = capacity_curr_of(cpu); struct sched_capacity_reqs *scr; + unsigned long req_cap; scr = &per_cpu(cpu_sched_capacity_reqs, cpu); if (sum_capacity_reqs(cpu_utilization, scr) < capacity_curr) return; + req_cap = cpu_utilization * SCHED_CAPACITY_SCALE / capacity_orig_of(cpu); + /* * To make free room for a task that is building up its "real" * utilization and to harm its performance the least, request * a jump to a higher OPP as soon as the margin of free capacity * is impacted (specified by capacity_margin). */ - set_cfs_cpu_capacity(cpu, true, cpu_utilization); + set_cfs_cpu_capacity(cpu, true, req_cap); } #ifdef CONFIG_SCHED_WALT static void sched_freq_tick_walt(int cpu) { - unsigned long cpu_utilization = cpu_util(cpu); + unsigned long cpu_utilization = boosted_cpu_util(cpu); unsigned long capacity_curr = capacity_curr_of(cpu); + unsigned long req_cap; if (walt_disabled || !sysctl_sched_use_walt_cpu_util) return sched_freq_tick_pelt(cpu); @@ -2983,12 +2987,14 @@ static void sched_freq_tick_walt(int cpu) if (cpu_utilization <= capacity_curr) return; + req_cap = cpu_utilization * SCHED_CAPACITY_SCALE / capacity_orig_of(cpu); + /* * It is likely that the load is growing so we * keep the added margin in our request as an * extra boost. */ - set_cfs_cpu_capacity(cpu, true, cpu_utilization); + set_cfs_cpu_capacity(cpu, true, req_cap); } #define _sched_freq_tick(cpu) sched_freq_tick_walt(cpu) -- 1.9.1

7 years, 5 months

DEV, Courier was unable to deliver the parcel, ID0000177848

by FedEx Ground

Dear Dev, We could not deliver your item. You can review complete details of your order in the find attached. Yours faithfully, Javier Warner, Support Agent.

7 years, 5 months

Re: [Eas-dev] [PATCH] sched: ensure periodic update of cpu usage

by Dietmar Eggemann

Hi Vincent, like promised in our last last 'technical sync-up meeting' here is some feed-back on your patch. The version of the patch is from March this year so a lot of stuff has changed in the meantime but I hope this feedback is still valuable. You might already have addressed some of the issues in your current rebase of the patch. The overall idea seems to be to piggyback NOHZ_STATS_KICK onto the NOHZ_BALANCE_KICK machinery so if the back-end (SCHED_SOFTIRQ) can make a distinction between the need to nohz-stats-update or nohz-balance. -- Dietmar On 14/03/16 09:55, Vincent Guittot wrote: > Conflicts: > kernel/sched/fair.c > --- > > Hi Morten, > > I have finally been able to fix my connection issue. This patch uses the > update_blocked_averages mecanism that is present in the ILB to ensure that the > blocked load will be updated often enough tostay meaningful. > There is still some part that should be fixed like the fixed 5 tick after next > update to trig an update. > > Vincent > > > kernel/sched/fair.c | 64 ++++++++++++++++++++++++++++++++++++++++++++++++---- > kernel/sched/sched.h | 1 + > 2 files changed, 61 insertions(+), 4 deletions(-) > > diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c > index d2d0df4..a716299 100644 > --- a/kernel/sched/fair.c > +++ b/kernel/sched/fair.c > @@ -5108,6 +5108,9 @@ static int get_cpu_usage(int cpu) > return (usage * capacity) >> SCHED_LOAD_SHIFT; > } > > +static inline bool nohz_stat_kick_needed(int cpu); > +static void nohz_balancer_kick(bool only_update); > + > /* > * select_task_rq_fair: Select target runqueue for the waking task in domains > * that have the 'sd_flag' flag set. In practice, this is SD_BALANCE_WAKE, > @@ -5167,6 +5170,10 @@ select_task_rq_fair(struct task_struct *p, int prev_cpu, int sd_flag, int wake_f > if (sd_flag & SD_BALANCE_WAKE) /* XXX always ? */ > new_cpu = select_idle_sibling(p, new_cpu); > > +#ifdef CONFIG_NO_HZ_COMMON > + if (nohz_stat_kick_needed(new_cpu)) > + nohz_balancer_kick(true); > +#endif Why do you ask the update thing only in the select idle sibling path? > } else while (sd) { > struct sched_group *group; > int weight; > @@ -7313,6 +7320,12 @@ static int load_balance(int this_cpu, struct rq *this_rq, > } > > group = find_busiest_group(&env); > + > + if (test_and_clear_bit(NOHZ_STATS_KICK, nohz_flags(this_cpu))) { > + ld_moved = 0; > + goto out; > + } > + > if (!group) { > schedstat_inc(sd, lb_nobusyg[idle]); > goto out_balanced; > @@ -7585,8 +7598,9 @@ static int idle_balance(struct rq *this_rq) > */ > this_rq->idle_stamp = rq_clock(this_rq); > > - if (this_rq->avg_idle < sysctl_sched_migration_cost || > - !this_rq->rd->overload) { > + if (!test_bit(NOHZ_STATS_KICK, nohz_flags(this_cpu)) && > + (this_rq->avg_idle < sysctl_sched_migration_cost || > + !this_rq->rd->overload)) { In case 'NOHZ_STATS_KICK' is set you want to call the update_blocked_averages(this_cpu) below but do you also want to do the actual idle load balancing? > rcu_read_lock(); > sd = rcu_dereference_check_sched_domain(this_rq->sd); > if (sd) > @@ -7639,6 +7653,8 @@ static int idle_balance(struct rq *this_rq) > > raw_spin_lock(&this_rq->lock); > > + clear_bit(NOHZ_STATS_KICK, nohz_flags(this_cpu)); > + > if (curr_cost > this_rq->max_idle_balance_cost) > this_rq->max_idle_balance_cost = curr_cost; > > @@ -7776,6 +7792,9 @@ static inline int find_new_ilb(void) > ilb = cpumask_first_and(sched_domain_span(sd), > nohz.idle_cpus_mask); > Didn't compile for me. I can't find an sd in find_new_ilb() in mainline. Did you add it in a previous patch? > + if (ilb == smp_processor_id()) > + ilb = cpumask_next_and(ilb, sched_domain_span(sd), > + nohz.idle_cpus_mask); > if (ilb < nr_cpu_ids) > break; > } > @@ -7793,7 +7812,7 @@ static inline int find_new_ilb(void) > * nohz_load_balancer CPU (if there is one) otherwise fallback to any idle > * CPU (if there is one). > */ > -static void nohz_balancer_kick(void) > +static void nohz_balancer_kick(bool only_update) > { > int ilb_cpu; > > @@ -7806,6 +7825,9 @@ static void nohz_balancer_kick(void) > > if (test_and_set_bit(NOHZ_BALANCE_KICK, nohz_flags(ilb_cpu))) > return; > + > + if(only_update) > + set_bit(NOHZ_STATS_KICK, nohz_flags(ilb_cpu)); > /* > * Use smp_send_reschedule() instead of resched_cpu(). > * This way we generate a sched IPI on the target cpu which > @@ -8000,6 +8022,8 @@ static void rebalance_domains(struct rq *rq, enum cpu_idle_type idle) > } > rcu_read_unlock(); > > + /* clear any pending stats update request */ > + clear_bit(NOHZ_STATS_KICK, nohz_flags(cpu)); > /* > * next_balance will be updated only when there is a need. > * When the cpu is attached to null domain for ex, it will not be > @@ -8019,11 +8043,14 @@ static void nohz_idle_balance(struct rq *this_rq, enum cpu_idle_type idle) > int this_cpu = this_rq->cpu; > struct rq *rq; > int balance_cpu; > + int update_stats_only = 0; > > if (idle != CPU_IDLE || > !test_bit(NOHZ_BALANCE_KICK, nohz_flags(this_cpu))) > goto end; > > + if (test_bit(NOHZ_STATS_KICK, nohz_flags(this_cpu))) > + update_stats_only = 1; > for_each_cpu(balance_cpu, nohz.idle_cpus_mask) { > if (balance_cpu == this_cpu || !idle_cpu(balance_cpu)) > continue; > @@ -8043,6 +8070,11 @@ static void nohz_idle_balance(struct rq *this_rq, enum cpu_idle_type idle) > * do the balance. > */ > if (time_after_eq(jiffies, rq->next_balance)) { > + > + /* only stats update is required */ > + if (update_stats_only) > + set_bit(NOHZ_STATS_KICK, nohz_flags(balance_cpu)); Why don't you call update_blocked_averages(balance_cpu) here in skip the rebalance_domains() call in case of update_stats_only = 1 (i.e. in case NOHZ_STATS_KICK was set on this_cpu. I assume here that when NOHZ_STATS_KICK is set we really only want to do the update and no actual load balancing. > + > raw_spin_lock_irq(&rq->lock); > update_rq_clock(rq); > update_idle_cpu_load(rq); > @@ -8137,8 +8169,32 @@ static inline bool nohz_kick_needed(struct rq *rq) > rcu_read_unlock(); > return kick; > } > + > +static inline bool nohz_stat_kick_needed(int cpu) > +{ > + unsigned long now = jiffies; You don't bail here if rq->idle_balance is set like nohz_kick_needed() does? > + /* > + * None are in tickless mode and hence no need for NOHZ idle load > + * balancing. > + */ > + if (likely(!atomic_read(&nohz.nr_cpus))) > + return false; > + > + if (time_before(now, nohz.next_balance+5)) > + return false; > + > + /* ensure that this cpu statistics will be updated */ > + set_bit(NOHZ_STATS_KICK, nohz_flags(cpu)); > + > + return true; > +} > + > + > #else > static void nohz_idle_balance(struct rq *this_rq, enum cpu_idle_type idle) { } > +static inline bool nohz_stat_kick_neede/d(int cpu) { return false } > #endif > > /* > @@ -8176,7 +8232,7 @@ void trigger_load_balance(struct rq *rq) > raise_softirq(SCHED_SOFTIRQ); > #ifdef CONFIG_NO_HZ_COMMON > if (nohz_kick_needed(rq)) > - nohz_balancer_kick(); > + nohz_balancer_kick(false); > #endif > } > > diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h > index 676be22c..9cf53df 100644 > --- a/kernel/sched/sched.h > +++ b/kernel/sched/sched.h > @@ -1716,6 +1716,7 @@ extern void cfs_bandwidth_usage_dec(void); > enum rq_nohz_flag_bits { > NOHZ_TICK_STOPPED, > NOHZ_BALANCE_KICK, > + NOHZ_STATS_KICK, > }; > > #define nohz_flags(cpu) (&cpu_rq(cpu)->nohz_flags) >

7 years, 5 months

DEV, We could not deliver your parcel, #00892052

by FedEx Ground

Dear Dev, We could not deliver your item. Please, open email attachment to print shipment label. Kind regards, Ian Mcfarland, Sr. Delivery Agent.

7 years, 5 months

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

eas-dev November 2016