From mboxrd@z Thu Jan 1 00:00:00 1970 From: Trond Myklebust Subject: Re: [RFC v3 38/45] nfs/sunrpc: Return status code from encode functions Date: Thu, 28 May 2015 19:39:54 -0400 Message-ID: References: <9c755df47acdef7f6a1d0195af888dafb7d3e7bf.1429868795.git.agruenba@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Linux Kernel Mailing List , Linux FS-devel Mailing List , Linux NFS Mailing List To: Andreas Gruenbacher Return-path: In-Reply-To: <9c755df47acdef7f6a1d0195af888dafb7d3e7bf.1429868795.git.agruenba-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-fsdevel.vger.kernel.org On Fri, Apr 24, 2015 at 7:04 AM, Andreas Gruenbacher wrote: > Return a status code from the sunrpc xdr encode functions. While these > functions were originally returning a status code before commit 9f06c719 from > December 2010, it was found that none of them can actually fail and so they > were turned into void functions. With more complex objects like NFSv4 ACLs to > encode, it makes sense to do memory allocations and some of the error checking > inside the encode functions, so those functions may fail. The standard way to do this is to allocate the xdr buffer and perform the xdr encoding outside the RPC call itself. Please see the layoutcommit function, for instance. Trond -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html