Tony Bark@pawb.social to Linux@lemmy.world · edit-24 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-24 months agomessage-square13fedilink
minus-squareVeryFrugal@sh.itjust.workslinkfedilinkEnglisharrow-up12·4 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·4 months agoIt really is kinda hard to justify wrapping a << 16 | b in a function.
minus-squareTheTechnician27@lemmy.worldlinkfedilinkEnglisharrow-up3·edit-24 months agoEspecially given that one actually tells you where the values get placed.
minus-squareMysteriousSophon21@lemmy.worldlinkfedilinkEnglisharrow-up2·4 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.