linux-kernel - Re: [PATCH] genirq: provide dummy set_irq

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <552D1DEF.6080303@free-electrons.com>
Date:	Tue, 14 Apr 2015 16:02:23 +0200
From:	Gregory CLEMENT <gregory.clement@...e-electrons.com>
To:	Roger Quadros <rogerq@...com>
CC:	tglx@...utronix.de, cw00.choi@...sung.com, balbi@...com,
	tony@...mide.com, linux-usb@...r.kernel.org,
	linux-omap@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] genirq: provide dummy set_irq_wake()

Hi Roger,

On 14/04/2015 12:13, Roger Quadros wrote:
> Hi Thomas,
> 
> On 30/03/15 16:15, Roger Quadros wrote:
>> Without this system suspend is broken on systems that have
>> drivers calling enable/disable_irq_wake() for interrupts based off
>> the dummy irq hook.
>> (e.g. drivers/gpio/gpio-pcf857x.c)
>>
>> http://article.gmane.org/gmane.linux.kernel/1879035
>>
>> Signed-off-by: Roger Quadros <rogerq@...com>
> 
> Any comments on this patch?

I read the url you pointed and I wonder why, at then end, did
you choose to add a dummy set_irq_wake() instead of using
IRQCHIP_SKIP_SET_WAKE ?

Thanks,

Gregory


> 
> cheers,
> -roger
> 
>> ---
>>  kernel/irq/dummychip.c | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/kernel/irq/dummychip.c b/kernel/irq/dummychip.c
>> index 988dc58..2405d7a 100644
>> --- a/kernel/irq/dummychip.c
>> +++ b/kernel/irq/dummychip.c
>> @@ -32,6 +32,11 @@ static unsigned int noop_ret(struct irq_data *data)
>>  	return 0;
>>  }
>>  
>> +static int noop_int_ret(struct irq_data *data, unsigned int val)
>> +{
>> +	return 0;
>> +}
>> +
>>  /*
>>   * Generic no controller implementation
>>   */
>> @@ -57,5 +62,6 @@ struct irq_chip dummy_irq_chip = {
>>  	.irq_ack	= noop,
>>  	.irq_mask	= noop,
>>  	.irq_unmask	= noop,
>> +	.irq_set_wake	= noop_int_ret,
>>  };
>>  EXPORT_SYMBOL_GPL(dummy_irq_chip);
>>
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-usb" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
--
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/