Re: [PATCH 00/11 V2] Removed a bunch random warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 07/25/2017 04:53 PM, J. Bruce Fields wrote:
> On Fri, Jul 21, 2017 at 12:37:55PM -0400, Steve Dickson wrote:
>> I upgraded to Fedora 26 and went to apply Neil's recent mount 
>> patches and ended up seeing all these warning. 
>>
>> These patches fix those warnings
>>
>> v2: change a changelog that explicitly fixes a bug.
>>     Changed a number of break statements into FALLTHRU comments.
> 
> Looks good to me, except for the obvious mistake on #3 (and one
> whitespace nit).
I'll fix both on the commit... 

Thanks for the review! 

steved.
> 
> Thanks!--b.
> 
>>     
>> Steve Dickson (11):
>>   rpcdebug.c: remove a warning
>>   present_address: Fixed error path
>>   atomicio: removed a warning
>>   cache.c: removed a couple warning
>>   nfsd.c: removed a few warnings
>>   bldev_read_serial: removed a couple warnings
>>   device-discovery.c: removed a warning
>>   network.c: removed some warnings
>>   nfsmount: remove a warning
>>   nfs4mount: removed a warning
>>   nfsdcltrack.c: remove a warning
>>
>>  support/misc/tcpwrapper.c        | 1 +
>>  support/nfs/atomicio.c           | 2 ++
>>  tools/rpcdebug/rpcdebug.c        | 3 ++-
>>  utils/blkmapd/device-discovery.c | 1 +
>>  utils/blkmapd/device-inq.c       | 2 ++
>>  utils/mount/network.c            | 5 +++++
>>  utils/mount/nfs4mount.c          | 1 +
>>  utils/mount/nfsmount.c           | 1 +
>>  utils/mountd/cache.c             | 1 +
>>  utils/nfsd/nfsd.c                | 3 +++
>>  utils/nfsdcltrack/nfsdcltrack.c  | 1 +
>>  11 files changed, 20 insertions(+), 1 deletion(-)
>>
>> -- 
>> 2.13.3
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux