* next-20090406: nfsd build fails @ 2009-04-06 7:27 Alexander Beregalov [not found] ` <a4423d670904060027y44550d99g2cb40ce415dbb048-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 8+ messages in thread From: Alexander Beregalov @ 2009-04-06 7:27 UTC (permalink / raw) To: linux-next, linux-nfs, andros, bhalevy, bfields Hi fs/nfsd/nfssvc.c: In function 'set_max_drc': fs/nfsd/nfssvc.c:240: error: 'NFSD_DRC_SIZE_SHIFT' undeclared CONFIG_NFSD_V4 is not set ^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <a4423d670904060027y44550d99g2cb40ce415dbb048-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: next-20090406: nfsd build fails [not found] ` <a4423d670904060027y44550d99g2cb40ce415dbb048-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2009-04-06 8:53 ` Benny Halevy [not found] ` <49D9C308.6020500-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> 0 siblings, 1 reply; 8+ messages in thread From: Benny Halevy @ 2009-04-06 8:53 UTC (permalink / raw) To: Alexander Beregalov, andros-HgOvQuBEEgTQT0dZR+AlfA, bfields-vtMw8L3fJ9vSiEDVxGk4TQ Cc: linux-next-u79uwXL29TY76Z2rM5mHXA, linux-nfs-u79uwXL29TY76Z2rM5mHXA, Stephen Rothwell, pNFS Mailing List On Apr. 06, 2009, 10:27 +0300, Alexander Beregalov <a.beregalov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: > Hi > > fs/nfsd/nfssvc.c: In function 'set_max_drc': > fs/nfsd/nfssvc.c:240: error: 'NFSD_DRC_SIZE_SHIFT' undeclared > > CONFIG_NFSD_V4 is not set Hi Alexander, Thanks for reporting this! Andy, Bruce: please see attached 2 patches fixing compile/link errors with DRC under !defined(CONFIG_NFSD_V4): [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 Benny -- 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <49D9C308.6020500-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>]
* [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc [not found] ` <49D9C308.6020500-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> @ 2009-04-06 9:00 ` Benny Halevy 2009-04-06 9:00 ` [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 Benny Halevy 2009-04-06 17:11 ` next-20090406: nfsd build fails J. Bruce Fields 2 siblings, 0 replies; 8+ messages in thread From: Benny Halevy @ 2009-04-06 9:00 UTC (permalink / raw) To: J. Bruce Fields, Andy Adamson Cc: Alexander Beregalov, Stephen Rothwell, linux-next-u79uwXL29TY76Z2rM5mHXA, pnfs-6DNke4IJHB0gsBAKwltoeQ, linux-nfs-u79uwXL29TY76Z2rM5mHXA, Benny Halevy Fixes the following compiler error: fs/nfsd/nfssvc.c: In function 'set_max_drc': fs/nfsd/nfssvc.c:240: error: 'NFSD_DRC_SIZE_SHIFT' undeclared CONFIG_NFSD_V4 is not set Reported-by: Alexander Beregalov <a.beregalov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Signed-off-by: Benny Halevy <bhalevy-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> --- fs/nfsd/nfssvc.c | 2 ++ include/linux/nfsd/nfsd.h | 3 --- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/fs/nfsd/nfssvc.c b/fs/nfsd/nfssvc.c index e9d5773..469c931 100644 --- a/fs/nfsd/nfssvc.c +++ b/fs/nfsd/nfssvc.c @@ -236,6 +236,8 @@ void nfsd_reset_versions(void) */ static void set_max_drc(void) { + /* The percent of nr_free_buffer_pages used by the V4.1 server DRC */ + #define NFSD_DRC_SIZE_SHIFT 7 nfsd_serv->sv_drc_max_pages = nr_free_buffer_pages() >> NFSD_DRC_SIZE_SHIFT; nfsd_serv->sv_drc_pages_used = 0; diff --git a/include/linux/nfsd/nfsd.h b/include/linux/nfsd/nfsd.h index 0ec4d14..2b49d67 100644 --- a/include/linux/nfsd/nfsd.h +++ b/include/linux/nfsd/nfsd.h @@ -333,9 +333,6 @@ extern struct timeval nfssvc_boot; #define NFSD_LEASE_TIME (nfs4_lease_time()) #define NFSD_LAUNDROMAT_MINTIMEOUT 10 /* seconds */ -/* The percent of nr_free_buffer_pages used by the V4.1 server DRC */ -#define NFSD_DRC_SIZE_SHIFT 7 - /* * The following attributes are currently not supported by the NFSv4 server: * ARCHIVE (deprecated anyway) -- 1.6.2.1 -- 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 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 [not found] ` <49D9C308.6020500-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> 2009-04-06 9:00 ` [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc Benny Halevy @ 2009-04-06 9:00 ` Benny Halevy 2009-04-06 17:11 ` next-20090406: nfsd build fails J. Bruce Fields 2 siblings, 0 replies; 8+ messages in thread From: Benny Halevy @ 2009-04-06 9:00 UTC (permalink / raw) To: J. Bruce Fields, Andy Adamson Cc: Alexander Beregalov, Stephen Rothwell, linux-next-u79uwXL29TY76Z2rM5mHXA, pnfs-6DNke4IJHB0gsBAKwltoeQ, linux-nfs-u79uwXL29TY76Z2rM5mHXA, Benny Halevy Fixes following modpost error: ERROR: "nfsd4_set_statp" [fs/nfsd/nfsd.ko] undefined! Signed-off-by: Benny Halevy <bhalevy-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> --- include/linux/nfsd/cache.h | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/include/linux/nfsd/cache.h b/include/linux/nfsd/cache.h index a59a2df..5bccaab 100644 --- a/include/linux/nfsd/cache.h +++ b/include/linux/nfsd/cache.h @@ -75,6 +75,13 @@ int nfsd_reply_cache_init(void); void nfsd_reply_cache_shutdown(void); int nfsd_cache_lookup(struct svc_rqst *, int); void nfsd_cache_update(struct svc_rqst *, int, __be32 *); + +#ifdef CONFIG_NFSD_V4 void nfsd4_set_statp(struct svc_rqst *rqstp, __be32 *statp); +#else /* CONFIG_NFSD_V4 */ +static inline void nfsd4_set_statp(struct svc_rqst *rqstp, __be32 *statp) +{ +} +#endif /* CONFIG_NFSD_V4 */ #endif /* NFSCACHE_H */ -- 1.6.2.1 -- 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 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: next-20090406: nfsd build fails [not found] ` <49D9C308.6020500-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> 2009-04-06 9:00 ` [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc Benny Halevy 2009-04-06 9:00 ` [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 Benny Halevy @ 2009-04-06 17:11 ` J. Bruce Fields 2009-04-07 18:53 ` Benny Halevy 2 siblings, 1 reply; 8+ messages in thread From: J. Bruce Fields @ 2009-04-06 17:11 UTC (permalink / raw) To: Benny Halevy Cc: Alexander Beregalov, andros-HgOvQuBEEgTQT0dZR+AlfA, linux-next-u79uwXL29TY76Z2rM5mHXA, linux-nfs-u79uwXL29TY76Z2rM5mHXA, Stephen Rothwell, pNFS Mailing List On Mon, Apr 06, 2009 at 11:53:28AM +0300, Benny Halevy wrote: > On Apr. 06, 2009, 10:27 +0300, Alexander Beregalov <a.beregalov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: > > Hi > > > > fs/nfsd/nfssvc.c: In function 'set_max_drc': > > fs/nfsd/nfssvc.c:240: error: 'NFSD_DRC_SIZE_SHIFT' undeclared > > > > CONFIG_NFSD_V4 is not set > > Hi Alexander, > > Thanks for reporting this! > > Andy, Bruce: please see attached 2 patches fixing compile/link errors > with DRC under !defined(CONFIG_NFSD_V4): > > [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc > [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 Thanks, applied. Committing on top (not squashing) since I'd rather not rewrite history on a branch I've already sent a pull request for. (In fact, I'll try to stop rebasing those for-next branches at all this time around.) --b. -- 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: next-20090406: nfsd build fails 2009-04-06 17:11 ` next-20090406: nfsd build fails J. Bruce Fields @ 2009-04-07 18:53 ` Benny Halevy 2009-04-07 20:05 ` [pnfs] " William A. (Andy) Adamson 2009-04-09 16:12 ` J. Bruce Fields 0 siblings, 2 replies; 8+ messages in thread From: Benny Halevy @ 2009-04-07 18:53 UTC (permalink / raw) To: J. Bruce Fields, Trond Myklebust Cc: Alexander Beregalov, andros, linux-next, linux-nfs, Stephen Rothwell, pNFS Mailing List On Apr. 06, 2009, 20:11 +0300, "J. Bruce Fields" <bfields@fieldses.org> wrote: > On Mon, Apr 06, 2009 at 11:53:28AM +0300, Benny Halevy wrote: >> On Apr. 06, 2009, 10:27 +0300, Alexander Beregalov <a.beregalov@gmail.com> wrote: >>> Hi >>> >>> fs/nfsd/nfssvc.c: In function 'set_max_drc': >>> fs/nfsd/nfssvc.c:240: error: 'NFSD_DRC_SIZE_SHIFT' undeclared >>> >>> CONFIG_NFSD_V4 is not set >> Hi Alexander, >> >> Thanks for reporting this! >> >> Andy, Bruce: please see attached 2 patches fixing compile/link errors >> with DRC under !defined(CONFIG_NFSD_V4): >> >> [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc >> [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 > > Thanks, applied. > > Committing on top (not squashing) since I'd rather not rewrite history > on a branch I've already sent a pull request for. Cool. Thanks! I wasn't aware that you already sent a pull request already... I got it from mainline now and I'm rebasing and testing the rest of our stuff on top of it. > > (In fact, I'll try to stop rebasing those for-next branches at all this > time around.) I'm with you on that. For 2.6.31, I'd like to send easy-to-swallow patch sets that we can review and agree upon well ahead of the merge window (i.e. start, e.g. with the backchannel stuff shortly after 2.6.30-rc1 is cut) and once we're in agreement on it we can put it in linux-next to be visible to others and get some soak time. This will also be a good time to freeze it, so it won't be rebased, unless something important enough requires that. How does that sound? I think that the same strategy should work for the client side too. Trond what do you think? Benny > > --b. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [pnfs] next-20090406: nfsd build fails 2009-04-07 18:53 ` Benny Halevy @ 2009-04-07 20:05 ` William A. (Andy) Adamson 2009-04-09 16:12 ` J. Bruce Fields 1 sibling, 0 replies; 8+ messages in thread From: William A. (Andy) Adamson @ 2009-04-07 20:05 UTC (permalink / raw) To: Benny Halevy Cc: J. Bruce Fields, Trond Myklebust, Stephen Rothwell, linux-nfs, pNFS Mailing List, linux-next, Alexander Beregalov On Tue, Apr 7, 2009 at 2:53 PM, Benny Halevy <bhalevy@panasas.com> wrote: > On Apr. 06, 2009, 20:11 +0300, "J. Bruce Fields" <bfields@fieldses.org> wrote: >> On Mon, Apr 06, 2009 at 11:53:28AM +0300, Benny Halevy wrote: >>> On Apr. 06, 2009, 10:27 +0300, Alexander Beregalov <a.beregalov@gmail.com> wrote: >>>> Hi >>>> >>>> fs/nfsd/nfssvc.c: In function 'set_max_drc': >>>> fs/nfsd/nfssvc.c:240: error: 'NFSD_DRC_SIZE_SHIFT' undeclared >>>> >>>> CONFIG_NFSD_V4 is not set >>> Hi Alexander, >>> >>> Thanks for reporting this! >>> >>> Andy, Bruce: please see attached 2 patches fixing compile/link errors >>> with DRC under !defined(CONFIG_NFSD_V4): >>> >>> [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc >>> [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 >> >> Thanks, applied. >> >> Committing on top (not squashing) since I'd rather not rewrite history >> on a branch I've already sent a pull request for. > > Cool. Thanks! > I wasn't aware that you already sent a pull request already... > I got it from mainline now and I'm rebasing and testing > the rest of our stuff on top of it. > >> >> (In fact, I'll try to stop rebasing those for-next branches at all this >> time around.) > > I'm with you on that. > > For 2.6.31, I'd like to send easy-to-swallow patch sets > that we can review and agree upon well ahead of the merge window > (i.e. start, e.g. with the backchannel stuff shortly after 2.6.30-rc1 > is cut) and once we're in agreement on it we can put it in linux-next > to be visible to others and get some soak time. This sounds good to me. I'm continuing with Bruce's review comments WRT the DRC. > This will also > be a good time to freeze it, so it won't be rebased, unless something > important enough requires that. How does that sound? > > I think that the same strategy should work for the client side too. > Trond what do you think? > > Benny > >> >> --b. > _______________________________________________ > pNFS mailing list > pNFS@linux-nfs.org > http://linux-nfs.org/cgi-bin/mailman/listinfo/pnfs > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: next-20090406: nfsd build fails 2009-04-07 18:53 ` Benny Halevy 2009-04-07 20:05 ` [pnfs] " William A. (Andy) Adamson @ 2009-04-09 16:12 ` J. Bruce Fields 1 sibling, 0 replies; 8+ messages in thread From: J. Bruce Fields @ 2009-04-09 16:12 UTC (permalink / raw) To: Benny Halevy Cc: Trond Myklebust, Alexander Beregalov, andros, linux-next, linux-nfs, Stephen Rothwell, pNFS Mailing List On Tue, Apr 07, 2009 at 09:53:38PM +0300, Benny Halevy wrote: > On Apr. 06, 2009, 20:11 +0300, "J. Bruce Fields" <bfields@fieldses.org> wrote: > > On Mon, Apr 06, 2009 at 11:53:28AM +0300, Benny Halevy wrote: > >> On Apr. 06, 2009, 10:27 +0300, Alexander Beregalov <a.beregalov@gmail.com> wrote: > >>> Hi > >>> > >>> fs/nfsd/nfssvc.c: In function 'set_max_drc': > >>> fs/nfsd/nfssvc.c:240: error: 'NFSD_DRC_SIZE_SHIFT' undeclared > >>> > >>> CONFIG_NFSD_V4 is not set > >> Hi Alexander, > >> > >> Thanks for reporting this! > >> > >> Andy, Bruce: please see attached 2 patches fixing compile/link errors > >> with DRC under !defined(CONFIG_NFSD_V4): > >> > >> [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc > >> [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 > > > > Thanks, applied. > > > > Committing on top (not squashing) since I'd rather not rewrite history > > on a branch I've already sent a pull request for. > > Cool. Thanks! > I wasn't aware that you already sent a pull request already... > I got it from mainline now and I'm rebasing and testing > the rest of our stuff on top of it. > > > > > (In fact, I'll try to stop rebasing those for-next branches at all this > > time around.) > > I'm with you on that. > > For 2.6.31, I'd like to send easy-to-swallow patch sets > that we can review and agree upon well ahead of the merge window > (i.e. start, e.g. with the backchannel stuff shortly after 2.6.30-rc1 > is cut) and once we're in agreement on it we can put it in linux-next > to be visible to others and get some soak time. This will also > be a good time to freeze it, so it won't be rebased, Note that it's OK to rebase things in -next; so it doesn't have to be frozen till it goes into one of our trees. (And I've tried once or twice to maintain the discpline of not rebasing stuff in my tree and have failed.... I'll make a better effort this time.) > unless something > important enough requires that. How does that sound? Sounds fine. If you send in things as they're ready, I'll try to turn them around a little faster (within a few days or a week anyway). --b. > I think that the same strategy should work for the client side too. > Trond what do you think? > > Benny > > > > > --b. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2009-04-09 16:12 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2009-04-06 7:27 next-20090406: nfsd build fails Alexander Beregalov [not found] ` <a4423d670904060027y44550d99g2cb40ce415dbb048-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2009-04-06 8:53 ` Benny Halevy [not found] ` <49D9C308.6020500-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org> 2009-04-06 9:00 ` [PATCH 1/2] SQUASHEM: nfsd41: define NFSD_DRC_SIZE_SHIFT in set_max_drc Benny Halevy 2009-04-06 9:00 ` [PATCH 2/2] SQUASHME: nfsd41: define nfsd4_set_statp as noop for !CONFIG_NFSD_V4 Benny Halevy 2009-04-06 17:11 ` next-20090406: nfsd build fails J. Bruce Fields 2009-04-07 18:53 ` Benny Halevy 2009-04-07 20:05 ` [pnfs] " William A. (Andy) Adamson 2009-04-09 16:12 ` J. Bruce Fields
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).