Looks correct. Acked-by: Shirish Pargaonkar <[email protected]>
On Mon, Aug 26, 2013 at 1:34 AM, Wei Yongjun <[email protected]> wrote: > From: Wei Yongjun <[email protected]> > > Convert cpu_to_le32(le32_to_cpu(E1) + E2) to use le32_add_cpu(). > > Signed-off-by: Wei Yongjun <[email protected]> > --- > fs/cifs/smb2pdu.c | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c > index abc9c28..4ce66bc 100644 > --- a/fs/cifs/smb2pdu.c > +++ b/fs/cifs/smb2pdu.c > @@ -939,9 +939,7 @@ add_lease_context(struct kvec *iov, unsigned int > *num_iovec, __u8 *oplock) > req->CreateContextsOffset = cpu_to_le32( > sizeof(struct smb2_create_req) - 4 + > iov[num - 1].iov_len); > - req->CreateContextsLength = cpu_to_le32( > - le32_to_cpu(req->CreateContextsLength) + > - sizeof(struct create_lease)); > + le32_add_cpu(&req->CreateContextsLength, sizeof(struct create_lease)); > inc_rfc1001_len(&req->hdr, sizeof(struct create_lease)); > *num_iovec = num + 1; > return 0; > @@ -967,9 +965,7 @@ add_durable_context(struct kvec *iov, unsigned int > *num_iovec, > req->CreateContextsOffset = > cpu_to_le32(sizeof(struct smb2_create_req) - 4 + > > iov[1].iov_len); > - req->CreateContextsLength = > - cpu_to_le32(le32_to_cpu(req->CreateContextsLength) + > - sizeof(struct > create_durable)); > + le32_add_cpu(&req->CreateContextsLength, sizeof(struct > create_durable)); > inc_rfc1001_len(&req->hdr, sizeof(struct create_durable)); > *num_iovec = num + 1; > return 0; > -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
