Re: [PATCH 2.6.21-rc2] ehca: fix mismatched sync between completion handler and destroy cq

2007-03-02 Thread Hoang-Nam Nguyen
>  > +#include 
> This can just be , because you're only using
> wait_queue_head_t and not struct completion, right?
> I fixed this up before merging.
Yes, right. Thanks for your help!
Regards
Nam 

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2.6.21-rc2] ehca: fix mismatched sync between completion handler and destroy cq

2007-03-02 Thread Hoang-Nam Nguyen
   +#include linux/completion.h
 This can just be linux/wait.h, because you're only using
 wait_queue_head_t and not struct completion, right?
 I fixed this up before merging.
Yes, right. Thanks for your help!
Regards
Nam 

-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2.6.21-rc2] ehca: fix mismatched sync between completion handler and destroy cq

2007-03-01 Thread Roland Dreier
Looks OK to me, queued for 2.6.21, except:

 > +#include 

This can just be , because you're only using
wait_queue_head_t and not struct completion, right?

I fixed this up before merging.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2.6.21-rc2] ehca: fix mismatched sync between completion handler and destroy cq

2007-03-01 Thread Roland Dreier
Looks OK to me, queued for 2.6.21, except:

  +#include linux/completion.h

This can just be linux/wait.h, because you're only using
wait_queue_head_t and not struct completion, right?

I fixed this up before merging.
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/