Re: CVS commit: src/sys/external/bsd/libnv/dist

[Timo Buhrmester]

> Can you try this patch?

With the patch I get a different panic:
| # npfctl reload
| panic: kernel diagnostic assertion "(nvp) != ((void *)0)" failed: file 
"/home/fstd/qemutest/src/sys/external/bsd/libnv/dist/nvpair.c", line 2140 
| cpu0: Begin traceback...
| 
vpanic(c15035d4,c94c0b80,c94c0b80,c94c0b90,c0d6b321,c15035d4,c15035c6,c15035b1,c1503574,85c)
 at netbsd:vpanic+0x1b4
| 
kern_assert(c15035d4,c15035c6,c15035b1,c1503574,85c,c94c0bac,c0d6313c,0,c94c0bc8,c0bf4da5)
 at netbsd:kern_assert+0x27
| 
nvpair_free_structure(0,c94c0bc8,c0bf4da5,c1e9dd80,c25ff000,c94c0bd0,c0d66ef4,c2439a5c,0,1)
 at netbsd:nvpair_free_structure+0x40
| 
nvlist_set_array_next(c2439a5c,0,1,0,c1ec1dbc,c2439a5c,0,c94c0bec,c0d6700d,c294b00c)
 at netbsd:nvlist_set_array_next+0xb8
| 
nvpair_remove_nvlist_array(c294b00c,c94c0be8,c0d6b459,c25ff00c,c94c0c08,c94c0c08,c0d664bc,c2439ac0,c294b00c,c2439aac)
 at netbsd:nvpair_remove_nvlist_array+0x6a
| 
nvpair_remove(c2439ac0,c294b00c,c2439aac,c94c0c28,c08a443e,c94c0c28,c0d62d61,c2439aac,c294b00c,c0d62900)
 at netbsd:nvpair_remove+0xf7
| 
nvlist_remove_nvpair(c2439aac,c294b00c,c0d62900,439aac,c1fdefa8,c294b00c,c94c0c7c,c08a5ad3,c2439aac,c20640c8)
 at netbsd:nvlist_remove_nvpair+0xeb
| nvlist_destroy(c2439aac,c20640c8,c203dda8,c25706c8,c1ec1d68,0,0,1,0,0) at 
netbsd:nvlist_destroy+0x9e
| 
npfctl_load_nvlist(c1fd8700,c2439aac,c25706f4,f,c2439aac,0,c25706f4,c94c0cc8,c08b4709,c1fd8700)
 at netbsd:npfctl_load_nvlist+0x25f
| 
npfctl_load(c1fd8700,c00c4e66,c94c0e74,0,0,0,c94c0cdc,c1fd8700,c94c0cf4,c0beb8cf)
 at netbsd:npfctl_load+0x55
| 
npf_dev_ioctl(c600,0,c00c4e66,c94c0e74,1,c225fd20,c14370e0,100,c225fd20,c94c0d28)
 at netbsd:npf_dev_ioctl+0x161
| cdev_ioctl(c600,0,c00c4e66,c94c0e74,1,c225fd20,c94c0d4c,c258cba8,c600,0) at 
netbsd:cdev_ioctl+0x7c
| 
spec_ioctl(c94c0d4c,c94c0d44,c94c0d43,0,17d,c2134008,0,c14df640,c258cba8,c00c4e66)
 at netbsd:spec_ioctl+0xde
| 
VOP_IOCTL(c258cba8,c00c4e66,c94c0e74,1,c1ed7a80,0,c258cba8,c0bddbf5,c225fd20,c17bddc0)
 at netbsd:VOP_IOCTL+0x77
| vn_ioctl(c21e3780,c00c4e66,c94c0e74,c,0,0,0,0,c1ea4180,c00c4e66) at 
netbsd:vn_ioctl+0x230
| 
sys_ioctl(c225fd20,c94c0f74,c94c0f6c,c94c0f30,c94c0f3c,c0169e2b,c1807b98,c225fd20,c94c0f74,c94c0f6c)
 at netbsd:sys_ioctl+0x43e
| 
sy_call(c1807b98,c225fd20,c94c0f74,c94c0f6c,c0bcc36c,25fd20,c225fd20,c94c0f9c,c016a0a2,c1807b98)
 at c0169d5b
| 
sy_invoke(c1807b98,c225fd20,c94c0f74,c94c0f6c,36,0,c225fd20,c223bbd4,36,c1807b98)
 at netbsd:sy_invoke+0xbb
| syscall() at netbsd:syscall+0xd7
| --- syscall (number 54) ---
| acde9af7:
| cpu0: End traceback...

Re: CVS commit: src/sys/external/bsd/libnv/dist

[Timo Buhrmester]

This commit (or the following one) gives me the following
panic on 'npfctl reload':

| # npfctl reload
| [ 169.6257771] panic: kmem_free(0xc1fd76a0, 46) != allocated size 4294967295
| [ 169.6361202] cpu0: Begin traceback...
| [ 169.6361202] 
vpanic(c14cc1ac,c94e0b30,c94e0b30,c94e0b44,c0bf557e,c14cc1ac,c1fd76a0,2e,,c1fd7698)
 at netbsd:vpanic+0x1b4
| [ 169.6457867] 
vpanic(c14cc1ac,c1fd76a0,2e,,c1fd7698,,c94e0b70,c0bf4d80,c1fd7698,2e)
 at netbsd:vpanic
