[meta-baryon][PATCH 0/1] nfs regression workaround

[meta-baryon][PATCH 0/1] nfs regression workaround

[Kevin Strasser]

The new recovery mechanism used by nfs in 3.4 kernels is currently
failing when building baryon against poky 1.3_M3. This workaround
causes nfs to revert back to the old recovery mechanism.

The issue is discussed in more detail here:
https://lkml.org/lkml/2012/6/11/243

The following changes since commit e4efadec4a1ded0a66fa67c7cca868b7a4b6221b:

  talloc: specify the version of LGPL and include the license text (2012-09-27 16:54:08 -0700)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib strassek/baryon-nfsd-regression
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=strassek/baryon-nfsd-regression

Kevin Strasser (1):
  nfs-utils: workaround for nfsd regression in the 3.4 kernel

 .../nfs-utils/nfs-utils_1.2.3.bbappend             |    8 ++++++++
 1 file changed, 8 insertions(+)
 create mode 100644 recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend

-- 
1.7.9.5

[meta-baryon][PATCH 1/1] nfs-utils: workaround for nfsd regression in the 3.4 kernel

[Kevin Strasser]

The version of nfsd used in 3.4 kernels tries to upcall the
new reboot-recovery daemon and gets stuck if it is not found.
This causes client mounts to fail and prints the following
error message during boot:

"NFSD: starting 90-second grace period
NFSD: Unable to end grace period: -110"

If the directory "/var/lib/nfs/v4recovery" exists, nfsd will
revert back to the old method.

Signed-off-by: Kevin Strasser <kevin.strasser@linux.intel.com>
---
 .../nfs-utils/nfs-utils_1.2.3.bbappend             |    8 ++++++++
 1 file changed, 8 insertions(+)
 create mode 100644 recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend

diff --git a/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
new file mode 100644
index 0000000..2c91a93
--- /dev/null
+++ b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
@@ -0,0 +1,8 @@
+PR = "r5"
+
+# Work around linux 3.4 nfsd regression
+do_install_prepend () {
+	install -d ${D}/var/lib/nfs/v4recovery
+}
+
+FILES_${PN} += "/var/lib/nfs/v4recovery"
-- 
1.7.9.5

Re: [meta-baryon][PATCH 1/1] nfs-utils: workaround for nfsd regression in the 3.4 kernel

[Paul Eggleton]

Hi Kevin,

On Friday 28 September 2012 12:44:06 Kevin Strasser wrote:
> The version of nfsd used in 3.4 kernels tries to upcall the
> new reboot-recovery daemon and gets stuck if it is not found.
> This causes client mounts to fail and prints the following
> error message during boot:
> 
> "NFSD: starting 90-second grace period
> NFSD: Unable to end grace period: -110"
> 
> If the directory "/var/lib/nfs/v4recovery" exists, nfsd will
> revert back to the old method.
> 
> Signed-off-by: Kevin Strasser <kevin.strasser@linux.intel.com>
> ---
>  .../nfs-utils/nfs-utils_1.2.3.bbappend             |    8 ++++++++
>  1 file changed, 8 insertions(+)
>  create mode 100644 recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
> 
> diff --git a/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
> b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend new file mode
> 100644
> index 0000000..2c91a93
> --- /dev/null
> +++ b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
> @@ -0,0 +1,8 @@
> +PR = "r5"

This needs to be:

PRINC = "1"

or, to ensure other layers can also increment the value:

PRINC := "${@int(PRINC) + 1}"

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [meta-baryon][PATCH 1/1] nfs-utils: workaround for nfsd regression in the 3.4 kernel

[kevin.strasser]

> Hi Kevin,
>
> On Friday 28 September 2012 12:44:06 Kevin Strasser wrote:
>> The version of nfsd used in 3.4 kernels tries to upcall the
>> new reboot-recovery daemon and gets stuck if it is not found.
>> This causes client mounts to fail and prints the following
>> error message during boot:
>>
>> "NFSD: starting 90-second grace period
>> NFSD: Unable to end grace period: -110"
>>
>> If the directory "/var/lib/nfs/v4recovery" exists, nfsd will
>> revert back to the old method.
>>
>> Signed-off-by: Kevin Strasser <kevin.strasser@linux.intel.com>
>> ---
>>  .../nfs-utils/nfs-utils_1.2.3.bbappend             |    8 ++++++++
>>  1 file changed, 8 insertions(+)
>>  create mode 100644
>> recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
>>
>> diff --git a/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
>> b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend new file mode
>> 100644
>> index 0000000..2c91a93
>> --- /dev/null
>> +++ b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
>> @@ -0,0 +1,8 @@
>> +PR = "r5"
>
> This needs to be:
>
> PRINC = "1"
>
> or, to ensure other layers can also increment the value:
>
> PRINC := "${@int(PRINC) + 1}"

OK, that makes sense. I've made the changes and pushed them again to:

  git://git.yoctoproject.org/poky-contrib strassek/baryon-nfsd-regression
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=strassek/baryon-nfsd-regression

While we are on the subject, do you think this is something that should be
added to poky?

-Kevin

Re: [meta-baryon][PATCH 1/1] nfs-utils: workaround for nfsd regression in the 3.4 kernel

[Paul Eggleton]

On Monday 01 October 2012 10:25:47 kevin.strasser@linux.intel.com wrote:
> > Hi Kevin,
> > 
> > On Friday 28 September 2012 12:44:06 Kevin Strasser wrote:
> >> The version of nfsd used in 3.4 kernels tries to upcall the
> >> new reboot-recovery daemon and gets stuck if it is not found.
> >> This causes client mounts to fail and prints the following
> >> error message during boot:
> >> 
> >> "NFSD: starting 90-second grace period
> >> NFSD: Unable to end grace period: -110"
> >> 
> >> If the directory "/var/lib/nfs/v4recovery" exists, nfsd will
> >> revert back to the old method.
> >> 
> >> Signed-off-by: Kevin Strasser <kevin.strasser@linux.intel.com>
> >> ---
> >> 
> >>  .../nfs-utils/nfs-utils_1.2.3.bbappend             |    8 ++++++++
> >>  1 file changed, 8 insertions(+)
> >>  create mode 100644
> >> 
> >> recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
> >> 
> >> diff --git a/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
> >> b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend new file mode
> >> 100644
> >> index 0000000..2c91a93
> >> --- /dev/null
> >> +++ b/recipes-connectivity/nfs-utils/nfs-utils_1.2.3.bbappend
> >> @@ -0,0 +1,8 @@
> >> +PR = "r5"
> > 
> > This needs to be:
> > 
> > PRINC = "1"
> > 
> > or, to ensure other layers can also increment the value:
> > 
> > PRINC := "${@int(PRINC) + 1}"
> 
> OK, that makes sense. I've made the changes and pushed them again to:
> 
>   git://git.yoctoproject.org/poky-contrib strassek/baryon-nfsd-regression
>  
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=strassek/baryon-nf
> sd-regression

Merged to meta-baryon master, thanks.

> While we are on the subject, do you think this is something that should be
> added to poky?

Quite possibly. Would you mind opening up a bug for the issue?

Thanks,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre