Re: [PATCH] dspbridge: wait less and check the mailbox more

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Felipe,

From: ext Felipe Contreras <felipe.contreras@xxxxxxxxx>
Subject: [PATCH] dspbridge: wait less and check the mailbox more
Date: Sat, 21 Feb 2009 00:04:17 +0100

> From: Felipe Contreras <felipe.contrers@xxxxxxxxx>
> 
> Profiling showed the __delay function is called a lot; checking the mbox
> more often seems to decrease the usage by 2/3 according to OProfile.

Good finding.

> The changes are based on 'arch/arm/plat-omap/mailbox.c'. Tested on OMAP3430.
> 
> Also, remove HW_MBOX_IsFull since it's not used by anybody.

Woundn't it be better to split this one into some logical parts,
tunning up part(decreasing delay) and cleanup part(the others)?

Home brewed MACROs themselves like UTIL_Wait() are basically evil from
kernel source readibility perspective, but keeping consistency within
specific module may be more important and this kind of cleaning up
would be better to be done at onece over a module, in separate
patches.

Hari,
Any comment? if it's ok, please merge it or newer ones into o-z as
well.

      Hiroshi DOYU
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux