RE: ACPI locking bugs?

2013-03-21 Thread Moore, Robert
d- > a...@freebsd.org] On Behalf Of Moore, Robert > Sent: Wednesday, February 27, 2013 12:34 PM > To: Jung-uk Kim; Hans Petter Selasky > Cc: freebsd-acpi@freebsd.org > Subject: RE: ACPI locking bugs? > > A couple comments below. > > > -Original Message- >

Re: ACPI locking bugs?

2013-03-01 Thread Jung-uk Kim
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2013-03-01 02:52:12 -0500, Hans Petter Selasky wrote: > The problem is not locking or locking order, but that the locking > wait for signals. You know when you have a thread, you kan kill it > using a signal. I think at shutdown all threads and proc

Re: ACPI locking bugs?

2013-03-01 Thread Jung-uk Kim
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2013-02-27 09:54:53 -0500, Andriy Gapon wrote: > on 27/02/2013 15:53 Hans Petter Selasky said the following: >> Hi, >> >> What is the reason for using cv_wait_sig() and cv_timedwait_sig() >> instead of cv_wait() and cv_timedwait() inside of >> Acpi

Re: ACPI locking bugs?

2013-02-28 Thread Hans Petter Selasky
On Wednesday 27 February 2013 14:28:12 Jung-uk Kim wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 2013-02-27 08:53:43 -0500, Hans Petter Selasky wrote: > > Hi, > > > > What is the reason for using cv_wait_sig() and cv_timedwait_sig() > > instead of cv_wait() and cv_timedwait() in

RE: ACPI locking bugs?

2013-02-27 Thread Moore, Robert
A couple comments below. > -Original Message- > From: owner-freebsd-a...@freebsd.org [mailto:owner-freebsd- > a...@freebsd.org] On Behalf Of Jung-uk Kim > Sent: Wednesday, February 27, 2013 11:28 AM > To: Hans Petter Selasky > Cc: freebsd-acpi@freebsd.org > Subject:

Re: ACPI locking bugs?

2013-02-27 Thread Jung-uk Kim
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2013-02-27 08:53:43 -0500, Hans Petter Selasky wrote: > Hi, > > What is the reason for using cv_wait_sig() and cv_timedwait_sig() > instead of cv_wait() and cv_timedwait() inside of > AcpiOsWaitSemaphore(). What signals are supposed to be catched >

Re: ACPI locking bugs?

2013-02-27 Thread Hans Petter Selasky
On Wednesday 27 February 2013 15:54:53 Andriy Gapon wrote: > kib drew my attention to this issue some time ago and I also pinged jkim > about it. I completely agree with you that the signal handling should be > removed. Are you willing to make the change or would you prefer me doing > it? > > Than

Re: ACPI locking bugs?

2013-02-27 Thread Andriy Gapon
on 27/02/2013 15:53 Hans Petter Selasky said the following: > Hi, > > What is the reason for using cv_wait_sig() and cv_timedwait_sig() instead of > cv_wait() and cv_timedwait() inside of AcpiOsWaitSemaphore(). What signals > are > supposed to be catched here? > >switch (Timeout) { >

ACPI locking bugs?

2013-02-27 Thread Hans Petter Selasky
Hi, What is the reason for using cv_wait_sig() and cv_timedwait_sig() instead of cv_wait() and cv_timedwait() inside of AcpiOsWaitSemaphore(). What signals are supposed to be catched here? switch (Timeout) { case ACPI_DO_NOT_WAIT: if (!ACPISEM_AVAIL(as, Units))