Changes since the first send: - split to five separate patches - fixed a bug in cache_get_filehandle() that made result parsing not work - fixed to check result of write() calls The review mentioned my patches adding: nfssvc.c:71:8: warning: ignoring return value of 'system', declared with attribute warn_unused_result [-Wunused-result] nfssvc.c:325:9: warning: ignoring return value of 'write', declared with attribute warn_unused_result [-Wunused-result] but this does not make any sense: I'm only adding one #include there. Is that perhaps uncovering some other issues? I'm not sure if the NFSv3 related issues were caused by the cache_get_filehandle() issue or not, so this still needs testing. Another potential cause is that the kernel is sending to user land requests longer than RPC_CHAN_BUF_SIZE bytes, but that does not seem likely. Timo Teräs (5): Add string.h to source files that need it mountd: talk to kernel using file descriptors instead of FILE gssd: talk to kernel using file descriptors instead of FILE nfsexport: talk to kernel using file descriptors instead of FILE nfslib: remove now unused FILE helpers support/include/exportfs.h | 1 + support/include/nfslib.h | 7 - support/include/nfsrpc.h | 1 + support/nfs/cacheio.c | 111 +------------ support/nfs/nfsexport.c | 77 +++++---- utils/gssd/gssd_proc.c | 9 +- utils/gssd/svcgssd.h | 2 +- utils/gssd/svcgssd_main_loop.c | 9 +- utils/gssd/svcgssd_proc.c | 51 +++--- utils/gssd/write_bytes.h | 1 + utils/mountd/cache.c | 343 ++++++++++++++++++++++------------------- utils/nfsd/nfssvc.c | 1 + 12 files changed, 270 insertions(+), 343 deletions(-) -- 2.1.2 -- 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
