Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home
« Groups Home
exofs: drop lock/unlock super
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  7 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Marco Stornelli  
View profile  
 More options Oct 6 2012, 6:50 am
Newsgroups: linux.kernel
From: Marco Stornelli <marco.storne...@gmail.com>
Date: Sat, 06 Oct 2012 12:50:02 +0200
Local: Sat, Oct 6 2012 6:50 am
Subject: [PATCH 1/8] exofs: drop lock/unlock super
Removed lock/unlock super.

Acked-by: Artem Bityutskiy <artem.bityuts...@linux.intel.com>
Acked-by: Boaz Harrosh <bharr...@panasas.com>
Signed-off-by: Marco Stornelli <marco.storne...@gmail.com>
---
 fs/exofs/super.c |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/fs/exofs/super.c b/fs/exofs/super.c
index 59e3bbf..5e59280 100644
--- a/fs/exofs/super.c
+++ b/fs/exofs/super.c
@@ -389,8 +389,6 @@ static int exofs_sync_fs(struct super_block *sb, int wait)
        if (unlikely(ret))
                goto out;

-       lock_super(sb);
-
        ios->length = offsetof(struct exofs_fscb, s_dev_table_oid);
        memset(fscb, 0, ios->length);
        fscb->s_nextid = cpu_to_le64(sbi->s_nextid);
@@ -406,8 +404,6 @@ static int exofs_sync_fs(struct super_block *sb, int wait)
        if (unlikely(ret))
                EXOFS_ERR("%s: ore_write failed.\n", __func__);

-
-       unlock_super(sb);
 out:
        EXOFS_DBGMSG("s_nextid=0x%llx ret=%d\n", _LLU(sbi->s_nextid), ret);
        ore_put_io_state(ios);
--
1.7.3.4
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Boaz Harrosh  
View profile  
 More options Oct 8 2012, 6:00 pm
Newsgroups: linux.kernel
From: Boaz Harrosh <bharr...@panasas.com>
Date: Tue, 09 Oct 2012 00:00:02 +0200
Local: Mon, Oct 8 2012 6:00 pm
Subject: Re: [PATCH 1/8] exofs: drop lock/unlock super
On 10/06/2012 03:38 AM, Marco Stornelli wrote:

> Removed lock/unlock super.

Hi Marco

I was sure you guys where pushing this patch through some
vfs tree. (Hence my Acked-by below). I have just sent Linus
a pull request for the 3.7 Kernel. I could perhaps append this
one and resend.

I do want this patch, and it is completely independent and can
go through my tree. Please tell me what do you want to do with
this patch, should I push it? Or are you pushing it through Al

Thanks
Boaz

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Al Viro  
View profile  
 More options Oct 8 2012, 7:30 pm
Newsgroups: linux.kernel
From: Al Viro <v...@ZenIV.linux.org.uk>
Date: Tue, 09 Oct 2012 01:30:02 +0200
Local: Mon, Oct 8 2012 7:30 pm
Subject: Re: [PATCH 1/8] exofs: drop lock/unlock super

On Mon, Oct 08, 2012 at 02:58:05PM -0700, Boaz Harrosh wrote:
> Hi Marco

> I was sure you guys where pushing this patch through some
> vfs tree. (Hence my Acked-by below). I have just sent Linus
> a pull request for the 3.7 Kernel. I could perhaps append this
> one and resend.

> I do want this patch, and it is completely independent and can
> go through my tree. Please tell me what do you want to do with
> this patch, should I push it? Or are you pushing it through Al

This series is slated for tonight push; I'm testing the tree right
now...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Marco Stornelli  
View profile  
 More options Oct 9 2012, 12:10 pm
Newsgroups: linux.kernel
From: Marco Stornelli <marco.storne...@gmail.com>
Date: Tue, 09 Oct 2012 18:10:04 +0200
Local: Tues, Oct 9 2012 12:10 pm
Subject: Re: [PATCH 1/8] exofs: drop lock/unlock super
Il 09/10/2012 01:24, Al Viro ha scritto:

> On Mon, Oct 08, 2012 at 02:58:05PM -0700, Boaz Harrosh wrote:
>> Hi Marco

>> I was sure you guys where pushing this patch through some
>> vfs tree. (Hence my Acked-by below). I have just sent Linus
>> a pull request for the 3.7 Kernel. I could perhaps append this
>> one and resend.

>> I do want this patch, and it is completely independent and can
>> go through my tree. Please tell me what do you want to do with
>> this patch, should I push it? Or are you pushing it through Al

> This series is slated for tonight push; I'm testing the tree right
> now...

Thanks Al. For me there aren't problems to push it via Al or via other
trees. For example the patch for ext4 is in Ted's tree. You tell me what
you want and I do it :)

Marco
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Al Viro  
View profile  
 More options Oct 9 2012, 12:50 pm
Newsgroups: linux.kernel
From: Al Viro <v...@ZenIV.linux.org.uk>
Date: Tue, 09 Oct 2012 18:50:01 +0200
Local: Tues, Oct 9 2012 12:50 pm
Subject: Re: [PATCH 1/8] exofs: drop lock/unlock super

On Tue, Oct 09, 2012 at 05:55:12PM +0200, Marco Stornelli wrote:
> Thanks Al. For me there aren't problems to push it via Al or via
> other trees. For example the patch for ext4 is in Ted's tree. You
> tell me what you want and I do it :)

Um...  Then I'd better drop the ext4 part and hold the final one back,
right?  Are there other commits I need to skip to avoid duplicates with
the stuff already in other trees?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Theodore Ts'o  
View profile  
 More options Oct 9 2012, 1:50 pm
Newsgroups: linux.kernel
From: Theodore Ts'o <ty...@mit.edu>
Date: Tue, 09 Oct 2012 19:50:02 +0200
Local: Tues, Oct 9 2012 1:50 pm
Subject: Re: [PATCH 1/8] exofs: drop lock/unlock super
Just to confirm, the ext4 lock_super removal is commit 07724f98978a,
and Linus has already pulled it into his tree.

                                     - Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Marco Stornelli  
View profile  
 More options Oct 10 2012, 4:10 am
Newsgroups: linux.kernel
From: Marco Stornelli <marco.storne...@gmail.com>
Date: Wed, 10 Oct 2012 10:10:02 +0200
Local: Wed, Oct 10 2012 4:10 am
Subject: Re: [PATCH 1/8] exofs: drop lock/unlock super
2012/10/9 Al Viro <v...@zeniv.linux.org.uk>:

> On Tue, Oct 09, 2012 at 05:55:12PM +0200, Marco Stornelli wrote:

>> Thanks Al. For me there aren't problems to push it via Al or via
>> other trees. For example the patch for ext4 is in Ted's tree. You
>> tell me what you want and I do it :)

> Um...  Then I'd better drop the ext4 part and hold the final one back,
> right?  Are there other commits I need to skip to avoid duplicates with
> the stuff already in other trees?

No, or at least I know that only Ted (for ext4) has included the patch
in his tree, so no other problem.

Marco
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »