On Fri, Feb 24, 2017 at 09:33:05AM -0800, no-re...@patchew.org wrote: > === OUTPUT BEGIN === > Checking PATCH 1/1: os: don't corrupt pre-existing memory-backend data with > prealloc... > ERROR: Use of volatile is usually wrong: see > Documentation/volatile-considered-harmful.txt
ERROR: checkpatch.pl is usually wrong ;-P Heh, it is refering to a doc in the kernel source tree, which does not even exist at that path location anymore :-) > #42: FILE: util/oslib-posix.c:370: > + volatile char val = *(area + (hpagesize * i)); > > total: 1 errors, 0 warnings, 21 lines checked > > Your patch has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. > > === OUTPUT END === Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|
