Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Meelis Roos <mroos@linux.ee>
Subject: w83627ehf crash in 5.6.0-rc2-00055-gca7e1fd1026c
To:     linux-hwmon@vger.kernel.org
Cc:     LKML <linux-kernel@vger.kernel.org>,
        "Dr. David Alan Gilbert" <linux@treblig.org>,
        Chen Zhou <chenzhou10@huawei.com>
Message-ID: <434212bb-4eb9-7366-3255-79826d0e65bc@linux.ee>
Date:   Thu, 20 Feb 2020 13:18:39 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.2.2
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

While reading w83627ehf sensors output on D425KT mainboard, I consistently get NULL dereference as below.

5.5.0 worked OK but gave a warning on driver load:
[  104.514954] w83627ehf: Found W83627DHG-P chip at 0x290
[  104.515634] w83627ehf w83627ehf.656: hwmon_device_register() is deprecated. Please convert the driver to use hwmon_device_register_with_info().

This is dmesg from current git (loading the driver and reading sensors with lm-sensors - no driver loading warning any more):

[  764.718192] w83627ehf: Found W83627DHG-P chip at 0x290
[  774.574874] BUG: kernel NULL pointer dereference, address: 0000000000000000
[  774.574889] #PF: supervisor read access in kernel mode
[  774.574895] #PF: error_code(0x0000) - not-present page
[  774.574901] PGD 0 P4D 0
[  774.574909] Oops: 0000 [#1] SMP NOPTI
[  774.574917] CPU: 0 PID: 604 Comm: sensors Not tainted 5.6.0-rc2-00055-gca7e1fd1026c #29
[  774.574923] Hardware name:  /D425KT, BIOS MWPNT10N.86A.0132.2013.0726.1534 07/26/2013
[  774.574939] RIP: 0010:w83627ehf_read_string+0x27/0x70 [w83627ehf]
[  774.574947] Code: 00 00 00 55 53 48 8d 64 24 f0 83 fa 15 48 8b 5f 78 75 29 83 fe 01 75 24 48 63 c9 48 8b 6b 58 48 83 f9 03 77 24 0f b6 44 0b 50 <48> 8b 44 c5 00 49 89 00 48 8d 64 24 10 5b 31 c0 5d c3 48 8d 64 24
[  774.574958] RSP: 0018:ffffb95980657df8 EFLAGS: 00010293
[  774.574965] RAX: 0000000000000000 RBX: ffff96caaa7f5218 RCX: 0000000000000000
[  774.574972] RDX: 0000000000000015 RSI: 0000000000000001 RDI: ffff96caa736ec08
[  774.574978] RBP: 0000000000000000 R08: ffffb95980657e20 R09: 0000000000000001
[  774.574985] R10: ffff96caaa635cc0 R11: 0000000000000000 R12: ffff96caa9f7cf00
[  774.574991] R13: ffff96caa9ec3d00 R14: ffff96caa9ec3d28 R15: ffff96caa9ec3d40
[  774.574999] FS:  00007fbc7c4e2740(0000) GS:ffff96caabc00000(0000) knlGS:0000000000000000
[  774.575008] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[  774.575015] CR2: 0000000000000000 CR3: 0000000129d58000 CR4: 00000000000006f0
[  774.575021] Call Trace:
[  774.575036]  ? cp_new_stat+0x12d/0x160
[  774.575048]  hwmon_attr_show_string+0x37/0x70 [hwmon]
[  774.575060]  dev_attr_show+0x14/0x50
[  774.575071]  sysfs_kf_seq_show+0xb5/0x1b0
[  774.575081]  seq_read+0xcf/0x460
[  774.575091]  vfs_read+0x9b/0x150
[  774.575100]  ksys_read+0x5f/0xe0
[  774.575111]  do_syscall_64+0x48/0x190
[  774.575121]  entry_SYSCALL_64_after_hwframe+0x44/0xa9
[  774.575130] RIP: 0033:0x7fbc7c715871
[  774.575138] Code: fe ff ff 50 48 8d 3d 76 e5 09 00 e8 e9 ef 01 00 66 0f 1f 84 00 00 00 00 00 48 8d 05 69 3b 0d 00 8b 00 85 c0 75 13 31 c0 0f 05 <48> 3d 00 f0 ff ff 77 57 c3 66 0f 1f 44 00 00 48 83 ec 28 48 89 54
[  774.575151] RSP: 002b:00007ffe5092d848 EFLAGS: 00000246 ORIG_RAX: 0000000000000000
[  774.575161] RAX: ffffffffffffffda RBX: 00005630bdfb1330 RCX: 00007fbc7c715871
[  774.575168] RDX: 0000000000001000 RSI: 00007ffe5092d8e0 RDI: 0000000000000003
[  774.575175] RBP: 00007fbc7c7e5560 R08: 0000000000000003 R09: 00007fbc7c7e43b0
[  774.575182] R10: 0000000000000000 R11: 0000000000000246 R12: 0000000000001000
[  774.575190] R13: 00007ffe5092d8e0 R14: 0000000000000d68 R15: 00007fbc7c7e4960
[  774.575199] Modules linked in: w83627ehf hwmon_vid snd_hda_codec_realtek snd_hda_codec_generic snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep snd_pcsp snd_hda_core ir_rc6_decoder rc_rc6_mce uas r8169 mceusb snd_pcm
iTCO_wdt rc_core snd_timer iTCO_vendor_support realtek snd libphy soundcore i2c_i801 lpc_ich parport_pc mfd_core parport coretemp hwmon autofs4
[  774.575247] CR2: 0000000000000000
[  774.575254] ---[ end trace 607462057ab8a988 ]---
[  774.575264] RIP: 0010:w83627ehf_read_string+0x27/0x70 [w83627ehf]
[  774.575273] Code: 00 00 00 55 53 48 8d 64 24 f0 83 fa 15 48 8b 5f 78 75 29 83 fe 01 75 24 48 63 c9 48 8b 6b 58 48 83 f9 03 77 24 0f b6 44 0b 50 <48> 8b 44 c5 00 49 89 00 48 8d 64 24 10 5b 31 c0 5d c3 48 8d 64 24
[  774.575287] RSP: 0018:ffffb95980657df8 EFLAGS: 00010293
[  774.575294] RAX: 0000000000000000 RBX: ffff96caaa7f5218 RCX: 0000000000000000
[  774.575301] RDX: 0000000000000015 RSI: 0000000000000001 RDI: ffff96caa736ec08
[  774.575308] RBP: 0000000000000000 R08: ffffb95980657e20 R09: 0000000000000001
[  774.575316] R10: ffff96caaa635cc0 R11: 0000000000000000 R12: ffff96caa9f7cf00
[  774.575323] R13: ffff96caa9ec3d00 R14: ffff96caa9ec3d28 R15: ffff96caa9ec3d40
[  774.575331] FS:  00007fbc7c4e2740(0000) GS:ffff96caabc00000(0000) knlGS:0000000000000000
[  774.575340] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[  774.575347] CR2: 0000000000000000 CR3: 0000000129d58000 CR4: 00000000000006f0


-- 
Meelis Roos <mroos@linux.ee>