<!doctype html public "-//w3c//dtd html 4.0 transitional//en"> <html> <tt></tt> <tt>Hi Castor,</tt><tt></tt> <tt>While I have had some success with this patch on x86 and</tt> <tt>x86_64 machines, I haven't been so fortunate with other</tt> <tt>architectures.</tt><tt></tt> <tt>On ia64 machines for example, it quite often is unable</tt> <tt>to determine the .data segment address of a module, leaving</tt> <tt>it as 0, so there is no improvement over the current way it works.</tt><tt></tt> <tt>On the s390x, it actually causes a pretty serious regression.</tt> <tt>For example without the patch, here are the data addresses</tt> <tt>of the ext3 module before and after it gets loaded:</tt><tt></tt> <tt>$ /usr/bin/crash -s vmlinux-2.6.18-1.2767.el5</tt> <tt>crash> sym -m ext3 | grep "([dD])"</tt> <tt>208bfea0 (d) ext3_dir_operations</tt> <tt>208bff88 (d) ext3_file_operations</tt> <tt>208c0070 (d) ext3_ordered_aops</tt> <tt>208c00e0 (d) ext3_writeback_aops</tt> <tt>208c0150 (d) ext3_journalled_aops</tt> <tt>208c01c0 (d) ext3_xattr_handler_map</tt> <tt>208c01f8 (d) ext3_file_inode_operations</tt> <tt>208c02a0 (d) ext3_dir_inode_operations</tt> <tt>208c0348 (d) ext3_special_inode_operations</tt> <tt>208c03f0 (d) ext3_fs_type</tt> <tt>208c0430 (d) ext3_sops</tt> <tt>208c04d8 (d) ext3_export_ops</tt> <tt>208c0508 (d) ext3_qctl_operations</tt> <tt>208c0560 (d) ext3_quota_operations</tt> <tt>208c05c0 (d) ext3_symlink_inode_operations</tt> <tt>208c0668 (d) ext3_fast_symlink_inode_operations</tt> <tt>208c0710 (d) ext3_xattr_handlers</tt> <tt>208c0740 (d) tokens</tt> <tt>208c0a40 (d) ext3_xattr_user_handler</tt> <tt>208c0a60 (d) ext3_xattr_trusted_handler</tt> <tt>208c0a80 (d) ext3_xattr_acl_access_handler</tt> <tt>208c0aa0 (d) ext3_xattr_acl_default_handler</tt> <tt>208c0ac0 (d) ext3_xattr_security_handler</tt> <tt>208c0b00 (d) __this_module</tt> <tt>crash> mod -s ext3</tt> <tt> MODULE NAME SIZE OBJECT FILE</tt> <tt> 208c0b00 ext3 200208 /lib/modules/2.6.18-1.2767.el5/kernel/fs/ext3/ext3.ko</tt> <tt>crash> sym -m ext3 | grep "([dD])"</tt> <tt>208bfea0 (d) ext3_dir_operations</tt> <tt>208bff88 (d) ext3_file_operations</tt> <tt>208c0070 (d) ext3_ordered_aops</tt> <tt>208c00e0 (d) ext3_writeback_aops</tt> <tt>208c0150 (d) ext3_journalled_aops</tt> <tt>208c01c0 (d) ext3_xattr_handler_map</tt> <tt>208c01f8 (d) ext3_file_inode_operations</tt> <tt>208c02a0 (d) ext3_dir_inode_operations</tt> <tt>208c0348 (d) ext3_special_inode_operations</tt> <tt>208c03f0 (d) ext3_fs_type</tt> <tt>208c0430 (d) ext3_sops</tt> <tt>208c04d8 (d) ext3_export_ops</tt> <tt>208c0508 (d) ext3_qctl_operations</tt> <tt>208c0560 (d) ext3_quota_operations</tt> <tt>208c05c0 (d) ext3_symlink_inode_operations</tt> <tt>208c0668 (d) ext3_fast_symlink_inode_operations</tt> <tt>208c0710 (d) ext3_xattr_handlers</tt> <tt>208c0740 (d) tokens</tt> <tt>208c0a40 (d) ext3_xattr_user_handler</tt> <tt>208c0a60 (d) ext3_xattr_trusted_handler</tt> <tt>208c0a80 (d) ext3_xattr_acl_access_handler</tt> <tt>208c0aa0 (d) ext3_xattr_acl_default_handler</tt> <tt>208c0ac0 (d) ext3_xattr_security_handler</tt> <tt>208c0b00 (d) __this_module</tt> <tt>crash></tt><tt></tt> <tt>That looks fine...</tt><tt></tt> <tt>However, with the patch applied, I get the following behavior:</tt><tt></tt> <tt>$ crash*14/crash -s vmlinux-2.6.18-1.2767.el5</tt> <tt>crash> sym -m ext3 | grep "([dD])"</tt> <tt>208bfea0 (d) ext3_dir_operations</tt> <tt>208bff88 (d) ext3_file_operations</tt> <tt>208c0070 (d) ext3_ordered_aops</tt> <tt>208c00e0 (d) ext3_writeback_aops</tt> <tt>208c0150 (d) ext3_journalled_aops</tt> <tt>208c01c0 (d) ext3_xattr_handler_map</tt> <tt>208c01f8 (d) ext3_file_inode_operations</tt> <tt>208c02a0 (d) ext3_dir_inode_operations</tt> <tt>208c0348 (d) ext3_special_inode_operations</tt> <tt>208c03f0 (d) ext3_fs_type</tt> <tt>208c0430 (d) ext3_sops</tt> <tt>208c04d8 (d) ext3_export_ops</tt> <tt>208c0508 (d) ext3_qctl_operations</tt> <tt>208c0560 (d) ext3_quota_operations</tt> <tt>208c05c0 (d) ext3_symlink_inode_operations</tt> <tt>208c0668 (d) ext3_fast_symlink_inode_operations</tt> <tt>208c0710 (d) ext3_xattr_handlers</tt> <tt>208c0740 (d) tokens</tt> <tt>208c0a40 (d) ext3_xattr_user_handler</tt> <tt>208c0a60 (d) ext3_xattr_trusted_handler</tt> <tt>208c0a80 (d) ext3_xattr_acl_access_handler</tt> <tt>208c0aa0 (d) ext3_xattr_acl_default_handler</tt> <tt>208c0ac0 (d) ext3_xattr_security_handler</tt> <tt>208c0b00 (d) __this_module</tt> <tt>crash> mod -s ext3</tt> <tt> MODULE NAME SIZE OBJECT FILE</tt> <tt> 208c0b00 ext3 200208 /lib/modules/2.6.18-1.2767.el5/kernel/fs/ext3/ext3.ko</tt> <tt>crash> sym -m ext3 | grep "([dD])"</tt> <tt>0 (D) __this_module</tt> <tt>0 (D) ext3_dir_operations</tt> <tt>0 (D) ext3_file_inode_operations</tt> <tt>0 (d) tokens</tt> <tt>a8 (D) ext3_dir_inode_operations</tt> <tt>e8 (D) ext3_file_operations</tt> <tt>150 (D) ext3_special_inode_operations</tt> <tt>1d0 (d) ext3_ordered_aops</tt> <tt>1f8 (d) ext3_fs_type</tt> <tt>238 (d) ext3_sops</tt> <tt>240 (d) ext3_writeback_aops</tt> <tt>2b0 (d) ext3_journalled_aops</tt> <tt>2e0 (d) ext3_export_ops</tt> <tt>300 (D) ext3_xattr_user_handler</tt> <tt>310 (d) ext3_qctl_operations</tt> <tt>320 (d) ext3_xattr_handler_map</tt> <tt>320 (D) ext3_xattr_trusted_handler</tt> <tt>340 (D) ext3_xattr_acl_access_handler</tt> <tt>360 (D) ext3_xattr_acl_default_handler</tt> <tt>368 (d) ext3_quota_operations</tt> <tt>380 (D) ext3_xattr_security_handler</tt> <tt>3c8 (D) ext3_symlink_inode_operations</tt> <tt>470 (D) ext3_fast_symlink_inode_operations</tt> <tt>518 (D) ext3_xattr_handlers</tt> <tt>crash></tt><tt></tt> <tt>I haven't been able to test it on a ppc64 or s390. (That's</tt> <tt>why I asked for help from others on the mailing list to test</tt> <tt>it out, but I didn't get any takers...)</tt><tt></tt> <tt>In any case, as much as I like what the patch wants to do,</tt> <tt>I can't accept it given such a serious regression. Perhaps</tt> <tt>it could be made an experimental run-time option that could</tt> <tt>be turned on prior to doing any "mod" commands, leaving the</tt> <tt>current behaviour in place by default?</tt><tt></tt> <tt>Thanks,</tt> <tt> Dave</tt> <tt></tt> </html>