diff options
author | Imre Kaloz <kaloz@openwrt.org> | 2015-03-26 09:42:59 +0000 |
---|---|---|
committer | Imre Kaloz <kaloz@openwrt.org> | 2015-03-26 09:42:59 +0000 |
commit | af69bf56b6810419879f1aecad8160697877a8c6 (patch) | |
tree | 3904731ce5bfb9f3c9f44ef4aa3350114ee4a46e /target/linux/mpc85xx/patches-3.19/210-spi-fsl-espi-preallocate-local-buffer.patch | |
parent | 6c0d6a3cb32b0b366b6c28aa7735f00a44226edd (diff) | |
download | upstream-af69bf56b6810419879f1aecad8160697877a8c6.tar.gz upstream-af69bf56b6810419879f1aecad8160697877a8c6.tar.bz2 upstream-af69bf56b6810419879f1aecad8160697877a8c6.zip |
drop 3.19 support
Signed-off-by: Imre Kaloz <kaloz@openwrt.org>
SVN-Revision: 45002
Diffstat (limited to 'target/linux/mpc85xx/patches-3.19/210-spi-fsl-espi-preallocate-local-buffer.patch')
-rw-r--r-- | target/linux/mpc85xx/patches-3.19/210-spi-fsl-espi-preallocate-local-buffer.patch | 154 |
1 files changed, 0 insertions, 154 deletions
diff --git a/target/linux/mpc85xx/patches-3.19/210-spi-fsl-espi-preallocate-local-buffer.patch b/target/linux/mpc85xx/patches-3.19/210-spi-fsl-espi-preallocate-local-buffer.patch deleted file mode 100644 index 579c7642ee..0000000000 --- a/target/linux/mpc85xx/patches-3.19/210-spi-fsl-espi-preallocate-local-buffer.patch +++ /dev/null @@ -1,154 +0,0 @@ -From: Gabor Juhos <juhosg@openwrt.org> -Subject: spi-fsl-espi: avoid frequent high order allocations - -The driver allocates 64KiB of memory fro a local buffer before -each transfer and releases that afterwards. When the memory is -fragmented this allocation often fails and causes a warning like -this: - - kworker/u2:2: page allocation failure: order:4, mode:0x10c0d0 - CPU: 0 PID: 7011 Comm: kworker/u2:2 Not tainted 3.10.24 #1 - Workqueue: ffe07000.spi mpc8xxx_spi_work - Call Trace: - [c1c6dcf0] [c0006914] show_stack+0x50/0x170 (unreliable) - [c1c6dd30] [c0259858] dump_stack+0x24/0x34 - [c1c6dd40] [c00672e8] warn_alloc_failed+0x120/0x13c - [c1c6dd90] [c0069920] __alloc_pages_nodemask+0x574/0x5c8 - [c1c6de20] [c0069990] __get_free_pages+0x1c/0x4c - [c1c6de30] [c0185174] fsl_espi_do_one_msg+0x128/0x2a0 - [c1c6de90] [c0184290] mpc8xxx_spi_work+0x50/0x7c - [c1c6dea0] [c0037af8] process_one_work+0x208/0x30c - [c1c6dec0] [c00387a0] worker_thread+0x20c/0x308 - [c1c6def0] [c003de60] kthread+0xa4/0xa8 - [c1c6df40] [c000c4bc] ret_from_kernel_thread+0x5c/0x64 - - m25p80 spi0.0: error -12 reading SR - end_request: I/O error, dev mtdblock3, sector 680 - SQUASHFS error: squashfs_read_data failed to read block 0x54a4a - SQUASHFS error: Unable to read data cache entry [54a4a] - -Preallocate the buffer from the probe routine to avoid -this. - -Signed-off-by: Gabor Juhos <juhosg@openwrt.org> ---- - drivers/spi/spi-fsl-espi.c | 34 ++++++++++++++++------------------ - drivers/spi/spi-fsl-lib.h | 1 + - 2 files changed, 17 insertions(+), 18 deletions(-) - -diff --git a/drivers/spi/spi-fsl-espi.c b/drivers/spi/spi-fsl-espi.c -index d0a73a0..f461dde 100644 ---- a/drivers/spi/spi-fsl-espi.c -+++ b/drivers/spi/spi-fsl-espi.c -@@ -329,17 +329,13 @@ static void fsl_espi_do_trans(struct spi_message *m, - static void fsl_espi_cmd_trans(struct spi_message *m, - struct fsl_espi_transfer *trans, u8 *rx_buff) - { -+ struct spi_device *spi = m->spi; -+ struct mpc8xxx_spi *mspi = spi_master_get_devdata(spi->master); - struct spi_transfer *t; -- u8 *local_buf; -+ u8 *local_buf = mspi->local_buf; - int i = 0; - struct fsl_espi_transfer *espi_trans = trans; - -- local_buf = kzalloc(SPCOM_TRANLEN_MAX, GFP_KERNEL); -- if (!local_buf) { -- espi_trans->status = -ENOMEM; -- return; -- } -- - list_for_each_entry(t, &m->transfers, transfer_list) { - if (t->tx_buf) { - memcpy(local_buf + i, t->tx_buf, t->len); -@@ -352,28 +348,23 @@ static void fsl_espi_cmd_trans(struct spi_message *m, - fsl_espi_do_trans(m, espi_trans); - - espi_trans->actual_length = espi_trans->len; -- kfree(local_buf); - } - - static void fsl_espi_rw_trans(struct spi_message *m, - struct fsl_espi_transfer *trans, u8 *rx_buff) - { -+ struct spi_device *spi = m->spi; -+ struct mpc8xxx_spi *mspi = spi_master_get_devdata(spi->master); - struct fsl_espi_transfer *espi_trans = trans; - unsigned int n_tx = espi_trans->n_tx; - unsigned int n_rx = espi_trans->n_rx; - struct spi_transfer *t; -- u8 *local_buf; -+ u8 *local_buf = mspi->local_buf; - u8 *rx_buf = rx_buff; - unsigned int trans_len; - unsigned int addr; - int i, pos, loop; - -- local_buf = kzalloc(SPCOM_TRANLEN_MAX, GFP_KERNEL); -- if (!local_buf) { -- espi_trans->status = -ENOMEM; -- return; -- } -- - for (pos = 0, loop = 0; pos < n_rx; pos += trans_len, loop++) { - trans_len = n_rx - pos; - if (trans_len > SPCOM_TRANLEN_MAX - n_tx) -@@ -407,8 +398,6 @@ static void fsl_espi_rw_trans(struct spi_message *m, - else - espi_trans->actual_length += espi_trans->len; - } -- -- kfree(local_buf); - } - - static int fsl_espi_do_one_msg(struct spi_master *master, -@@ -586,6 +575,7 @@ static irqreturn_t fsl_espi_irq(s32 irq, void *context_data) - static void fsl_espi_remove(struct mpc8xxx_spi *mspi) - { - iounmap(mspi->reg_base); -+ kfree(mspi->local_buf); - } - - static int fsl_espi_suspend(struct spi_master *master) -@@ -652,10 +642,16 @@ static struct spi_master * fsl_espi_probe(struct device *dev, - mpc8xxx_spi = spi_master_get_devdata(master); - mpc8xxx_spi->spi_remove = fsl_espi_remove; - -+ mpc8xxx_spi->local_buf = kzalloc(SPCOM_TRANLEN_MAX, GFP_KERNEL); -+ if (!mpc8xxx_spi->local_buf) { -+ ret = -ENOMEM; -+ goto err_probe; -+ } -+ - mpc8xxx_spi->reg_base = ioremap(mem->start, resource_size(mem)); - if (!mpc8xxx_spi->reg_base) { - ret = -ENOMEM; -- goto err_probe; -+ goto free_buf; - } - - reg_base = mpc8xxx_spi->reg_base; -@@ -722,6 +718,8 @@ unreg_master: - free_irq(mpc8xxx_spi->irq, mpc8xxx_spi); - free_irq: - iounmap(mpc8xxx_spi->reg_base); -+free_buf: -+ kfree(mpc8xxx_spi->local_buf); - err_probe: - spi_master_put(master); - err: -diff --git a/drivers/spi/spi-fsl-lib.h b/drivers/spi/spi-fsl-lib.h -index b4ed04e..3ffa984 100644 ---- a/drivers/spi/spi-fsl-lib.h -+++ b/drivers/spi/spi-fsl-lib.h -@@ -30,6 +30,7 @@ struct mpc8xxx_spi { - void *rx; - #ifdef CONFIG_SPI_FSL_ESPI - int len; -+ u8 *local_buf; - #endif - - int subblock; --- -2.3.0 - |