From: Tejun Heo <tj@kernel.org>
To: Axel Birndt <towerlexa@gmx.de>
Cc: Jesse Barnes <jbarnes@virtuousgeek.org>, Greg KH <greg@kroah.com>,
Robert Hancock <hancockr@shaw.ca>,
Alan Cox <alan@lxorguk.ukuu.org.uk>,
linux-pci@vger.kernel.org,
Linux Kernel <linux-kernel@vger.kernel.org>,
Daniel Ritz <daniel.ritz@gmx.ch>,
Dominik Brodowski <linux@dominikbrodowski.net>,
Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>,
Tony Luck <tony.luck@intel.com>,
David Miller <davem@davemloft.net>
Subject: Re: [PATCH 3/3 pci#linux-next] pccard: configure CLS on attach
Date: Wed, 23 Sep 2009 10:18:46 +0900 [thread overview]
Message-ID: <4AB97776.3070906@kernel.org> (raw)
In-Reply-To: <4AB8A0A2.3010300@gmx.de>
Hello, Axel.
Axel Birndt wrote:
> Tejun Heo schrieb:
>> For non hotplug PCI devices, the system firmware usually configures
>> CLS correctly. For pccard devices system firmware can't do it and
>> Linux PCI layer doesn't do it either. Unfortunately this leads to
>> poor performance for certain devices (sata_sil). Unless MWI, which
>> requires separate configuration, is to be used, CLS doesn't affect
>> correctness, so the configuration should be harmless.
>>
>> This patch makes pci_set_cacheline_size() always built and export it
>> and make pccard call it during attach.
>>
>> Please note that some other PCI hotplug drivers (shpchp and pciehp)
>> also configure CLS on hotplug.
>
> i would like to ask, when this patch is usable for normal guy's, like me.
> could you tell me in which kernel release or in which bugfix this will
> be implemented?
>
> currently i'am using Kubuntu 8.04 with all current released bugfixes and
> kernel updates...
This patch is rather dangerous and should sit out for a while in
linux-next and then -rc. In mainline, I think it will appear in
2.6.33. Distros may choose to backport them but I don't think they'll
do it before the change at least lands on -rc which is at least 3+
months away. It's also possible for distros to backport it but enable
it only if certain kernel parameter is set but you'll have to ask each
distro whether they're willing to do such thing.
Thanks.
--
tejun
next prev parent reply other threads:[~2009-09-23 1:20 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-04 5:08 [PATCH 1/3] pci: determine CLS more intelligently Tejun Heo
2009-07-04 5:09 ` [PATCH 2/3] pci,sparc64: drop PCI_CACHE_LINE_BYTES Tejun Heo
2009-07-04 5:10 ` [PATCH 3/3] pccard: configure CLS on attach Tejun Heo
2009-07-04 9:29 ` [PATCH 1/3] pci: determine CLS more intelligently Ingo Molnar
2009-09-17 17:31 ` Jesse Barnes
2009-09-22 8:33 ` [PATCH 1/3 pci#linux-next] " Tejun Heo
2009-09-23 5:24 ` Benjamin Herrenschmidt
2009-10-06 16:53 ` Jesse Barnes
2009-09-22 8:34 ` [PATCH 2/3 pci#linux-next] pci,sparc64: drop PCI_CACHE_LINE_BYTES Tejun Heo
2009-09-22 8:34 ` [PATCH 3/3 pci#linux-next] pccard: configure CLS on attach Tejun Heo
2009-09-22 10:02 ` Axel Birndt
2009-09-23 1:18 ` Tejun Heo [this message]
2009-09-23 5:28 ` Benjamin Herrenschmidt
2009-09-28 17:52 ` [PATCH 1/3] pci: determine CLS more intelligently Jesse Barnes
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4AB97776.3070906@kernel.org \
--to=tj@kernel.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=benh@kernel.crashing.org \
--cc=daniel.ritz@gmx.ch \
--cc=davem@davemloft.net \
--cc=greg@kroah.com \
--cc=hancockr@shaw.ca \
--cc=jbarnes@virtuousgeek.org \
--cc=kaneshige.kenji@jp.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux@dominikbrodowski.net \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
--cc=tony.luck@intel.com \
--cc=towerlexa@gmx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox