On Tue, Jul 20, 2010 at 07:37:15PM -0400, Steve Dickson wrote: > nfsctl.c: In function 'expsetup': > nfsctl.c:112: warning: signed and unsigned type in conditional expression > > Signed-off-by: Steve Dickson <steved@xxxxxxxxxx> > --- > support/export/nfsctl.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/support/export/nfsctl.c b/support/export/nfsctl.c > index 3b9876a..f89c644 100644 > --- a/support/export/nfsctl.c > +++ b/support/export/nfsctl.c > @@ -109,7 +109,7 @@ expsetup(struct nfsctl_export *exparg, nfs_export *exp, int unexport) > str_tolower(exparg->ex_client); > exparg->ex_flags = exp->m_export.e_flags; > exparg->ex_dev = (!unexport && (exp->m_export.e_flags & NFSEXP_FSID)) ? > - exp->m_export.e_fsid : stb.st_dev; > + (__nfsd_dev_t)exp->m_export.e_fsid : stb.st_dev; Any reason not to just define e_fsid as an __nfsd_dev_t to start off with? --b. > exparg->ex_ino = stb.st_ino; > exparg->ex_anon_uid = exp->m_export.e_anonuid; > exparg->ex_anon_gid = exp->m_export.e_anongid; > -- > 1.7.0.1 > > -- > 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
