On Mon 22-10-12 18:18:10, Fabio Coatti wrote: > 2012/10/22 Jan Kara <j...@suse.cz>: > > On Mon 22-10-12 18:01:56, Fabio Coatti wrote: > >> 2012/10/22 Jan Kara <j...@suse.cz>: > >> >> Uhu, my fault, I was sure to have answered to your request, sorry. > >> >> So yes, we have an ext3 R/O partition mounted on that machine. The > >> >> 3.6.3, with patch applied, dmesg is here: > >> > Interesting. You seem to be using data=journal on R/O partition, don't > >> > you? Can you also attach /proc/mounts? > >> > > >> > Also please try running a kernel with the attached patch and report the > >> > warning. Thanks! > >> > >> Do you want to try the kernel without touching the mount > >> configuration, right? (I mean, without (say) mounting ext2 the ext3 ro > >> partition, right?) > > Yes, leave things as they are now... > > > >> right now I can't access the machine, but I recall that the partition > >> is mounted rw, something is wrote onto it and then changed to r/o and > >> among the options there is data=ordered. > > /proc/mounts should show us whether data=ordered is in use. Actually the > > warning is in function which is called only in data=journal mode so that's > > why I think you are using it... > > Ok, got the options: > > ro,noatime,errors=continue,barrier=1,data=ordered 0 0 Can you please post *full* /proc/mounts content? Thanks. It is hard to believe ext3_journalled_writepage() gets called on data=ordered filesystem.
Honza > and as expected, the usual warning triggered, but not the message from > your patch: > > [ 173.408622] ------------[ cut here ]------------ > [ 173.408725] WARNING: at fs/ext3/inode.c:1754 > ext3_journalled_writepage+0x55/0x1a7() > [ 173.408850] Hardware name: ProLiant BL465c G7 > [ 173.408940] Pid: 2142, comm: flush-8:0 Not tainted 3.6.3-p #2 > [ 173.409034] Call Trace: > [ 173.409123] [<ffffffff81057884>] ? warn_slowpath_common+0x73/0x87 > [ 173.409219] [<ffffffff8115cd04>] ? ext3_journalled_writepage+0x55/0x1a7 > [ 173.409320] [<ffffffff810a2833>] ? __writepage+0xa/0x21 > [ 173.409414] [<ffffffff810a31db>] ? write_cache_pages+0x206/0x2f8 > [ 173.409509] [<ffffffff810a2829>] ? set_page_dirty+0x5e/0x5e > [ 173.409604] [<ffffffff81297cfb>] ? queue_unplugged+0x28/0x34 > [ 173.409698] [<ffffffff810a330b>] ? generic_writepages+0x3e/0x55 > [ 173.409793] [<ffffffff810f4eb0>] ? __writeback_single_inode+0x39/0xd1 > [ 173.409889] [<ffffffff810f5c69>] ? writeback_sb_inodes+0x206/0x392 > [ 173.409983] [<ffffffff810f5e5c>] ? __writeback_inodes_wb+0x67/0xa2 > [ 173.410079] [<ffffffff810f5ffa>] ? wb_writeback+0xfd/0x18b > [ 173.410172] [<ffffffff810a36d1>] ? bdi_dirty_limit+0x27/0x81 > [ 173.410348] [<ffffffff810f61c5>] ? wb_do_writeback+0x13d/0x1a2 > [ 173.410461] [<ffffffff81061e9b>] ? add_timer_on+0x61/0x61 > [ 173.410584] [<ffffffff810f62a9>] ? bdi_writeback_thread+0x7f/0x13e > [ 173.410700] [<ffffffff8106a54d>] ? queue_work_on+0x16/0x1e > [ 173.410809] [<ffffffff810f622a>] ? wb_do_writeback+0x1a2/0x1a2 > [ 173.410902] [<ffffffff810f622a>] ? wb_do_writeback+0x1a2/0x1a2 > [ 173.410996] [<ffffffff8106e134>] ? kthread+0x81/0x89 > [ 173.411089] [<ffffffff81607e74>] ? kernel_thread_helper+0x4/0x10 > [ 173.411184] [<ffffffff8106e0b3>] ? kthread_worker_fn+0xe0/0xe0 > [ 173.411277] [<ffffffff81607e70>] ? gs_change+0xb/0xb > [ 173.411367] ---[ end trace 8243403b245b6cfc ]--- > > -- > Fabio -- Jan Kara <j...@suse.cz> SUSE Labs, CR -- 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/