Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Rich Dubielzig <rich.dubielzig@windriver.com>
Cc: Openembedded-core@lists.openembedded.org
Subject: Re: [PATCH V2] Add scratch directory for NFS upcall state storage
Date: Fri, 01 Mar 2013 14:47:37 +0000	[thread overview]
Message-ID: <1362149257.1055.58.camel@ted> (raw)
In-Reply-To: <1362077027-25696-1-git-send-email-rich.dubielzig@windriver.com>

On Thu, 2013-02-28 at 10:43 -0800, Rich Dubielzig wrote:
> When the /var/lib/nfs/v4recovery directory is not present, NFSD cannot
> record the upcall state and cannot end the 90-second grace period on
> startup.  This is true even when NFS4 is not being served according to
> nfsstat.
> 
> More details and discussion here:
> https://lkml.org/lkml/2012/6/11/206
> 
> Signed-off-by: Rich Dubielzig <rich.dubielzig@windriver.com>
> ---
>  .../nfs-utils/nfs-utils_1.2.3.bb                   |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)

The subject line is missing the prefix ("nfs-utils: ") and this patch
doesn't apply against master which has nfs-utils 1.2.7?

Cheers,

Richard

> diff --git a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bb b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bb
> index 84c4464..138791b 100644
> --- a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bb
> +++ b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bb
> @@ -12,7 +12,7 @@ DEPENDS = "libcap libnfsidmap libevent util-linux tcp-wrappers"
>  RDEPENDS_${PN} = "rpcbind"
>  RRECOMMENDS_${PN} = "kernel-module-nfsd"
>  
> -PR = "r4"
> +PR = "r5"
>  
>  SRC_URI = "${SOURCEFORGE_MIRROR}/nfs/nfs-utils-${PV}.tar.bz2 \
>             file://nfs-utils-1.0.6-uclibc.patch \
> @@ -53,6 +53,8 @@ RDEPENDS_${PN}-stats = "python"
>  do_install_append () {
>  	install -d ${D}${sysconfdir}/init.d
>  	install -m 0755 ${WORKDIR}/nfsserver ${D}${sysconfdir}/init.d/nfsserver
> +	# kernel code as of 3.8 hard-codes this path as a default
> +	install -d ${D}/var/lib/nfs/v4recovery
>  
>  	# the following are built by CC_FOR_BUILD
>  	rm -f ${D}${sbindir}/rpcdebug





  reply	other threads:[~2013-03-01 15:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-28 18:43 [PATCH V2] Add scratch directory for NFS upcall state storage Rich Dubielzig
2013-03-01 14:47 ` Richard Purdie [this message]
     [not found] ` <1362180962-13024-1-git-send-email-rich.dubielzig@windriver.com>
2013-03-01 23:41   ` [V3] nfs-utils: Add scratch directory for NFS upcall state storage (denzil) Dubielzig, Rich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1362149257.1055.58.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=Openembedded-core@lists.openembedded.org \
    --cc=rich.dubielzig@windriver.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox