Skip to contentSkip to conversation

Ext2: remove the overhead check about sb in the function ext2_new_blocks

PATCH In review Wang Shilong 3 participants 5 messages Feb 6, 2013 – Feb 8, 2013

lore
Share & tools
lore
#0 lore raw

PATCH 1/3 Ext2: remove the overhead check about sb in the function ext2_new_blocks

From: Wang Shilong <[email protected]>

It can be guranteed that inode->i_sb should not be null in vfs. So here the check about it is overhead.

---

sorry about wrong format patch, i will resend it later.

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

#1 lore raw
Quoted 9 lines · Wang Shilong

From: Wang Shilong <[email protected]>

It can be guranteed that inode->i_sb should not be null in vfs. So here the check about it is overhead.

---

sorry about wrong format patch, i will resend it later.

Hi Shilong,

If you want to re-send a patch series, add [RESEND] in subject, please. Because it could make us know which is the latest one. :-)

Regards,
- Zheng
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

#2 lore raw
Quoted 3 lines · Zheng Liu

If you want to re-send a patch series, add [RESEND] in subject, please. Because it could make us know which is the latest one. :-)

Ideally, use RESEND if you haven't changed the patch series since the last time you've sent it. If you have changed it, please use [PATCH v2], [PATCH v3], etc.

Thanks,

- Ted

#3 lore raw

2013/2/7 Theodore Ts'o <[email protected]>:

Quoted 8 lines · Zheng Liu
Quoted 3 lines

If you want to re-send a patch series, add [RESEND] in subject, please. Because it could make us know which is the latest one. :-)

Ideally, use RESEND if you haven't changed the patch series since the last time you've sent it. If you have changed it, please use [PATCH v2], [PATCH v3], etc.

I get it..
Thanks, thanks every one for teaching me so much...

Now i am trying to do something for ext4 development in my leisure time.. Since i don't know anything about filesystem before.. i have to study about it from minix to ext2..ext3..ext4..btrfs..

However, it still seems a little hard for me to figure everything out just by reading code... So i wonder where i can get
some papers or articles about the new concept of filesystems.. for example extent management or something else for ext4..etc Maybe it helps a lot for me...

Thanks,
Wang

Quoted 4 lines

Thanks,

- Ted

#4 lore raw
Quoted 17 lines · shilong wang

2013/2/7 Theodore Ts'o <[email protected]>:

Quoted 8 lines · Zheng Liu
Quoted 3 lines

If you want to re-send a patch series, add [RESEND] in subject, please. Because it could make us know which is the latest one. :-)

Ideally, use RESEND if you haven't changed the patch series since the last time you've sent it. If you have changed it, please use [PATCH v2], [PATCH v3], etc.

I get it..
Thanks, thanks every one for teaching me so much...

Now i am trying to do something for ext4 development in my leisure time.. Since i don't know anything about filesystem before.. i have to study about it from minix to ext2..ext3..ext4..btrfs..

Quoted 5 lines

However, it still seems a little hard for me to figure everything out just by reading code... So i wonder where i can get
some papers or articles about the new concept of filesystems.. for example extent management or something else for ext4..etc Maybe it helps a lot for me...

This link (https://ext4.wiki.kernel.org/index.php/Main_Page) might be useful for you.

Regards,
- Zheng