[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20100901135041.GC11107@pengutronix.de>
Date: Wed, 1 Sep 2010 15:50:42 +0200
From: Uwe Kleine-König
<u.kleine-koenig@...gutronix.de>
To: Trond Myklebust <Trond.Myklebust@...app.com>
Cc: linux-nfs@...r.kernel.org, linux-kernel@...r.kernel.org,
akpm@...ux-foundation.org,
"J. Bruce Fields" <bfields@...ldses.org>,
Randy Dunlap <randy.dunlap@...cle.com>,
Neil Brown <neilb@...e.de>,
Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: [REGRESSION PATCH v2] NFS: let NFS_V4 and NFSD_V4 enforce
CRYPTO
On Wed, Sep 01, 2010 at 09:17:04AM -0400, Trond Myklebust wrote:
> On Wed, 2010-09-01 at 10:52 +0200, Uwe Kleine-König wrote:
> > Hello,
> >
> > here comes a v2 of the patch that improves the commit log with a more
> > detailed analysis of the breakage introduced by df486a2
> > (= v2.6.36-rc2~34^2~1 BTW) and additionally undoes the "default y" for
> > RPCSEC_GSS_KRB5.
> >
> > So compared to the state before df486a2 the changes are:
> >
> > NFS_V4 selects CRYPTO
> > NFSD_V4 selects CRYPTO
> > RPCSEC_GSS_KRB5 doesn't depend on EXPERIMENTAL anymore
> > RPCSEC_GSS_KRB5 now depends on CRYPTO instead of selecting it
> >
> > Best regards
> > Uwe
> >
> > ----------------------------->8----------------------------
> >
> > This is a follow up to
> >
> > df486a2 (NFS: Fix the selection of security flavours in Kconfig)
> >
> > Before df486a2 NFS_V4 selected RPCSEC_GSS_KRB5 but didn't enforce the
> > latter's dependency EXPERIMENTAL. df486a2 removed RPCSEC_GSS_KRB5's
> > dependency on EXPERIMENTAL but additionally let it depend on CRYPTO
> > (instead of select CRYPTO before). So it was still possible to have a
> > config that has NFS_V4 but not RPCSEC_GSS_KRB5. Moreover df486a2
> > changed the dependency of NFS_V4 and NFSD_V4 on RPCSEC_GSS_KRB5 from
>
> As I said, the fix is to remove that dependency. I have a fix for the
> NFS client, but the server has more insidious dependencies on RPCSEC_GSS
> due to a poorly designed SECINFO implementation.
Yes, I still remember this, so I suggest to take my patch before 2.6.36
and you can fix it in the merge window for 2.6.37, no?
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists