All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] [ubifs] add update vfs super block infomation when remount to ro mode.
@ 2010-04-08  7:00 ZhangJieJing
  2010-04-08  7:13 ` Artem Bityutskiy
  0 siblings, 1 reply; 18+ messages in thread
From: ZhangJieJing @ 2010-04-08  7:00 UTC (permalink / raw)
  To: linux-mtd

If some read/write error happens(eg.CRC error), ubifs make self a read
only mode,
but the vfs infomation still not update.
This patch add this also make /proc/mounts update.

Signed-off-by: zhangjiejing <kzjeef@gmail.com>

Change-Id: I709ada21d1f6168d2e785eac389510e2b5edb2a4
---
 fs/ubifs/io.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
index 0168271..d33c233 100644
--- a/fs/ubifs/io.c
+++ b/fs/ubifs/io.c
@@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
        if (!c->ro_media) {
                c->ro_media = 1;
                c->no_chk_data_crc = 0;
+                c->vfs_sb->s_flags |= MS_RDONLY;
                ubifs_warn("switched to read-only mode, error %d", err);
                dbg_dump_stack();
        }
--
1.6.3.3

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* Re: [PATCH] [ubifs] add update vfs super block infomation when remount to ro mode.
  2010-04-08  7:00 [PATCH] [ubifs] add update vfs super block infomation when remount to ro mode ZhangJieJing
@ 2010-04-08  7:13 ` Artem Bityutskiy
  0 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-08  7:13 UTC (permalink / raw)
  To: ZhangJieJing; +Cc: linux-mtd

Hi,

On Thu, 2010-04-08 at 15:00 +0800, ZhangJieJing wrote:
> If some read/write error happens(eg.CRC error), ubifs make self a read
> only mode,
> but the vfs infomation still not update.
> This patch add this also make /proc/mounts update.
> 
> Signed-off-by: zhangjiejing <kzjeef@gmail.com>
> 
> Change-Id: I709ada21d1f6168d2e785eac389510e2b5edb2a4
> ---
>  fs/ubifs/io.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)

Could you please re-send this patch with CC to
linux-fsdevel@vger.kernel.org


-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount  to ro mode
  2010-04-29  6:36         ` Zhang JieJing
@ 2010-04-29  6:44           ` Artem Bityutskiy
  -1 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-29  6:44 UTC (permalink / raw)
  To: Zhang JieJing; +Cc: linux-fsdevel, linux-mtd

On Thu, 2010-04-29 at 14:36 +0800, Zhang JieJing wrote:
> 2010/4/29 Artem Bityutskiy <dedekind1@gmail.com>:
> > On Thu, 2010-04-29 at 10:14 +0800, Zhang JieJing wrote:
> >> (resend since HTML mail rejected by linux-fsdevel.)
> >> Hi Artem,
> >>
> >> I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
> >> patch, It is very clean patched, so, maybe your git repo code is very
> >> new, so the patch can't cleaned apply.
> >>
> >> This is my first patch give back to upstream, even I read the kernel
> >> document 'how-to send patch' but still something leaked.
> >> If I did something wrong, Please show me the right way or something like that,
> >> Thanks for your time.
> >
> > Did you save your own mail _from the mailing_ list and apply it with git
> > am ?
> 
> Really sorry for that, I tried use my mail sent to mailing list, I
> found it appear the same error,
> I learned this important lesson, never send patch by Rich formatting.
> 
> I will send this patch by plain text after this mail, Maybe there is
> some way to send it in the same mail with patch and answer ?

I already applied your patch yesterday, you do not have to re-send.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount  to ro mode
@ 2010-04-29  6:44           ` Artem Bityutskiy
  0 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-29  6:44 UTC (permalink / raw)
  To: Zhang JieJing; +Cc: linux-fsdevel, linux-mtd

On Thu, 2010-04-29 at 14:36 +0800, Zhang JieJing wrote:
> 2010/4/29 Artem Bityutskiy <dedekind1@gmail.com>:
> > On Thu, 2010-04-29 at 10:14 +0800, Zhang JieJing wrote:
> >> (resend since HTML mail rejected by linux-fsdevel.)
> >> Hi Artem,
> >>
> >> I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
> >> patch, It is very clean patched, so, maybe your git repo code is very
> >> new, so the patch can't cleaned apply.
> >>
> >> This is my first patch give back to upstream, even I read the kernel
> >> document 'how-to send patch' but still something leaked.
> >> If I did something wrong, Please show me the right way or something like that,
> >> Thanks for your time.
> >
> > Did you save your own mail _from the mailing_ list and apply it with git
> > am ?
> 
> Really sorry for that, I tried use my mail sent to mailing list, I
> found it appear the same error,
> I learned this important lesson, never send patch by Rich formatting.
> 
> I will send this patch by plain text after this mail, Maybe there is
> some way to send it in the same mail with patch and answer ?

I already applied your patch yesterday, you do not have to re-send.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
  2010-04-29  3:46       ` Artem Bityutskiy
@ 2010-04-29  6:38         ` Zhang JieJing
  -1 siblings, 0 replies; 18+ messages in thread
From: Zhang JieJing @ 2010-04-29  6:38 UTC (permalink / raw)
  To: dedekind1; +Cc: linux-fsdevel, linux-mtd

If some read/write error happens, ubifs make self a read only mode,
but the vfs infomation still not update.
This patch add this also make /proc/mounts update.

Signed-off-by: zhangjiejing <kzjeef@gmail.com>

Change-Id: I709ada21d1f6168d2e785eac389510e2b5edb2a4
---
 fs/ubifs/io.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
index 0168271..d33c233 100644
--- a/fs/ubifs/io.c
+++ b/fs/ubifs/io.c
@@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
 	if (!c->ro_media) {
 		c->ro_media = 1;
 		c->no_chk_data_crc = 0;
+                c->vfs_sb->s_flags |= MS_RDONLY;
 		ubifs_warn("switched to read-only mode, error %d", err);
 		dbg_dump_stack();
 	}
-- 
1.6.3.3

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
@ 2010-04-29  6:38         ` Zhang JieJing
  0 siblings, 0 replies; 18+ messages in thread
From: Zhang JieJing @ 2010-04-29  6:38 UTC (permalink / raw)
  To: dedekind1; +Cc: linux-fsdevel, linux-mtd

If some read/write error happens, ubifs make self a read only mode,
but the vfs infomation still not update.
This patch add this also make /proc/mounts update.

Signed-off-by: zhangjiejing <kzjeef@gmail.com>

Change-Id: I709ada21d1f6168d2e785eac389510e2b5edb2a4
---
 fs/ubifs/io.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
index 0168271..d33c233 100644
--- a/fs/ubifs/io.c
+++ b/fs/ubifs/io.c
@@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
 	if (!c->ro_media) {
 		c->ro_media = 1;
 		c->no_chk_data_crc = 0;
+                c->vfs_sb->s_flags |= MS_RDONLY;
 		ubifs_warn("switched to read-only mode, error %d", err);
 		dbg_dump_stack();
 	}
-- 
1.6.3.3

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
  2010-04-29  3:46       ` Artem Bityutskiy
@ 2010-04-29  6:36         ` Zhang JieJing
  -1 siblings, 0 replies; 18+ messages in thread
From: Zhang JieJing @ 2010-04-29  6:36 UTC (permalink / raw)
  To: dedekind1; +Cc: linux-fsdevel, linux-mtd

2010/4/29 Artem Bityutskiy <dedekind1@gmail.com>:
> On Thu, 2010-04-29 at 10:14 +0800, Zhang JieJing wrote:
>> (resend since HTML mail rejected by linux-fsdevel.)
>> Hi Artem,
>>
>> I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
>> patch, It is very clean patched, so, maybe your git repo code is very
>> new, so the patch can't cleaned apply.
>>
>> This is my first patch give back to upstream, even I read the kernel
>> document 'how-to send patch' but still something leaked.
>> If I did something wrong, Please show me the right way or something like that,
>> Thanks for your time.
>
> Did you save your own mail _from the mailing_ list and apply it with git
> am ?

Really sorry for that, I tried use my mail sent to mailing list, I
found it appear the same error,
I learned this important lesson, never send patch by Rich formatting.

I will send this patch by plain text after this mail, Maybe there is
some way to send it in the same mail with patch and answer ?
>
> --
> Best Regards,
> Artem Bityutskiy (Артём Битюцкий)
>
>

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
@ 2010-04-29  6:36         ` Zhang JieJing
  0 siblings, 0 replies; 18+ messages in thread
From: Zhang JieJing @ 2010-04-29  6:36 UTC (permalink / raw)
  To: dedekind1; +Cc: linux-fsdevel, linux-mtd

2010/4/29 Artem Bityutskiy <dedekind1@gmail.com>:
> On Thu, 2010-04-29 at 10:14 +0800, Zhang JieJing wrote:
>> (resend since HTML mail rejected by linux-fsdevel.)
>> Hi Artem,
>>
>> I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
>> patch, It is very clean patched, so, maybe your git repo code is very
>> new, so the patch can't cleaned apply.
>>
>> This is my first patch give back to upstream, even I read the kernel
>> document 'how-to send patch' but still something leaked.
>> If I did something wrong, Please show me the right way or something like that,
>> Thanks for your time.
>
> Did you save your own mail _from the mailing_ list and apply it with git
> am ?

Really sorry for that, I tried use my mail sent to mailing list, I
found it appear the same error,
I learned this important lesson, never send patch by Rich formatting.

I will send this patch by plain text after this mail, Maybe there is
some way to send it in the same mail with patch and answer ?
>
> --
> Best Regards,
> Artem Bityutskiy (Артём Битюцкий)
>
>

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount  to ro mode
  2010-04-29  2:14     ` Zhang JieJing
@ 2010-04-29  3:46       ` Artem Bityutskiy
  -1 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-29  3:46 UTC (permalink / raw)
  To: Zhang JieJing; +Cc: linux-mtd, linux-fsdevel

On Thu, 2010-04-29 at 10:14 +0800, Zhang JieJing wrote:
> (resend since HTML mail rejected by linux-fsdevel.)
> Hi Artem,
> 
> I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
> patch, It is very clean patched, so, maybe your git repo code is very
> new, so the patch can't cleaned apply.
> 
> This is my first patch give back to upstream, even I read the kernel
> document 'how-to send patch' but still something leaked.
> If I did something wrong, Please show me the right way or something like that,
> Thanks for your time.

Did you save your own mail _from the mailing_ list and apply it with git
am ?

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount  to ro mode
@ 2010-04-29  3:46       ` Artem Bityutskiy
  0 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-29  3:46 UTC (permalink / raw)
  To: Zhang JieJing; +Cc: linux-fsdevel, linux-mtd

On Thu, 2010-04-29 at 10:14 +0800, Zhang JieJing wrote:
> (resend since HTML mail rejected by linux-fsdevel.)
> Hi Artem,
> 
> I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
> patch, It is very clean patched, so, maybe your git repo code is very
> new, so the patch can't cleaned apply.
> 
> This is my first patch give back to upstream, even I read the kernel
> document 'how-to send patch' but still something leaked.
> If I did something wrong, Please show me the right way or something like that,
> Thanks for your time.

Did you save your own mail _from the mailing_ list and apply it with git
am ?

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
  2010-04-28  7:24   ` Artem Bityutskiy
@ 2010-04-29  2:14     ` Zhang JieJing
  -1 siblings, 0 replies; 18+ messages in thread
From: Zhang JieJing @ 2010-04-29  2:14 UTC (permalink / raw)
  To: dedekind1; +Cc: linux-mtd, linux-fsdevel

(resend since HTML mail rejected by linux-fsdevel.)
Hi Artem,

I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
patch, It is very clean patched, so, maybe your git repo code is very
new, so the patch can't cleaned apply.

This is my first patch give back to upstream, even I read the kernel
document 'how-to send patch' but still something leaked.
If I did something wrong, Please show me the right way or something like that,
Thanks for your time.

---
Best regards,
Zhang Jiejing



2010/4/28 Artem Bityutskiy <dedekind1@gmail.com>:
> On Fri, 2010-04-16 at 11:36 +0800, ZhangJieJing wrote:
>> If some read/write error happens(eg.CRC error), ubifs make self a read
>> only mode,
>> but the vfs infomation still not update.
>> This patch add this also make /proc/mounts update.
>>
>> Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
>> ---
>>  fs/ubifs/io.c |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
>> index 0168271..d33c233 100644
>> --- a/fs/ubifs/io.c
>> +++ b/fs/ubifs/io.c
>> @@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
>>        if (!c->ro_media) {
>>                c->ro_media = 1;
>>                c->no_chk_data_crc = 0;
>> +              c->vfs_sb->s_flags |= MS_RDONLY;
>>                ubifs_warn("switched to read-only mode, error %d", err);
>>                dbg_dump_stack();
>>        }
>> --
>> 1.6.3.3
>
> Your patch is not applicable. Try to save your own e-mail and apply the
> patch with 'git am'.
>
> Applying: UBIFS: mark VFS SB RO too
> fatal: corrupt patch at line 9
> Patch failed at 0001 UBIFS: mark VFS SB RO too
> When you have resolved this problem run "git am -i --resolved".
> If you would prefer to skip this patch, instead run "git am -i --skip".
> To restore the original branch and stop patching run "git am -i
> --abort".
> [dedekind@sauron ubifs-2.6]\$ patch -p1 < .git/rebase-apply/patch
> patching file fs/ubifs/io.c
> patch: **** malformed patch at line 9: �� � � if (!c->ro_media) {
>
>
> But I've fixed this up and applied manually. It is now in the
> ubifs-2.6.git tree, thanks.
>
> --
> Best Regards,
> Artem Bityutskiy (Артём Битюцкий)
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
@ 2010-04-29  2:14     ` Zhang JieJing
  0 siblings, 0 replies; 18+ messages in thread
From: Zhang JieJing @ 2010-04-29  2:14 UTC (permalink / raw)
  To: dedekind1; +Cc: linux-fsdevel, linux-mtd

(resend since HTML mail rejected by linux-fsdevel.)
Hi Artem,

I just clone the  git://git.infradead.org/ubifs-2.6.git, and git am my
patch, It is very clean patched, so, maybe your git repo code is very
new, so the patch can't cleaned apply.

This is my first patch give back to upstream, even I read the kernel
document 'how-to send patch' but still something leaked.
If I did something wrong, Please show me the right way or something like that,
Thanks for your time.

---
Best regards,
Zhang Jiejing



2010/4/28 Artem Bityutskiy <dedekind1@gmail.com>:
> On Fri, 2010-04-16 at 11:36 +0800, ZhangJieJing wrote:
>> If some read/write error happens(eg.CRC error), ubifs make self a read
>> only mode,
>> but the vfs infomation still not update.
>> This patch add this also make /proc/mounts update.
>>
>> Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
>> ---
>>  fs/ubifs/io.c |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
>> index 0168271..d33c233 100644
>> --- a/fs/ubifs/io.c
>> +++ b/fs/ubifs/io.c
>> @@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
>>        if (!c->ro_media) {
>>                c->ro_media = 1;
>>                c->no_chk_data_crc = 0;
>> +              c->vfs_sb->s_flags |= MS_RDONLY;
>>                ubifs_warn("switched to read-only mode, error %d", err);
>>                dbg_dump_stack();
>>        }
>> --
>> 1.6.3.3
>
> Your patch is not applicable. Try to save your own e-mail and apply the
> patch with 'git am'.
>
> Applying: UBIFS: mark VFS SB RO too
> fatal: corrupt patch at line 9
> Patch failed at 0001 UBIFS: mark VFS SB RO too
> When you have resolved this problem run "git am -i --resolved".
> If you would prefer to skip this patch, instead run "git am -i --skip".
> To restore the original branch and stop patching run "git am -i
> --abort".
> [dedekind@sauron ubifs-2.6]\$ patch -p1 < .git/rebase-apply/patch
> patching file fs/ubifs/io.c
> patch: **** malformed patch at line 9: �� � � if (!c->ro_media) {
>
>
> But I've fixed this up and applied manually. It is now in the
> ubifs-2.6.git tree, thanks.
>
> --
> Best Regards,
> Artem Bityutskiy (Артём Битюцкий)
>
>

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to  ro mode
  2010-04-16  3:36 ` ZhangJieJing
@ 2010-04-28  7:24   ` Artem Bityutskiy
  -1 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-28  7:24 UTC (permalink / raw)
  To: ZhangJieJing; +Cc: linux-mtd, linux-fsdevel

On Fri, 2010-04-16 at 11:36 +0800, ZhangJieJing wrote:
> If some read/write error happens(eg.CRC error), ubifs make self a read
> only mode,
> but the vfs infomation still not update.
> This patch add this also make /proc/mounts update.
> 
> Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
> ---
>  fs/ubifs/io.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
> index 0168271..d33c233 100644
> --- a/fs/ubifs/io.c
> +++ b/fs/ubifs/io.c
> @@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
>        if (!c->ro_media) {
>                c->ro_media = 1;
>                c->no_chk_data_crc = 0;
> +              c->vfs_sb->s_flags |= MS_RDONLY;
>                ubifs_warn("switched to read-only mode, error %d", err);
>                dbg_dump_stack();
>        }
> --
> 1.6.3.3

Your patch is not applicable. Try to save your own e-mail and apply the
patch with 'git am'.

Applying: UBIFS: mark VFS SB RO too
fatal: corrupt patch at line 9
Patch failed at 0001 UBIFS: mark VFS SB RO too
When you have resolved this problem run "git am -i --resolved".
If you would prefer to skip this patch, instead run "git am -i --skip".
To restore the original branch and stop patching run "git am -i
--abort".
[dedekind@sauron ubifs-2.6]\$ patch -p1 < .git/rebase-apply/patch 
patching file fs/ubifs/io.c
patch: **** malformed patch at line 9: �� � � if (!c->ro_media) {


But I've fixed this up and applied manually. It is now in the
ubifs-2.6.git tree, thanks.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to  ro mode
@ 2010-04-28  7:24   ` Artem Bityutskiy
  0 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-28  7:24 UTC (permalink / raw)
  To: ZhangJieJing; +Cc: linux-fsdevel, linux-mtd

On Fri, 2010-04-16 at 11:36 +0800, ZhangJieJing wrote:
> If some read/write error happens(eg.CRC error), ubifs make self a read
> only mode,
> but the vfs infomation still not update.
> This patch add this also make /proc/mounts update.
> 
> Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
> ---
>  fs/ubifs/io.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
> index 0168271..d33c233 100644
> --- a/fs/ubifs/io.c
> +++ b/fs/ubifs/io.c
> @@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
>        if (!c->ro_media) {
>                c->ro_media = 1;
>                c->no_chk_data_crc = 0;
> +              c->vfs_sb->s_flags |= MS_RDONLY;
>                ubifs_warn("switched to read-only mode, error %d", err);
>                dbg_dump_stack();
>        }
> --
> 1.6.3.3

Your patch is not applicable. Try to save your own e-mail and apply the
patch with 'git am'.

Applying: UBIFS: mark VFS SB RO too
fatal: corrupt patch at line 9
Patch failed at 0001 UBIFS: mark VFS SB RO too
When you have resolved this problem run "git am -i --resolved".
If you would prefer to skip this patch, instead run "git am -i --skip".
To restore the original branch and stop patching run "git am -i
--abort".
[dedekind@sauron ubifs-2.6]\$ patch -p1 < .git/rebase-apply/patch 
patching file fs/ubifs/io.c
patch: **** malformed patch at line 9: �� � � if (!c->ro_media) {


But I've fixed this up and applied manually. It is now in the
ubifs-2.6.git tree, thanks.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to  ro mode
  2010-04-16  3:36 ` ZhangJieJing
@ 2010-04-16 19:10   ` Artem Bityutskiy
  -1 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-16 19:10 UTC (permalink / raw)
  To: ZhangJieJing; +Cc: linux-mtd, linux-fsdevel

On Fri, 2010-04-16 at 11:36 +0800, ZhangJieJing wrote:
> If some read/write error happens(eg.CRC error), ubifs make self a read
> only mode,
> but the vfs infomation still not update.
> This patch add this also make /proc/mounts update.
> 
> Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
> ---
>  fs/ubifs/io.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
> index 0168271..d33c233 100644
> --- a/fs/ubifs/io.c
> +++ b/fs/ubifs/io.c
> @@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
>        if (!c->ro_media) {
>                c->ro_media = 1;
>                c->no_chk_data_crc = 0;
> +              c->vfs_sb->s_flags |= MS_RDONLY;
>                ubifs_warn("switched to read-only mode, error %d", err);
>                dbg_dump_stack();
>        }

Looks ok for me, thanks, I'll take it to the ubifs-2.6.git tree as soon
as I return to my work office (in 2 weeks).

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] ubifs: add update vfs super block infomation when remount to  ro mode
@ 2010-04-16 19:10   ` Artem Bityutskiy
  0 siblings, 0 replies; 18+ messages in thread
From: Artem Bityutskiy @ 2010-04-16 19:10 UTC (permalink / raw)
  To: ZhangJieJing; +Cc: linux-fsdevel, linux-mtd

On Fri, 2010-04-16 at 11:36 +0800, ZhangJieJing wrote:
> If some read/write error happens(eg.CRC error), ubifs make self a read
> only mode,
> but the vfs infomation still not update.
> This patch add this also make /proc/mounts update.
> 
> Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
> ---
>  fs/ubifs/io.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
> index 0168271..d33c233 100644
> --- a/fs/ubifs/io.c
> +++ b/fs/ubifs/io.c
> @@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
>        if (!c->ro_media) {
>                c->ro_media = 1;
>                c->no_chk_data_crc = 0;
> +              c->vfs_sb->s_flags |= MS_RDONLY;
>                ubifs_warn("switched to read-only mode, error %d", err);
>                dbg_dump_stack();
>        }

Looks ok for me, thanks, I'll take it to the ubifs-2.6.git tree as soon
as I return to my work office (in 2 weeks).

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
@ 2010-04-16  3:36 ` ZhangJieJing
  0 siblings, 0 replies; 18+ messages in thread
From: ZhangJieJing @ 2010-04-16  3:36 UTC (permalink / raw)
  To: linux-mtd; +Cc: linux-fsdevel, Artem Bityutskiy

If some read/write error happens(eg.CRC error), ubifs make self a read
only mode,
but the vfs infomation still not update.
This patch add this also make /proc/mounts update.

Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
---
 fs/ubifs/io.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
index 0168271..d33c233 100644
--- a/fs/ubifs/io.c
+++ b/fs/ubifs/io.c
@@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
       if (!c->ro_media) {
               c->ro_media = 1;
               c->no_chk_data_crc = 0;
+              c->vfs_sb->s_flags |= MS_RDONLY;
               ubifs_warn("switched to read-only mode, error %d", err);
               dbg_dump_stack();
       }
--
1.6.3.3
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [PATCH] ubifs: add update vfs super block infomation when remount to ro mode
@ 2010-04-16  3:36 ` ZhangJieJing
  0 siblings, 0 replies; 18+ messages in thread
From: ZhangJieJing @ 2010-04-16  3:36 UTC (permalink / raw)
  To: linux-mtd; +Cc: linux-fsdevel, Artem Bityutskiy

If some read/write error happens(eg.CRC error), ubifs make self a read
only mode,
but the vfs infomation still not update.
This patch add this also make /proc/mounts update.

Signed-off-by: Zhang Jiejing <kzjeef@gmail.com>
---
 fs/ubifs/io.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
index 0168271..d33c233 100644
--- a/fs/ubifs/io.c
+++ b/fs/ubifs/io.c
@@ -63,6 +63,7 @@ void ubifs_ro_mode(struct ubifs_info *c, int err)
       if (!c->ro_media) {
               c->ro_media = 1;
               c->no_chk_data_crc = 0;
+              c->vfs_sb->s_flags |= MS_RDONLY;
               ubifs_warn("switched to read-only mode, error %d", err);
               dbg_dump_stack();
       }
--
1.6.3.3

^ permalink raw reply related	[flat|nested] 18+ messages in thread

end of thread, other threads:[~2010-04-29  6:50 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-04-08  7:00 [PATCH] [ubifs] add update vfs super block infomation when remount to ro mode ZhangJieJing
2010-04-08  7:13 ` Artem Bityutskiy
2010-04-16  3:36 [PATCH] ubifs: " ZhangJieJing
2010-04-16  3:36 ` ZhangJieJing
2010-04-16 19:10 ` Artem Bityutskiy
2010-04-16 19:10   ` Artem Bityutskiy
2010-04-28  7:24 ` Artem Bityutskiy
2010-04-28  7:24   ` Artem Bityutskiy
2010-04-29  2:14   ` Zhang JieJing
2010-04-29  2:14     ` Zhang JieJing
2010-04-29  3:46     ` Artem Bityutskiy
2010-04-29  3:46       ` Artem Bityutskiy
2010-04-29  6:36       ` Zhang JieJing
2010-04-29  6:36         ` Zhang JieJing
2010-04-29  6:44         ` Artem Bityutskiy
2010-04-29  6:44           ` Artem Bityutskiy
2010-04-29  6:38       ` Zhang JieJing
2010-04-29  6:38         ` Zhang JieJing

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.