• Andy Shevchenko's avatar
    x86/io: Define readq()/writeq() to use 64-bit type · 6469a0ee
    Andy Shevchenko authored
    Since non atomic readq() and writeq() were added some of the drivers
    would like to use it in a manner of:
    
     #include <io-64-nonatomic-lo-hi.h>
     ...
     pr_debug("Debug value of some register: %016llx\n", readq(addr));
    
    However, lo_hi_readq() always returns __u64 data, while readq()
    on x86_64 defines it as unsigned long. and thus compiler warns
    about type mismatch, although they are both 64-bit on x86_64.
    
    Convert readq() and writeq() on x86 to operate on deterministic
    64-bit type. The most of architectures in the kernel already are using
    either unsigned long long, or u64 type for readq() / writeq().
    This change propagates consistency in that sense.
    
    While this is not an issue per se, though if someone wants to address it,
    the anchor could be the commit:
    
      797a796a ("asm-generic: architecture independent readq/writeq for 32bit environment")
    
    where non-atomic variants had been introduced.
    
    Note, there are only few users of above pattern and they will not be
    affected because they do cast returned value. The actual warning has
    been issued on not-yet-upstreamed code.
    
    Potentially we might get a new warnings if some 64-bit only code
    assigns returned value to unsigned long type of variable. This is
    assumed to be addressed on case-by-case basis.
    Reported-by: default avatarlkp <lkp@intel.com>
    Tested-by: default avatarSohil Mehta <sohil.mehta@intel.com>
    Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
    Cc: Andy Lutomirski <luto@kernel.org>
    Cc: Borislav Petkov <bp@alien8.de>
    Cc: Brian Gerst <brgerst@gmail.com>
    Cc: Denys Vlasenko <dvlasenk@redhat.com>
    Cc: H. Peter Anvin <hpa@zytor.com>
    Cc: Josh Poimboeuf <jpoimboe@redhat.com>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Link: http://lkml.kernel.org/r/20180515115211.55050-1-andriy.shevchenko@linux.intel.comSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
    6469a0ee
io.h 11.9 KB