| [ 169.6558448] 
kmem_size_check(c1fd7698,2e,,c0bfa6db,6,c1e9a480,30,38,6,c94e0b84) at 
netbsd:kmem_size_check+0x40
| [ 169.6658635] 
kmem_intr_free(c1fd7698,2e,c1fd76a0,c94e0b90,c0d6b461,c1fd76a4,c94e0bac,c0d63144,c1fd76a4,c94e0bc8)
 at netbsd:kmem_intr_free+0x157
| [ 169.6758977] 
kern_free(c1fd76a4,c94e0bac,c0d63144,c1fd76a4,c94e0bc8,c0bf4dad,c1e9dd80,c2891000,c94e0bd0,c0d66efc)
 at netbsd:kern_free+0x51
| [ 169.6859073] 
nv_free(c1fd76a4,c94e0bc8,c0bf4dad,c1e9dd80,c2891000,c94e0bd0,c0d66efc,c23a235c,0,6)
 at netbsd:nv_free+0x17
| [ 169.6959382] 
nvlist_set_array_next(c23a235c,0,6,0,c23a2384,c23a235c,c1fd76a4,c94e0bec,c0d67015,c289600c)
 at netbsd:nvlist_set_array_next+0xb8
| [ 169.7059703] 
nvpair_remove_nvlist_array(c289600c,c94e0be8,c0d6b461,c289100c,c94e0c08,c94e0c08,c0d664c4,c23a2410,c289600c,c23a23fc)
 at netbsd:nvpair_remove_nvlist_array+0x6a
| [ 169.7160051] 
nvpair_remove(c23a2410,c289600c,c23a23fc,c94e0c28,1,c94e0c28,c0d62d69,c23a23fc,c289600c,c0d62900)
 at netbsd:nvpair_remove+0xf7
| [ 169.7260339] 
nvlist_remove_nvpair(c23a23fc,c289600c,c0d62900,3a23fc,c1fde2e8,c289600c,c94e0c7c,c08a5ad3,c23a23fc,c288cf88)
 at netbsd:nvlist_remove_nvpair+0xeb
| [ 169.7361114] 
nvlist_destroy(c23a23fc,c288cf88,c203df70,c247ca88,c25c6978,0,0,1,0,0) at 
netbsd:nvlist_destroy+0x9e
| [ 169.7461010] 
npfctl_load_nvlist(c1fd87c0,c23a23fc,c247cab4,f,c23a23fc,0,c247cab4,c94e0cc8,c08b4709,c1fd87c0)
 at netbsd:npfctl_load_nvlist+0x25f
| [ 169.7561366] 
npfctl_load(c1fd87c0,c00c4e66,c94e0e74,0,0,0,c0bcc374,c1fd87c0,c94e0cf4,c0beb8d7)
 at netbsd:npfctl_load+0x55
| [ 169.7762031] 
npf_dev_ioctl(c600,0,c00c4e66,c94e0e74,1,c21e6aa0,c1436860,100,c21e6aa0,c94e0d28)
 at netbsd:npf_dev_ioctl+0x161
| [ 169.7862327] 
cdev_ioctl(c600,0,c00c4e66,c94e0e74,1,c21e6aa0,c94e0d4c,c24461b0,c600,0) at 
netbsd:cdev_ioctl+0x7c
| [ 169.7962746] 
spec_ioctl(c94e0d4c,c94e0d44,c94e0d43,0,1b54698,c21be008,0,c14de57c,c24461b0,c00c4e66)
 at netbsd:spec_ioctl+0xde
| [ 169.8063312] 
VOP_IOCTL(c24461b0,c00c4e66,c94e0e74,1,c1ed7a80,c94e0e40,c24461b0,c94e0db4,c0b57109,c94e0e40)
 at netbsd:VOP_IOCTL+0x77
| [ 169.8163557] 
vn_ioctl(c21ccf00,c00c4e66,c94e0e74,c,0,c219ab98,cbc000,1000,c219ab98,c00c4e66) 
at netbsd:vn_ioctl+0x230
| [ 169.8263985] 
sys_ioctl(c21e6aa0,c94e0f74,c94e0f6c,1,c94e0f3c,c0169e2b,c1805b98,c21e6aa0,c94e0f74,c94e0f6c)
 at netbsd:sys_ioctl+0x43e
| [ 169.8364516] 
sy_call(c1805b98,c21e6aa0,c94e0f74,c94e0f6c,c94e0f3c,0,c21e6aa0,c94e0f9c,c016a0a2,c1805b98)
 at c0169d5b
| [ 169.8565044] 
sy_invoke(c1805b98,c21e6aa0,c94e0f74,c94e0f6c,36,0,c21e6aa0,c20745e8,36,c1805b98)
 at netbsd:sy_invoke+0xbb
| [ 169.8665328] syscall() at netbsd:syscall+0xd7
| [ 169.8765640] --- syscall (number 54) --- 
| [ 169.8765640] ba474af7:
| [ 169.8765640] cpu0: End traceback...

with a rather minimal npf.conf like this:

| procedure "log" { log: npflog0 }
| 
| group default {
|   block final all apply "log"
| }
| 
| group "wm0" on wm0 {
|   block final all apply "log"
| }
| 
| group "lo0" on lo0 {
| pass final all
| }