On Tue, Sep 7, 2021 at 11:04 AM Namjae Jeon <linkinjeon@xxxxxxxxxx> wrote: > > Hi Ronnie, > 2021-09-07 7:46 GMT+09:00, Ronnie Sahlberg <lsahlber@xxxxxxxxxx>: > > Steve, Namjae, > > > > Here is a start of work to share common definitions between the cifs client > > and the server. > > The patches build ontop of Namjaes patch to rework the smb2_hdr structure > > that he recently sent to the list. > > > > It creates a new shared smb2pdu.h file and starts moving definitions over. > > The two copies of smb2pdu.h, in cifs/ and ksmbd/ have diverged a bit > > so some things are being renamed in these patches. > > NegotiateProtocol is in two separate patches since for this funciton the > > changes are a little more than just renames, for example I change several > > arrays at the tail of structures from [number] to simply [] > > so that needs careful review. > > > > Two patches are for cifs and cifs_common and two patches are for ksmbd. > > The ksmbd patches depend on the cifs patches so the cifs patches have to go > > in first. > When I try build test with sparse, I can see build warnings. > I will test more. Thanks. I have fixed the sparse warning and resent. > > $ make fs/ksmbd/ksmbd.ko C=1 CHECK=/home/linkinjeon/sparse-dev/sparse > CF=-D__CHECK_ENDIAN__ > > ... > CC [M] fs/ksmbd/unicode.o > CHECK fs/ksmbd/unicode.c > CC [M] fs/ksmbd/auth.o > CHECK fs/ksmbd/auth.c > fs/ksmbd/auth.c:1267:39: warning: cast to restricted __le64 > CC [M] fs/ksmbd/vfs.o > CHECK fs/ksmbd/vfs.c > CC [M] fs/ksmbd/vfs_cache.o > CHECK fs/ksmbd/vfs_cache.c > CC [M] fs/ksmbd/server.o > CHECK fs/ksmbd/server.c > CC [M] fs/ksmbd/ndr.o > CHECK fs/ksmbd/ndr.c > CC [M] fs/ksmbd/misc.o > CHECK fs/ksmbd/misc.c > CC [M] fs/ksmbd/oplock.o > CHECK fs/ksmbd/oplock.c > CC [M] fs/ksmbd/connection.o > CHECK fs/ksmbd/connection.c > CC [M] fs/ksmbd/ksmbd_work.o > CHECK fs/ksmbd/ksmbd_work.c > CC [M] fs/ksmbd/crypto_ctx.o > CHECK fs/ksmbd/crypto_ctx.c > CC [M] fs/ksmbd/mgmt/ksmbd_ida.o > CHECK fs/ksmbd/mgmt/ksmbd_ida.c > CC [M] fs/ksmbd/mgmt/user_config.o > CHECK fs/ksmbd/mgmt/user_config.c > CC [M] fs/ksmbd/mgmt/share_config.o > CHECK fs/ksmbd/mgmt/share_config.c > CC [M] fs/ksmbd/mgmt/tree_connect.o > CHECK fs/ksmbd/mgmt/tree_connect.c > CC [M] fs/ksmbd/mgmt/user_session.o > CHECK fs/ksmbd/mgmt/user_session.c > CC [M] fs/ksmbd/smb_common.o > CHECK fs/ksmbd/smb_common.c > CC [M] fs/ksmbd/transport_tcp.o > CHECK fs/ksmbd/transport_tcp.c > CC [M] fs/ksmbd/transport_ipc.o > CHECK fs/ksmbd/transport_ipc.c > CC [M] fs/ksmbd/smbacl.o > CHECK fs/ksmbd/smbacl.c > CC [M] fs/ksmbd/smb2pdu.o > CHECK fs/ksmbd/smb2pdu.c > fs/ksmbd/smb2pdu.c:781:29: warning: incorrect type in assignment > (different base types) > fs/ksmbd/smb2pdu.c:781:29: expected unsigned int [usertype] Reserved > fs/ksmbd/smb2pdu.c:781:29: got restricted __le32 [usertype] > fs/ksmbd/smb2pdu.c:783:26: warning: incorrect type in assignment > (different base types) > fs/ksmbd/smb2pdu.c:783:26: expected unsigned int [usertype] Flags > fs/ksmbd/smb2pdu.c:783:26: got restricted __le32 [usertype] > fs/ksmbd/smb2pdu.c:794:29: warning: incorrect type in assignment > (different base types) > fs/ksmbd/smb2pdu.c:794:29: expected unsigned int [usertype] Reserved > fs/ksmbd/smb2pdu.c:794:29: got restricted __le32 [usertype] > fs/ksmbd/smb2pdu.c:8320:47: warning: cast to restricted __le64 > fs/ksmbd/smb2pdu.c:8322:17: warning: cast to restricted __le64 > CC [M] fs/ksmbd/smb2ops.o > CHECK fs/ksmbd/smb2ops.c > CC [M] fs/ksmbd/smb2misc.o > CHECK fs/ksmbd/smb2misc.c > ASN.1 fs/ksmbd/ksmbd_spnego_negtokeninit.asn1.[ch] > CC [M] fs/ksmbd/ksmbd_spnego_negtokeninit.asn1.o > CHECK fs/ksmbd/ksmbd_spnego_negtokeninit.asn1.c > ASN.1 fs/ksmbd/ksmbd_spnego_negtokentarg.asn1.[ch] > CC [M] fs/ksmbd/ksmbd_spnego_negtokentarg.asn1.o > CHECK fs/ksmbd/ksmbd_spnego_negtokentarg.asn1.c > CC [M] fs/ksmbd/asn1.o > CHECK fs/ksmbd/asn1.c > CC [M] fs/ksmbd/transport_rdma.o > CHECK fs/ksmbd/transport_rdma.c > LD [M] fs/ksmbd/ksmbd.o > > Thanks! > > > > > >
