scsi_target witness lock error

[Sean Bruno]

Scott Long wrote:
> Sean Bruno wrote:
>> Hidetoshi Shimokawa wrote:
>>> On 12/12/07, Sean Bruno <sbruno at miralink.com> wrote:
>>>  
>>>>> --- //depot/vendor/freebsd/src/sys/cam/scsi/scsi_target.c     
>>>>> 2007/04/15 08:53:22
>>>>> +++ 
>>>>> //depot/user/simokawa/firewire_lock/sys/cam/scsi/scsi_target.c    
>>>>> 2007/05/21 14:31:55
>>>>> @@ -372,10 +372,14 @@
>>>>>       int     retval;
>>>>>
>>>>>       softc = (struct targ_softc *)kn->kn_hook;
>>>>> +#if 0
>>>>>       cam_periph_lock(softc->periph);
>>>>> +#endif
>>>>>       retval = !TAILQ_EMPTY(&softc->user_ccb_queue) ||
>>>>>                !TAILQ_EMPTY(&softc->abort_queue);
>>>>> +#if 0
>>>>>       cam_periph_unlock(softc->periph);
>>>>> +#endif
>>>>>       return (retval);
>>>>>  }
>>>>>       
>>>> Are these two "if 0"'s supposed to be commented out?  I'm leaving this
>>>> out of my RELENG_6 test for now unless you say otherwise.
>>>>
>>>>
>>>> Sean
>>>>     
>>>
>>> Yes, it is same as comment out.
>>> The patch was for RELENG_7 or current. I'm not sure about RELENG_6.
>>>
>>>   
>> Ok, I merged the patch(RELENG_6) to the best of my ability.  The 
>> system now panics and has the same witness error.
>>
>> panic/witness/trace --> http://consultcsg.com/scsitarget_witness.txt
>> scsi_target.c diff --> http://consultcsg.com/scsi_target.c.diff
>>
>
> CAM locking in RELENG_6 is completely different from locking in
> RELENG_7.  I'm not saying that your problems are imaginary, just
> pointing out that any problems in 6.x are going to almost certainly
> be completely different from problems in 7.x.  I'll look at both.
>
> Scott
Scott:

Thanks.  Let me know when/if you would like some testing/debugging with 
your updates.

Sean