Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-14 Thread Rafael J. Wysocki
On Thursday, November 14, 2013 09:59:44 PM Amit Pundir wrote: > On 14 November 2013 03:14, Rafael J. Wysocki wrote: > > On Wednesday, November 13, 2013 01:35:38 PM Amit Pundir wrote: > >> On 13 November 2013 05:29, Rafael J. Wysocki wrote: > >> > On Wednesday, November 13, 2013 02:22:28 AM Amit

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-14 Thread Amit Pundir
On 14 November 2013 03:14, Rafael J. Wysocki wrote: > On Wednesday, November 13, 2013 01:35:38 PM Amit Pundir wrote: >> On 13 November 2013 05:29, Rafael J. Wysocki wrote: >> > On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: >> >> ep_create_wakeup_source() reports ENOMEM >> > >> >

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-14 Thread Amit Pundir
On 14 November 2013 03:14, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, November 13, 2013 01:35:38 PM Amit Pundir wrote: On 13 November 2013 05:29, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: ep_create_wakeup_source() reports

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-14 Thread Rafael J. Wysocki
On Thursday, November 14, 2013 09:59:44 PM Amit Pundir wrote: On 14 November 2013 03:14, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, November 13, 2013 01:35:38 PM Amit Pundir wrote: On 13 November 2013 05:29, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, November 13, 2013

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-13 Thread Rafael J. Wysocki
On Wednesday, November 13, 2013 01:35:38 PM Amit Pundir wrote: > On 13 November 2013 05:29, Rafael J. Wysocki wrote: > > On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: > >> ep_create_wakeup_source() reports ENOMEM > > > > That needs to be fixed too. I suppose we can make the > >

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-13 Thread Amit Pundir
On 13 November 2013 05:29, Rafael J. Wysocki wrote: > On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: >> ep_create_wakeup_source() reports ENOMEM > > That needs to be fixed too. I suppose we can make the > wakeup_source_register() > stub for CONFIG_PM_SLEEP unset return

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-13 Thread Amit Pundir
On 13 November 2013 05:29, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: ep_create_wakeup_source() reports ENOMEM That needs to be fixed too. I suppose we can make the wakeup_source_register() stub for CONFIG_PM_SLEEP unset return

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-13 Thread Rafael J. Wysocki
On Wednesday, November 13, 2013 01:35:38 PM Amit Pundir wrote: On 13 November 2013 05:29, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: ep_create_wakeup_source() reports ENOMEM That needs to be fixed too. I suppose we can make the

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-12 Thread Rafael J. Wysocki
On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: > ep_create_wakeup_source() reports ENOMEM That needs to be fixed too. I suppose we can make the wakeup_source_register() stub for CONFIG_PM_SLEEP unset return ERR_PTR(-ENOSYS) or something like that and ep_create_wakeup_source()

[RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-12 Thread Amit Pundir
ep_create_wakeup_source() reports ENOMEM if wakeup_source_register() returns NULL. ep_create_wakeup_source() assumes that NULL is only returned if we run into ENOMEM but NULL is also returned when CONFIG_PM_SLEEP is disabled. Signed-off-by: Amit Pundir --- Changed in v2: Using static inline

[RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-12 Thread Amit Pundir
ep_create_wakeup_source() reports ENOMEM if wakeup_source_register() returns NULL. ep_create_wakeup_source() assumes that NULL is only returned if we run into ENOMEM but NULL is also returned when CONFIG_PM_SLEEP is disabled. Signed-off-by: Amit Pundir amit.pun...@linaro.org --- Changed in v2:

Re: [RFC][PATCH v2] epoll: allow EPOLLWAKEUP flag if PM_SLEEP is enabled

2013-11-12 Thread Rafael J. Wysocki
On Wednesday, November 13, 2013 02:22:28 AM Amit Pundir wrote: ep_create_wakeup_source() reports ENOMEM That needs to be fixed too. I suppose we can make the wakeup_source_register() stub for CONFIG_PM_SLEEP unset return ERR_PTR(-ENOSYS) or something like that and ep_create_wakeup_source()