* ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" @ 2018-09-13 7:58 Koen Vandeputte 2018-09-13 9:46 ` gregkh 0 siblings, 1 reply; 7+ messages in thread From: Koen Vandeputte @ 2018-09-13 7:58 UTC (permalink / raw) To: linux-mtd; +Cc: gregkh Hi all, I'm currently in the process of updating the kernel version within OpenWrt. (4.14.68 to 4.14.69) Testing shows some issues on devices using specifically UBIFS. Altering a perfect valid writable file shows weird errors: [ Node 2 | node-2 ] ls -l /root/custom/scripts/banner.sh -rwxr-xr-x 1 root root 283 Sep 11 09:52 /root/custom/scripts/banner.sh [ Node 2 | node-2 ] cat /root/custom/scripts/banner.sh #!/bin/sh if [ ! -f /root/.banner_ok ] then RELEASE=$(cat /root/build_date) VERSION=$(cat /root/version) echo "Generating banner: $VERSION $RELEASE" sed s/VERSION/$VERSION/g /root/custom/banner > /etc/banner sed -i s/RELEASE/$RELEASE/g /etc/banner touch /root/.banner_ok fi [ Node 2 | node-2 ] echo "test" > /root/custom/scripts/banner.sh -ash: can't create /root/custom/scripts/banner.sh: nonexistent directory I'm also noticing other apps fail because /etc doesn't exists yet after UBIFS boot loading. these 2 issues were not seen on 4.14.68. The bootlog doesn't show any error: https://pastebin.com/raw/dJx47uBp I'm only seeing these issues on UBIFS enabled volumes. Reverting ("ubifs: xattr: Don't operate on deleted inodes") fixes these weird issues. Thanks, Koen ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" 2018-09-13 7:58 ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" Koen Vandeputte @ 2018-09-13 9:46 ` gregkh 0 siblings, 0 replies; 7+ messages in thread From: gregkh @ 2018-09-13 9:46 UTC (permalink / raw) To: Koen Vandeputte; +Cc: linux-mtd, stable adding stable@ for stable kernel issues... On Thu, Sep 13, 2018 at 09:58:35AM +0200, Koen Vandeputte wrote: > Hi all, > > I'm currently in the process of updating the kernel version within OpenWrt. > (4.14.68 to 4.14.69) > > Testing shows some issues on devices using specifically UBIFS. > Altering a perfect valid writable file shows weird errors: > > > [ Node 2 | node-2 ] ls -l /root/custom/scripts/banner.sh > -rwxr-xr-x��� 1 root���� root���������� 283 Sep 11 09:52 > /root/custom/scripts/banner.sh > > [ Node 2 | node-2 ] cat /root/custom/scripts/banner.sh > #!/bin/sh > > if [ ! -f /root/.banner_ok ] > then > ��� RELEASE=$(cat /root/build_date) > ��� VERSION=$(cat /root/version) > > ��� echo "Generating banner: $VERSION $RELEASE" > ��� sed s/VERSION/$VERSION/g /root/custom/banner > /etc/banner > ��� sed -i s/RELEASE/$RELEASE/g /etc/banner > > ��� touch /root/.banner_ok > fi > > [ Node 2 | node-2 ] echo "test" > /root/custom/scripts/banner.sh > -ash: can't create /root/custom/scripts/banner.sh: nonexistent directory > > > > I'm also noticing other apps fail because /etc doesn't exists yet after > UBIFS boot loading. > these 2 issues were not seen on 4.14.68. > > The bootlog doesn't show any error: > https://pastebin.com/raw/dJx47uBp > > > I'm only seeing these issues on UBIFS enabled volumes. > Reverting ("ubifs: xattr: Don't operate on deleted inodes") fixes these > weird issues. Do you also have the same problem on Linus's latest tree, or 4.18, with this change in the tree? thanks, greg k-h ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" @ 2018-09-13 9:46 ` gregkh 0 siblings, 0 replies; 7+ messages in thread From: gregkh @ 2018-09-13 9:46 UTC (permalink / raw) To: Koen Vandeputte; +Cc: linux-mtd, stable adding stable@ for stable kernel issues... On Thu, Sep 13, 2018 at 09:58:35AM +0200, Koen Vandeputte wrote: > Hi all, > > I'm currently in the process of updating the kernel version within OpenWrt. > (4.14.68 to 4.14.69) > > Testing shows some issues on devices using specifically UBIFS. > Altering a perfect valid writable file shows weird errors: > > > [ Node 2 | node-2 ] ls -l /root/custom/scripts/banner.sh > -rwxr-xr-x 1 root root 283 Sep 11 09:52 > /root/custom/scripts/banner.sh > > [ Node 2 | node-2 ] cat /root/custom/scripts/banner.sh > #!/bin/sh > > if [ ! -f /root/.banner_ok ] > then > RELEASE=$(cat /root/build_date) > VERSION=$(cat /root/version) > > echo "Generating banner: $VERSION $RELEASE" > sed s/VERSION/$VERSION/g /root/custom/banner > /etc/banner > sed -i s/RELEASE/$RELEASE/g /etc/banner > > touch /root/.banner_ok > fi > > [ Node 2 | node-2 ] echo "test" > /root/custom/scripts/banner.sh > -ash: can't create /root/custom/scripts/banner.sh: nonexistent directory > > > > I'm also noticing other apps fail because /etc doesn't exists yet after > UBIFS boot loading. > these 2 issues were not seen on 4.14.68. > > The bootlog doesn't show any error: > https://pastebin.com/raw/dJx47uBp > > > I'm only seeing these issues on UBIFS enabled volumes. > Reverting ("ubifs: xattr: Don't operate on deleted inodes") fixes these > weird issues. Do you also have the same problem on Linus's latest tree, or 4.18, with this change in the tree? thanks, greg k-h ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" 2018-09-13 9:46 ` gregkh (?) @ 2018-09-15 7:15 ` Richard Weinberger 2018-09-16 19:52 ` Koen Vandeputte -1 siblings, 1 reply; 7+ messages in thread From: Richard Weinberger @ 2018-09-15 7:15 UTC (permalink / raw) To: Greg KH; +Cc: koen.vandeputte, linux-mtd @ lists . infradead . org, stable Koen, On Thu, Sep 13, 2018 at 12:09 PM gregkh@linuxfoundation.org <gregkh@linuxfoundation.org> wrote: > > adding stable@ for stable kernel issues... > > On Thu, Sep 13, 2018 at 09:58:35AM +0200, Koen Vandeputte wrote: > > Hi all, > > > > I'm currently in the process of updating the kernel version within OpenWrt. > > (4.14.68 to 4.14.69) > > > > Testing shows some issues on devices using specifically UBIFS. > > Altering a perfect valid writable file shows weird errors: > > > > > > [ Node 2 | node-2 ] ls -l /root/custom/scripts/banner.sh > > -rwxr-xr-x 1 root root 283 Sep 11 09:52 > > /root/custom/scripts/banner.sh > > > > [ Node 2 | node-2 ] cat /root/custom/scripts/banner.sh > > #!/bin/sh > > > > if [ ! -f /root/.banner_ok ] > > then > > RELEASE=$(cat /root/build_date) > > VERSION=$(cat /root/version) > > > > echo "Generating banner: $VERSION $RELEASE" > > sed s/VERSION/$VERSION/g /root/custom/banner > /etc/banner > > sed -i s/RELEASE/$RELEASE/g /etc/banner > > > > touch /root/.banner_ok > > fi > > > > [ Node 2 | node-2 ] echo "test" > /root/custom/scripts/banner.sh > > -ash: can't create /root/custom/scripts/banner.sh: nonexistent directory > > > > > > > > I'm also noticing other apps fail because /etc doesn't exists yet after > > UBIFS boot loading. > > these 2 issues were not seen on 4.14.68. > > > > The bootlog doesn't show any error: > > https://pastebin.com/raw/dJx47uBp > > > > > > I'm only seeing these issues on UBIFS enabled volumes. > > Reverting ("ubifs: xattr: Don't operate on deleted inodes") fixes these > > weird issues. Please see my answer to the other thread. Always keep the patch author on CC and don't start multiple threads for the same issue on two mailing lists. Yes, I didn't answer for three days, I had no internet connection... -- Thanks, //richard ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" 2018-09-15 7:15 ` Richard Weinberger @ 2018-09-16 19:52 ` Koen Vandeputte 2018-09-17 9:38 ` Greg KH 0 siblings, 1 reply; 7+ messages in thread From: Koen Vandeputte @ 2018-09-16 19:52 UTC (permalink / raw) To: Richard Weinberger, Greg KH; +Cc: linux-mtd @ lists . infradead . org, stable On 15-09-18 09:15, Richard Weinberger wrote: > Koen, > > On Thu, Sep 13, 2018 at 12:09 PM gregkh@linuxfoundation.org > <gregkh@linuxfoundation.org> wrote: >> adding stable@ for stable kernel issues... >> >> On Thu, Sep 13, 2018 at 09:58:35AM +0200, Koen Vandeputte wrote: >>> Hi all, >>> >>> I'm currently in the process of updating the kernel version within OpenWrt. >>> (4.14.68 to 4.14.69) >>> >>> Testing shows some issues on devices using specifically UBIFS. >>> Altering a perfect valid writable file shows weird errors: >>> >>> >>> [ Node 2 | node-2 ] ls -l /root/custom/scripts/banner.sh >>> -rwxr-xr-x 1 root root 283 Sep 11 09:52 >>> /root/custom/scripts/banner.sh >>> >>> [ Node 2 | node-2 ] cat /root/custom/scripts/banner.sh >>> #!/bin/sh >>> >>> if [ ! -f /root/.banner_ok ] >>> then >>> RELEASE=$(cat /root/build_date) >>> VERSION=$(cat /root/version) >>> >>> echo "Generating banner: $VERSION $RELEASE" >>> sed s/VERSION/$VERSION/g /root/custom/banner > /etc/banner >>> sed -i s/RELEASE/$RELEASE/g /etc/banner >>> >>> touch /root/.banner_ok >>> fi >>> >>> [ Node 2 | node-2 ] echo "test" > /root/custom/scripts/banner.sh >>> -ash: can't create /root/custom/scripts/banner.sh: nonexistent directory >>> >>> >>> >>> I'm also noticing other apps fail because /etc doesn't exists yet after >>> UBIFS boot loading. >>> these 2 issues were not seen on 4.14.68. >>> >>> The bootlog doesn't show any error: >>> https://pastebin.com/raw/dJx47uBp >>> >>> >>> I'm only seeing these issues on UBIFS enabled volumes. >>> Reverting ("ubifs: xattr: Don't operate on deleted inodes") fixes these >>> weird issues. > Please see my answer to the other thread. > Always keep the patch author on CC and don't start multiple threads for the > same issue on two mailing lists. > Yes, I didn't answer for three days, I had no internet connection... > Richard, I indeed posted here too as we passed the 3 days marker, and I noticed 4.14.70 RC2 got staged. Also, a lot of developers tend to ignore questions unless the initial question got posted to the official mailinglist. Let's continue on the OpenWrt list as you propose, were the initial question was raised. Apologies for the (double) noise. Thank you, Koen ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" 2018-09-16 19:52 ` Koen Vandeputte @ 2018-09-17 9:38 ` Greg KH 2018-09-17 9:41 ` Richard Weinberger 0 siblings, 1 reply; 7+ messages in thread From: Greg KH @ 2018-09-17 9:38 UTC (permalink / raw) To: Koen Vandeputte Cc: Richard Weinberger, linux-mtd @ lists . infradead . org, stable On Sun, Sep 16, 2018 at 09:52:48PM +0200, Koen Vandeputte wrote: > > > On 15-09-18 09:15, Richard Weinberger wrote: > > Koen, > > > > On Thu, Sep 13, 2018 at 12:09 PM gregkh@linuxfoundation.org > > <gregkh@linuxfoundation.org> wrote: > > > adding stable@ for stable kernel issues... > > > > > > On Thu, Sep 13, 2018 at 09:58:35AM +0200, Koen Vandeputte wrote: > > > > Hi all, > > > > > > > > I'm currently in the process of updating the kernel version within OpenWrt. > > > > (4.14.68 to 4.14.69) > > > > > > > > Testing shows some issues on devices using specifically UBIFS. > > > > Altering a perfect valid writable file shows weird errors: > > > > > > > > > > > > [ Node 2 | node-2 ] ls -l /root/custom/scripts/banner.sh > > > > -rwxr-xr-x 1 root root 283 Sep 11 09:52 > > > > /root/custom/scripts/banner.sh > > > > > > > > [ Node 2 | node-2 ] cat /root/custom/scripts/banner.sh > > > > #!/bin/sh > > > > > > > > if [ ! -f /root/.banner_ok ] > > > > then > > > > RELEASE=$(cat /root/build_date) > > > > VERSION=$(cat /root/version) > > > > > > > > echo "Generating banner: $VERSION $RELEASE" > > > > sed s/VERSION/$VERSION/g /root/custom/banner > /etc/banner > > > > sed -i s/RELEASE/$RELEASE/g /etc/banner > > > > > > > > touch /root/.banner_ok > > > > fi > > > > > > > > [ Node 2 | node-2 ] echo "test" > /root/custom/scripts/banner.sh > > > > -ash: can't create /root/custom/scripts/banner.sh: nonexistent directory > > > > > > > > > > > > > > > > I'm also noticing other apps fail because /etc doesn't exists yet after > > > > UBIFS boot loading. > > > > these 2 issues were not seen on 4.14.68. > > > > > > > > The bootlog doesn't show any error: > > > > https://pastebin.com/raw/dJx47uBp > > > > > > > > > > > > I'm only seeing these issues on UBIFS enabled volumes. > > > > Reverting ("ubifs: xattr: Don't operate on deleted inodes") fixes these > > > > weird issues. > > Please see my answer to the other thread. > > Always keep the patch author on CC and don't start multiple threads for the > > same issue on two mailing lists. > > Yes, I didn't answer for three days, I had no internet connection... > > > Richard, > > I indeed posted here too as we passed the 3 days marker, and I noticed > 4.14.70 RC2 got staged. > Also, a lot of developers tend to ignore questions unless the initial > question got posted to the official mailinglist. > > Let's continue on the OpenWrt list as you propose, were the initial question > was raised. > > Apologies for the (double) noise. I will assume that if the resolution is that something needs to be done to the stable releases and a patch added/reverted, someone will eventually tell me? thanks, greg k-h ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" 2018-09-17 9:38 ` Greg KH @ 2018-09-17 9:41 ` Richard Weinberger 0 siblings, 0 replies; 7+ messages in thread From: Richard Weinberger @ 2018-09-17 9:41 UTC (permalink / raw) To: Greg KH, linux-mtd; +Cc: Koen Vandeputte, stable Am Montag, 17. September 2018, 11:38:04 CEST schrieb Greg KH: > On Sun, Sep 16, 2018 at 09:52:48PM +0200, Koen Vandeputte wrote: > > > > > > On 15-09-18 09:15, Richard Weinberger wrote: > > > Koen, > > > > > > On Thu, Sep 13, 2018 at 12:09 PM gregkh@linuxfoundation.org > > > <gregkh@linuxfoundation.org> wrote: > > > > adding stable@ for stable kernel issues... > > > > > > > > On Thu, Sep 13, 2018 at 09:58:35AM +0200, Koen Vandeputte wrote: > > > > > Hi all, > > > > > > > > > > I'm currently in the process of updating the kernel version within OpenWrt. > > > > > (4.14.68 to 4.14.69) > > > > > > > > > > Testing shows some issues on devices using specifically UBIFS. > > > > > Altering a perfect valid writable file shows weird errors: > > > > > > > > > > > > > > > [ Node 2 | node-2 ] ls -l /root/custom/scripts/banner.sh > > > > > -rwxr-xr-x 1 root root 283 Sep 11 09:52 > > > > > /root/custom/scripts/banner.sh > > > > > > > > > > [ Node 2 | node-2 ] cat /root/custom/scripts/banner.sh > > > > > #!/bin/sh > > > > > > > > > > if [ ! -f /root/.banner_ok ] > > > > > then > > > > > RELEASE=$(cat /root/build_date) > > > > > VERSION=$(cat /root/version) > > > > > > > > > > echo "Generating banner: $VERSION $RELEASE" > > > > > sed s/VERSION/$VERSION/g /root/custom/banner > /etc/banner > > > > > sed -i s/RELEASE/$RELEASE/g /etc/banner > > > > > > > > > > touch /root/.banner_ok > > > > > fi > > > > > > > > > > [ Node 2 | node-2 ] echo "test" > /root/custom/scripts/banner.sh > > > > > -ash: can't create /root/custom/scripts/banner.sh: nonexistent directory > > > > > > > > > > > > > > > > > > > > I'm also noticing other apps fail because /etc doesn't exists yet after > > > > > UBIFS boot loading. > > > > > these 2 issues were not seen on 4.14.68. > > > > > > > > > > The bootlog doesn't show any error: > > > > > https://pastebin.com/raw/dJx47uBp > > > > > > > > > > > > > > > I'm only seeing these issues on UBIFS enabled volumes. > > > > > Reverting ("ubifs: xattr: Don't operate on deleted inodes") fixes these > > > > > weird issues. > > > Please see my answer to the other thread. > > > Always keep the patch author on CC and don't start multiple threads for the > > > same issue on two mailing lists. > > > Yes, I didn't answer for three days, I had no internet connection... > > > > > Richard, > > > > I indeed posted here too as we passed the 3 days marker, and I noticed > > 4.14.70 RC2 got staged. > > Also, a lot of developers tend to ignore questions unless the initial > > question got posted to the official mailinglist. > > > > Let's continue on the OpenWrt list as you propose, were the initial question > > was raised. > > > > Apologies for the (double) noise. > > I will assume that if the resolution is that something needs to be done > to the stable releases and a patch added/reverted, someone will > eventually tell me? The revert was sent yesterday to the mailinglist. I'll send Linus^Wyou a pull request soon and from here it can go into stable. Thanks, //richard ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-09-17 15:08 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-09-13 7:58 ubifs: regression since "ubifs: xattr: Don't operate on deleted inodes" Koen Vandeputte 2018-09-13 9:46 ` gregkh 2018-09-13 9:46 ` gregkh 2018-09-15 7:15 ` Richard Weinberger 2018-09-16 19:52 ` Koen Vandeputte 2018-09-17 9:38 ` Greg KH 2018-09-17 9:41 ` Richard Weinberger
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.