PCMCIA stack reduction patch [Was: Re: Realtime Preemption, 2.6.12, Beginners Guide?]

Hi,

On Sat, Jul 09, 2005 at 03:26:57PM +0200, Ingo Molnar wrote:
> 
> > (gdb) ####################################
> > (gdb) # c02a0a26, stack size:  416 bytes #
> > (gdb) ####################################
> > (gdb) 0xc02a0a26 is in pcmcia_device_query (drivers/pcmcia/ds.c:436).
> 
> ----
> this patch reduces the stack footprint of pcmcia_device_query() from 416 
> bytes to 36 bytes. (patch only build-tested)

Applied and tested, but without the final hunk.

> @@ -856,7 +868,9 @@ static int bind_request(struct pcmcia_bu
>  rescan:
>  	p_dev->cardmgr = p_drv;
>  
> -	pcmcia_device_query(p_dev);
> +	ret = pcmcia_device_query(p_dev);
> +	if (ret)
> +		goto err_put_module;
>  
>  	/*
>  	 * Prevent this racing with a card insertion.


We don't check the return value here for a reason.

Thanks,

	Dominik
-
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/

---

• References:
  • Realtime Preemption, 2.6.12, Beginners Guide?
    • From: Alistair John Strachan <[email protected]>
  • Re: Realtime Preemption, 2.6.12, Beginners Guide?
    • From: Alistair John Strachan <[email protected]>
  • Re: Realtime Preemption, 2.6.12, Beginners Guide?
    • From: Ingo Molnar <[email protected]>
  • Re: Realtime Preemption, 2.6.12, Beginners Guide?
    • From: Alistair John Strachan <[email protected]>
  • Re: Realtime Preemption, 2.6.12, Beginners Guide?
    • From: Ingo Molnar <[email protected]>
  • Re: Realtime Preemption, 2.6.12, Beginners Guide?
    • From: Ingo Molnar <[email protected]>

• Prev by Date: Re: [PATCH] audit: file system auditing based on location and name
• Next by Date: Re: 2.6.12 vs. /sbin/cardmgr
• Previous by thread: Re: Realtime Preemption, 2.6.12, Beginners Guide?
• Next by thread: Re: Realtime Preemption, 2.6.12, Beginners Guide?
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]