Tony Bark@pawb.social to Linux@lemmy.world · edit-23 months agoTorvalds blasts kernel dev for late 'garbage' RISC-V patcheswww.theregister.comexternal-linkmessage-square13fedilinkarrow-up173arrow-down16
arrow-up167arrow-down1external-linkTorvalds blasts kernel dev for late 'garbage' RISC-V patcheswww.theregister.comTony Bark@pawb.social to Linux@lemmy.world · edit-23 months agomessage-square13fedilink
minus-squareVeryFrugal@sh.itjust.workslinkfedilinkEnglisharrow-up12·3 months ago Like this crazy and pointless make_u32_from_two_u16() ‘helper’. That thing makes the world actively a worse place to live. I love how enthusiastic he is about make_u32_from_two_u16()
minus-squaresquaresinger@lemmy.worldlinkfedilinkEnglisharrow-up7·3 months agoIt really is kinda hard to justify wrapping a << 16 | b in a function.
minus-squareTheTechnician27@lemmy.worldlinkfedilinkEnglisharrow-up3·edit-23 months agoEspecially given that one actually tells you where the values get placed.
minus-squareMysteriousSophon21@lemmy.worldlinkfedilinkEnglisharrow-up2·3 months agoYep, these kind of wrapper functions actually hurt kernel readability and add unecessary call overhead - in critical kernel paths every instruction counts.
I love how enthusiastic he is about
make_u32_from_two_u16()It really is kinda hard to justify wrapping
a << 16 | bin a function.Especially given that one actually tells you where the values get placed.
Yep, these kind of wrapper functions actually hurt kernel readability and add unecessary call overhead - in critical kernel paths every instruction counts.