• Linus Torvalds's avatar
    Merge tag 'unsigned-char-6.2-for-linus' of... · 3a28c2c8
    Linus Torvalds authored
    Merge tag 'unsigned-char-6.2-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/zx2c4/linux
    
    Pull unsigned-char conversion from Jason Donenfeld:
     "Enable -funsigned-char and fix code affected by that flag.
    
      During the 6.1 cycle, several patches already made it into the tree,
      which were for code that was already broken on at least one
      architecture, where the naked char had a different sign than the code
      author anticipated, or were part of some bug fix for an existing bug
      that this initiative unearthed.
    
      These 6.1-era fixes are:
    
        64806090 ("MIPS: pic32: treat port as signed integer")
        5c26159c ("ipvs: use explicitly signed chars")
        e6cb8769 ("wifi: airo: do not assign -1 to unsigned char")
        937ec9f7 ("staging: rtl8192e: remove bogus ssid character sign test")
        67704738 ("misc: sgi-gru: use explicitly signed char")
        50895a55 ("ALSA: rme9652: use explicitly signed char")
        ee03c0f2 ("ALSA: au88x0: use explicitly signed char")
        835bed1b ("fbdev: sisfb: use explicitly signed char")
        50f19697 ("parisc: Use signed char for hardware path in pdc.h")
        66063033 ("wifi: rt2x00: use explicitly signed or unsigned types")
    
      Regarding patches in this pull:
    
       - There is one patch in this pull that should have made it to you
         during 6.1 ("media: stv0288: use explicitly signed char"), but the
         maintainer was MIA during the cycle, so it's in here instead.
    
       - Two patches fix single architecture code affected by unsigned char
         ("perf/x86: Make struct p4_event_bind::cntr signed array" and
         "sparc: sbus: treat CPU index as integer"), while one patch fixes
         an unused typedef, in case it's ever used in the future ("media:
         atomisp: make hive_int8 explictly signed").
    
       - Finally, there's the change to actually enable -funsigned-char
         ("kbuild: treat char as always unsigned") and then the removal of
         some no longer useful !__CHAR_UNSIGNED__ selftest code ("lib:
         assume char is unsigned").
    
      The various fixes were found with a combination of diffing objdump
      output, a large variety of Coccinelle scripts, and plain old grep. In
      the end, things didn't seem as bad as I feared they would. But of
      course, it's also possible I missed things.
    
      However, this has been in linux-next for basically an entire cycle
      now, so I'm not overly worried. I've also been daily driving this on
      my laptop for all of 6.1. Still, this series, and the ones sent for
      6.1 don't total in quantity to what I thought it'd be, so I will be on
      the lookout for breakage.
    
      We could receive a few reports that are quickly fixable. Hopefully we
      won't receive a barrage of reports that would result in a revert. And
      just maybe we won't receive any reports at all and nobody will even
      notice. Knock on wood"
    
    * tag 'unsigned-char-6.2-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/zx2c4/linux:
      lib: assume char is unsigned
      kbuild: treat char as always unsigned
      media: atomisp: make hive_int8 explictly signed
      media: stv0288: use explicitly signed char
      sparc: sbus: treat CPU index as integer
      perf/x86: Make struct p4_event_bind::cntr signed array
    3a28c2c8
Makefile 69 KB