lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 16 Oct 2008 12:35:19 +0530 (IST) From: "Madhusudhan Chikkature" <madhu.cr@...com> To: "Evgeniy Polyakov" <johnpol@....mipt.ru> Cc: "Andrew Morton" <akpm@...ux-foundation.org>, gadiyar@...com, linux-kernel@...r.kernel.org, linux-omap@...r.kernel.org Subject: Re: [PATCH 1/5] HDQ Driver for OMAP2430/3430 > On Tue, Oct 14, 2008 at 07:30:58AM -0700, Andrew Morton > (akpm@...ux-foundation.org) wrote: >> > Why not just skipping the waiting and returning error pretending user >> > really sent a signal? >> >> Better than nothing, but because signal_pending() isn't actually true, >> upper layers wil behave differently. > > If they check... > > For example omap_hdq_break() can be interrupted and omap_hdq_probe() > does not check its return value. > > -- > Evgeniy Polyakov > > Yes. I got the point. But the omap_hdq_break is not significant for HDQ. It just resets the slave and in HDQ mode no response is expected. So the driver will still work even if omap_hdq_break fails. On the TX path it is important to check for the failure which was missing. I will add that check. On the other note, let me correct myself that catching the signal on that very small wait in TX path may not be desired by the driver. The ISR wakes it up as expected. I intend to use "wait_event_timeout" instead and exit in the error path if timeout elapsed. I will repost the whole series after fixing the comments provided by Andrew. Thanks, Madhu -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists