Re: si_drv1 used already?

2001-03-25 Thread Poul-Henning Kamp

In message <[EMAIL PROTECTED]>, Julian Elischer writes:
>Poul-Henning Kamp wrote:
>> 
>> In message <[EMAIL PROTECTED]>, Julian Elischer writes:
>> >
>> >The vmware vmmon driver used to use si_drv1 but suddenly it appears to
>> >have already been used (at least it is no-longer non-zero).
>> >It would fail to open because it checks it before using it during open.
>> >Changing the driver to use si_drv2 fixes the problem.
>> >
>> >Since drv1 is defined as 'for the use of the driver'
>> >I am not sure that it is wrong for the driver to use it.
>> >In fact many other drivers do use it without testing it.
>> >
>> >I see it's used in freedev() and make_dev_alias().
>> >Is it safe for the driver to use it even if it is non-zero?
>> 
>> I think you need to give me a bit more details before I an answer
>> meaningfully...
>> 
>> Are you sure that the value is not from previous use by vmmon ?
>
>no I am not sure.. I will test further.
>the sequence:
>1/ failed to open
>2/ unload module

Make sure vmmon calles destroy_dev() on all dev_t's created with
make_dev() or make_dev_alias() when unloaded.

This process is not automatic, and I am not sure we can safely
make it automatic.  We may be able to make it easier though.

--
Poul-Henning Kamp   | UNIX since Zilog Zeus 3.20
[EMAIL PROTECTED] | TCP/IP since RFC 956
FreeBSD committer   | BSD since 4.3-tahoe
Never attribute to malice what can adequately be explained by incompetence.

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: si_drv1 used already?

2001-03-25 Thread Julian Elischer

Poul-Henning Kamp wrote:
> 
> In message <[EMAIL PROTECTED]>, Julian Elischer writes:
> >
> >The vmware vmmon driver used to use si_drv1 but suddenly it appears to
> >have already been used (at least it is no-longer non-zero).
> >It would fail to open because it checks it before using it during open.
> >Changing the driver to use si_drv2 fixes the problem.
> >
> >Since drv1 is defined as 'for the use of the driver'
> >I am not sure that it is wrong for the driver to use it.
> >In fact many other drivers do use it without testing it.
> >
> >I see it's used in freedev() and make_dev_alias().
> >Is it safe for the driver to use it even if it is non-zero?
> 
> I think you need to give me a bit more details before I an answer
> meaningfully...
> 
> Are you sure that the value is not from previous use by vmmon ?

no I am not sure.. I will test further.
the sequence:
1/ failed to open
2/ unload module
3/ load newly compiled module with printfs
4/ discover si_drv1 in use
5/ unload, load new version using si_drv2
6/ succeed.

it is possible that the recompile fixed another problem
that had caused another abort, but had left si_drv1 non-zero.

I'll need to recompile and reboot to confirm.

> 
> --
> Poul-Henning Kamp   | UNIX since Zilog Zeus 3.20
> [EMAIL PROTECTED] | TCP/IP since RFC 956
> FreeBSD committer   | BSD since 4.3-tahoe
> Never attribute to malice what can adequately be explained by incompetence.

-- 
  __--_|\  Julian Elischer
 /   \ [EMAIL PROTECTED]
(   OZ) World tour 2000-2001
---> X_.---._/  
v

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: si_drv1 used already?

2001-03-25 Thread Poul-Henning Kamp

In message <[EMAIL PROTECTED]>, Julian Elischer writes:
>
>The vmware vmmon driver used to use si_drv1 but suddenly it appears to
>have already been used (at least it is no-longer non-zero).
>It would fail to open because it checks it before using it during open.
>Changing the driver to use si_drv2 fixes the problem. 
>
>Since drv1 is defined as 'for the use of the driver'
>I am not sure that it is wrong for the driver to use it.
>In fact many other drivers do use it without testing it.
>
>I see it's used in freedev() and make_dev_alias().
>Is it safe for the driver to use it even if it is non-zero?

I think you need to give me a bit more details before I an answer
meaningfully...

Are you sure that the value is not from previous use by vmmon ?



--
Poul-Henning Kamp   | UNIX since Zilog Zeus 3.20
[EMAIL PROTECTED] | TCP/IP since RFC 956
FreeBSD committer   | BSD since 4.3-tahoe
Never attribute to malice what can adequately be explained by incompetence.

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message