Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1289816
| From | SF Markus Elfring <elfring@users.sourceforge.net> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | [PATCH 1/2] zram: Less checks in zram_bvec_write() after error detection |
| Date | 2015-12-11 19:30 +0100 |
| Message-ID | <qEAC6-862-19@gated-at.bofh.it> (permalink) |
| References | <qEuGl-43C-5@gated-at.bofh.it> <qEAsq-81R-25@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Fri, 11 Dec 2015 18:20:59 +0100
This issue was detected by using the Coccinelle software.
A few checks could be repeated by the zram_bvec_write() function
at two places even if the passed variables contained a null pointer.
* This implementation detail could be improved by adjustments
for jump targets according to the Linux coding style convention.
* Let us return directly if a memory allocation failed.
* Drop unnecessary initialisations for the variables "uncmem"
and "zstrm" then.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
drivers/block/zram/zram_drv.c | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c
index 47915d7..69d7fcd 100644
--- a/drivers/block/zram/zram_drv.c
+++ b/drivers/block/zram/zram_drv.c
@@ -652,9 +652,9 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
size_t clen;
unsigned long handle;
struct page *page;
- unsigned char *user_mem, *cmem, *src, *uncmem = NULL;
+ unsigned char *user_mem, *cmem, *src, *uncmem;
struct zram_meta *meta = zram->meta;
- struct zcomp_strm *zstrm = NULL;
+ struct zcomp_strm *zstrm;
unsigned long alloced_pages;
page = bvec->bv_page;
@@ -664,13 +664,11 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
* before to write the changes.
*/
uncmem = kmalloc(PAGE_SIZE, GFP_NOIO);
- if (!uncmem) {
- ret = -ENOMEM;
- goto out;
- }
+ if (!uncmem)
+ return -ENOMEM;
ret = zram_decompress_page(zram, uncmem, index);
if (ret)
- goto out;
+ goto free_uncmem;
}
zstrm = zcomp_strm_find(zram->comp);
@@ -696,7 +694,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
atomic64_inc(&zram->stats.zero_pages);
ret = 0;
- goto out;
+ goto check_strm;
}
ret = zcomp_compress(zram->comp, zstrm, uncmem, &clen);
@@ -708,7 +706,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
if (unlikely(ret)) {
pr_err("Compression failed! err=%d\n", ret);
- goto out;
+ goto check_strm;
}
src = zstrm->buffer;
if (unlikely(clen > max_zpage_size)) {
@@ -722,7 +720,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
pr_err("Error allocating memory for compressed page: %u, size=%zu\n",
index, clen);
ret = -ENOMEM;
- goto out;
+ goto check_strm;
}
alloced_pages = zs_get_total_pages(meta->mem_pool);
@@ -731,7 +729,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
if (zram->limit_pages && alloced_pages > zram->limit_pages) {
zs_free(meta->mem_pool, handle);
ret = -ENOMEM;
- goto out;
+ goto check_strm;
}
cmem = zs_map_object(meta->mem_pool, handle, ZS_MM_WO);
@@ -762,11 +760,13 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
/* Update stats */
atomic64_add(clen, &zram->stats.compr_data_size);
atomic64_inc(&zram->stats.pages_stored);
-out:
+check_strm:
if (zstrm)
zcomp_strm_release(zram->comp, zstrm);
- if (is_partial_io(bvec))
+ if (is_partial_io(bvec)) {
+free_uncmem:
kfree(uncmem);
+ }
return ret;
}
--
2.6.3
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
Source code review around jump label usage SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-11 13:10 +0100
Re: Source code review around jump label usage Julia Lawall <julia.lawall@lip6.fr> - 2015-12-11 13:20 +0100
Re: Source code review around jump label usage Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-11 13:50 +0100
Re: Source code review around jump label usage Christophe JAILLET <christophe.jaillet@wanadoo.fr> - 2015-12-11 19:10 +0100
[PATCH 0/2] block: Fine-tuning for two function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-11 19:20 +0100
[PATCH 2/2] z2ram: Delete a jump label in z2_init() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-11 19:30 +0100
Re: [PATCH 2/2] z2ram: Delete a jump label in z2_init() Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2015-12-14 01:40 +0100
Re: [PATCH 2/2] z2ram: Delete a jump label in z2_init() Geert Uytterhoeven <geert@linux-m68k.org> - 2015-12-14 10:20 +0100
[PATCH 1/2] zram: Less checks in zram_bvec_write() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-11 19:30 +0100
Re: [PATCH 1/2] zram: Less checks in zram_bvec_write() after error detection Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2015-12-14 01:30 +0100
Re: [PATCH 1/2] zram: Less checks in zram_bvec_write() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 08:00 +0100
Re: [PATCH 1/2] zram: Less checks in zram_bvec_write() after error detection Julia Lawall <julia.lawall@lip6.fr> - 2015-12-14 08:20 +0100
Re: [PATCH 1/2] zram: Less checks in zram_bvec_write() after error detection Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2015-12-14 11:10 +0100
Re: [PATCH 1/2] zram: Less checks in zram_bvec_write() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 15:10 +0100
[PATCH] uinput: Rename a jump label in uinput_ioctl_handler() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 10:20 +0100
Re: [PATCH] uinput: Rename a jump label in uinput_ioctl_handler() Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2015-12-12 23:30 +0100
[PATCH 0/7] iSCSI-target: Fine-tuning for three function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:40 +0100
[PATCH 1/7] iscsi-target: Use a variable initialisation in iscsi_set_default_param() directly SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:40 +0100
Re: [PATCH 1/7] iscsi-target: Use a variable initialisation in iscsi_set_default_param() directly Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-12 21:00 +0100
Re: [PATCH 1/7] iscsi-target: Use a variable initialisation in iscsi_set_default_param() directly SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 22:30 +0100
Re: [PATCH 1/7] iscsi-target: Use a variable initialisation in iscsi_set_default_param() directly Johannes Thumshirn <jthumshirn@suse.de> - 2015-12-14 09:50 +0100
Re: [PATCH 1/7] iscsi-target: Use a variable initialisation in iscsi_set_default_param() directly SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 12:40 +0100
[PATCH 2/7] iscsi-target: Less checks in iscsi_set_default_param() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:40 +0100
[PATCH 3/7] iscsi-target: Delete an unnecessary variable initialisation in iscsi_create_default_params() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:50 +0100
Re: [PATCH 4/7] iscsi-target: Make a variable initialisation a bit more obvious in iscsi_create_default_params() Julia Lawall <julia.lawall@lip6.fr> - 2015-12-12 15:50 +0100
Re: [PATCH 4/7] iscsi-target: Make a variable initialisation a bit more obvious in iscsi_create_default_params() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 16:10 +0100
[PATCH 6/7] iscsi-target: Delete unnecessary variable initialisations in iscsi_check_valuelist_for_support() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:50 +0100
[PATCH 7/7] iscsi-target: Make two variable initialisations a bit more obvious in iscsi_check_valuelist_for_support() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:50 +0100
Re: [PATCH 7/7] iscsi-target: Make two variable initialisations a bit more obvious in iscsi_check_valuelist_for_support() walter harms <wharms@bfs.de> - 2015-12-12 18:20 +0100
[PATCH 4/7] iscsi-target: Make a variable initialisation a bit more obvious in iscsi_create_default_params() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:50 +0100
[PATCH 5/7] iscsi-target: Rename a jump label in iscsi_create_default_params() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-12 15:50 +0100
[PATCH 0/7] staging-Lustre: Fine-tuning for some function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 14:50 +0100
[PATCH 2/7] staging: lustre: Rename a jump label for ptlrpc_req_finished() calls SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 15:00 +0100
Re: [PATCH 2/7] staging: lustre: Rename a jump label for ptlrpc_req_finished() calls Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-14 08:00 +0100
Re: [PATCH 2/7] staging: lustre: Rename a jump label for ptlrpc_req_finished() calls SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 10:10 +0100
Re: [PATCH 2/7] staging: lustre: Rename a jump label for ptlrpc_req_finished() calls Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-14 10:40 +0100
Re: staging: lustre: Rename a jump label for ptlrpc_req_finished() calls SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 11:10 +0100
[PATCH 1/7] staging: lustre: Delete unnecessary goto statements in six functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 15:00 +0100
Re: [PATCH 1/7] staging: lustre: Delete unnecessary goto statements in six functions Joe Perches <joe@perches.com> - 2015-12-15 15:30 +0100
Re: [PATCH 1/7] staging: lustre: Delete unnecessary goto statements in six functions Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-15 15:50 +0100
Re: [PATCH 1/7] staging: lustre: Delete unnecessary goto statements in six functions Joe Perches <joe@perches.com> - 2015-12-15 16:10 +0100
Re: [PATCH 1/7] staging: lustre: Delete unnecessary goto statements in six functions Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-15 18:50 +0100
Re: [PATCH 1/7] staging: lustre: Delete unnecessary goto statements in six functions Joe Perches <joe@perches.com> - 2015-12-15 19:20 +0100
Re: staging: lustre: Delete unnecessary goto statements in six functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-15 19:30 +0100
Re: staging: lustre: Delete unnecessary goto statements in six functions Joe Perches <joe@perches.com> - 2015-12-15 19:40 +0100
Re: staging: lustre: Delete unnecessary goto statements in six functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-15 20:00 +0100
Re: staging: lustre: Delete unnecessary goto statements in six functions Joe Perches <joe@perches.com> - 2015-12-15 20:00 +0100
Re: staging: lustre: Delete unnecessary goto statements in six functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-15 19:10 +0100
Re: staging: lustre: Delete unnecessary goto statements in six functions Joe Perches <joe@perches.com> - 2015-12-15 19:30 +0100
[PATCH 3/7] staging: lustre: Rename a jump label for a kfree(key) call SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 15:00 +0100
[PATCH 6/7] staging: lustre: A few checks less in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 15:00 +0100
[PATCH 5/7] staging: lustre: Less checks in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 15:00 +0100
Re: [PATCH 5/7] staging: lustre: Less checks in mgc_process_recover_log() after error detection Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-14 12:10 +0100
Re: [PATCH 5/7] staging: lustre: Less checks in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 13:10 +0100
Re: [PATCH 5/7] staging: lustre: Less checks in mgc_process_recover_log() after error detection Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-14 13:40 +0100
Re: staging: lustre: Less checks in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 13:50 +0100
Re: staging: lustre: Less checks in mgc_process_recover_log() after error detection Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-14 15:00 +0100
Re: staging: lustre: Less checks in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 18:50 +0100
Re: staging: lustre: Less checks in mgc_process_recover_log() after error detection Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-15 12:50 +0100
Re: staging: lustre: Less checks in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-15 16:10 +0100
[PATCH 4/7] staging: lustre: Delete an unnecessary variable initialisation in mgc_process_recover_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 15:00 +0100
[PATCH 7/7] staging: lustre: Rename a jump label for module_put() calls SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-13 15:10 +0100
[PATCH v2 0/4] staging-Lustre: Fine-tuning for some function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-21 20:10 +0100
[PATCH v2 1/4] staging: lustre: Delete unnecessary goto statements in six functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-21 20:10 +0100
[PATCH v2 4/4] staging: lustre: Fix a jump label position in osc_get_info() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-21 20:20 +0100
[PATCH v2 3/4] staging: lustre: Less checks in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-21 20:20 +0100
Re: [PATCH v2 3/4] staging: lustre: Less checks in mgc_process_recover_log() after error detection Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2015-12-22 00:50 +0100
Re: [PATCH v2 3/4] staging: lustre: Less checks in mgc_process_recover_log() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-22 08:20 +0100
Re: [PATCH v2 3/4] staging: lustre: Less checks in mgc_process_recover_log() after error detection Dan Carpenter <dan.carpenter@oracle.com> - 2015-12-22 09:10 +0100
[PATCH v2 2/4] staging: lustre: Delete an unnecessary variable initialisation in mgc_process_recover_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-21 20:20 +0100
[POWERPC] bootwrapper: One check less in fsl_get_immr() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-14 23:20 +0100
Re: [POWERPC] bootwrapper: One check less in fsl_get_immr() after error detection Scott Wood <scottwood@freescale.com> - 2015-12-14 23:40 +0100
[PATCH] block-LDM: One function call less in ldm_validate_tocblocks() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-23 10:50 +0100
Re: [PATCH] block-LDM: One function call less in ldm_validate_tocblocks() after error detection Julia Lawall <julia.lawall@lip6.fr> - 2015-12-23 11:50 +0100
[PATCH 0/5] block-LDM: Improvements for exception handling SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-23 19:10 +0100
[PATCH 1/5] block-LDM: One function call less in ldm_validate_tocblocks() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-23 19:10 +0100
[PATCH 2/5] block-LDM: Delete extra log messages for memory allocation failures SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-23 19:10 +0100
[PATCH 3/5] block-LDM: One function call less in ldm_partition() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-23 19:20 +0100
[PATCH 5/5] block-LDM: Fine-tuning for the source code formatting SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-23 19:20 +0100
[PATCH 4/5] block-LDM: Less function calls in ldm_validate_privheads() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-23 19:20 +0100
[PATCH 0/3] Documentation-getdelays: Fine-tuning for two functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-24 13:40 +0100
[PATCH 2/3] Documentation-getdelays: Apply a recommendation from "checkpatch.pl" in main() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-24 13:40 +0100
Re: [PATCH 2/3] Documentation-getdelays: Apply a recommendation from "checkpatch.pl" in main() Jonathan Corbet <corbet@lwn.net> - 2015-12-24 15:30 +0100
[PATCH 1/3] Documentation-getdelays: Fix a check for container file usage in main() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-24 13:40 +0100
Re: [PATCH 1/3] Documentation-getdelays: Fix a check for container file usage in main() Jonathan Corbet <corbet@lwn.net> - 2015-12-24 15:30 +0100
Re: Documentation-getdelays: Fix a check for container file usage in main() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-24 19:00 +0100
[PATCH 3/3] Documentation-getdelays: Less function calls in usage() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-24 13:40 +0100
Re: [PATCH 3/3] Documentation-getdelays: Less function calls in usage() Jonathan Corbet <corbet@lwn.net> - 2015-12-24 15:30 +0100
Re: Documentation-getdelays: Less function calls in usage() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-24 20:50 +0100
ACPI-fan: Another source code review around null pointer handling? SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-25 11:40 +0100
sata_mv: Another source code review around exception handling? SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-25 17:10 +0100
Re: sata_mv: Another source code review around exception handling? Tejun Heo <tj@kernel.org> - 2015-12-28 17:20 +0100
[PATCH] gpio-ucb1400: Delete an unnecessary variable initialisation in ucb1400_gpio_probe() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-25 20:00 +0100
[PATCH] i2c-core: One function call less in acpi_i2c_space_handler() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 07:40 +0100
Re: [PATCH] i2c-core: One function call less in acpi_i2c_space_handler() after error detection kbuild test robot <lkp@intel.com> - 2015-12-26 07:50 +0100
[PATCH v2] i2c-core: One function call less in acpi_i2c_space_handler() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 08:10 +0100
Re: [PATCH v2] i2c-core: One function call less in acpi_i2c_space_handler() after error detection Wolfram Sang <wsa@the-dreams.de> - 2015-12-26 08:50 +0100
Re: i2c-core: One function call less in acpi_i2c_space_handler() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 10:00 +0100
Re: i2c-core: One function call less in acpi_i2c_space_handler() after error detection Wolfram Sang <wsa@the-dreams.de> - 2015-12-26 19:50 +0100
Re: i2c-core: One function call less in acpi_i2c_space_handler() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 20:40 +0100
[PATCH 0/3] IDE-ACPI: Fine-tuning for a function SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 11:20 +0100
[PATCH 2/3] IDE-ACPI: Delete unnecessary null pointer checks in ide_get_dev_handle() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 11:20 +0100
[PATCH 1/3] IDE-ACPI: One function call less in ide_get_dev_handle() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 11:20 +0100
[PATCH 3/3] IDE-ACPI: Move an assignment for one variable in ide_get_dev_handle() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 11:30 +0100
Re: [PATCH 0/3] IDE-ACPI: Fine-tuning for a function David Miller <davem@davemloft.net> - 2015-12-26 19:20 +0100
Re: [PATCH 0/3] IDE-ACPI: Fine-tuning for a function Joe Perches <joe@perches.com> - 2015-12-27 00:50 +0100
Re: [PATCH 0/3] IDE-ACPI: Fine-tuning for a function Julia Lawall <julia.lawall@lip6.fr> - 2015-12-27 07:10 +0100
[PATCH] iio: qcom-spmi-vadc: One check less in vadc_measure_ref_points() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 14:10 +0100
[PATCH 0/6] InfiniBand-ocrdma: Fine-tuning for some function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 19:40 +0100
[PATCH 2/6] InfiniBand-ocrdma: Delete unnecessary variable initialisations in 11 functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 19:50 +0100
[PATCH 4/6] InfiniBand-ocrdma: Return a value from a function call in _ocrdma_modify_qp() directly SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 19:50 +0100
[PATCH 1/6] InfiniBand-ocrdma: One variable and jump label less in ocrdma_alloc_ucontext_pd() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 19:50 +0100
Re: [PATCH 1/6] InfiniBand-ocrdma: One variable and jump label less in ocrdma_alloc_ucontext_pd() kbuild test robot <lkp@intel.com> - 2015-12-26 20:50 +0100
[PATCH v2 1/6] InfiniBand-ocrdma: One jump label less in ocrdma_alloc_ucontext_pd() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 22:30 +0100
[PATCH 3/6] InfiniBand-ocrdma: Returning only value constants in ocrdma_qp_state_change() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 19:50 +0100
[PATCH 5/6] InfiniBand-ocrdma: Returning only value constants in ocrdma_resize_cq() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 20:00 +0100
[PATCH 6/6] InfiniBand-ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-26 20:00 +0100
[PATCH 0/2] InfiniBand-iSER: Refactoring for two function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-27 13:40 +0100
[PATCH 1/2] InfiniBand-iSER: One jump label less in iser_reg_sig_mr() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-27 13:50 +0100
[PATCH 2/2] InfiniBand-iSER-target: One jump label less in isert_reg_sig_mr() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-27 13:50 +0100
Re: [PATCH 0/2] InfiniBand-iSER: Refactoring for two function implementations Leon Romanovsky <leon@leon.nu> - 2015-12-27 13:50 +0100
Re: [PATCH 0/2] InfiniBand-iSER: Refactoring for two function implementations Leon Romanovsky <leon@leon.nu> - 2015-12-27 14:00 +0100
Re: [PATCH 0/2] InfiniBand-iSER: Refactoring for two function implementations Sagi Grimberg <sagig@dev.mellanox.co.il> - 2015-12-27 16:30 +0100
[PATCH] [media] si2165: Refactoring for si2165_writereg_mask8() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 03:50 +0100
[PATCH] [media] bttv: Returning only value constants in two functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 03:50 +0100
[PATCH] [media] tuners: One check less in m88rs6000t_get_rf_strength() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 10:20 +0100
Re: [PATCH] [media] tuners: One check less in m88rs6000t_get_rf_strength() after error detection Julia Lawall <julia.lawall@lip6.fr> - 2015-12-28 10:30 +0100
Re: [media] tuners: One check less in m88rs6000t_get_rf_strength() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 11:40 +0100
Re: [media] tuners: One check less in m88rs6000t_get_rf_strength() after error detection Julia Lawall <julia.lawall@lip6.fr> - 2015-12-28 11:40 +0100
[PATCH 0/2] [media] m88rs6000t: Fine-tuning for some function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 15:40 +0100
[PATCH 1/2] [media] m88rs6000t: Better exception handling in five functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 15:40 +0100
Re: [PATCH 1/2] [media] m88rs6000t: Better exception handling in five functions Julia Lawall <julia.lawall@lip6.fr> - 2015-12-28 15:50 +0100
Re: [media] m88rs6000t: Better exception handling in five functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 16:10 +0100
Re: [media] m88rs6000t: Better exception handling in five functions Julia Lawall <julia.lawall@lip6.fr> - 2015-12-28 16:20 +0100
[PATCH 2/2] [media] tuners: Refactoring for m88rs6000t_sleep() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 15:50 +0100
[PATCH 0/2] [media] r820t: Fine-tuning for generic_set_freq() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 17:30 +0100
[PATCH 1/2] [media] r820t: Delete an unnecessary variable initialisation in generic_set_freq() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 17:40 +0100
[PATCH 2/2] [media] r820t: Better exception handling in generic_set_freq() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 17:40 +0100
[PATCH] [media] xc5000: Faster result reporting in xc_load_fw_and_init_tuner() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 20:30 +0100
[PATCH] [media] airspy: Better exception handling in two functions SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 22:20 +0100
[PATCH] [media] au0828: Refactoring for start_urb_transfer() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-28 23:00 +0100
[PATCH] [media] hdpvr: Refactoring for hdpvr_read() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 11:20 +0100
[PATCH] [media] msi2500: Delete an unnecessary check in msi2500_set_usb_adc() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 12:40 +0100
[PATCH] mfd-dm355evm_msp: One function call less in add_child() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 14:10 +0100
[PATCH 2/2] mfd: smsc-ece1099: Refactoring for smsc_i2c_probe() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 15:20 +0100
[PATCH 1/2] mfd: smsc-ece1099: Delete an unnecessary variable initialisation in smsc_i2c_probe() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 15:20 +0100
[PATCH 0/2] mfd: smsc-ece1099: Fine-tuning for smsc_i2c_probe() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 15:20 +0100
[PATCH] mfd: twl-core: One function call less in add_numbered_child() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 19:40 +0100
[PATCH] mmc-core: One check less in mmc_select_hs200() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 21:00 +0100
[PATCH 0/2] mmc-host: Fine-tuning for one function SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 22:00 +0100
[PATCH 1/2] mmc-sdricoh_cs: Delete unnecessary variable initialisations in sdricoh_init_mmc() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 22:10 +0100
[PATCH 2/2] mmc-sdricoh_cs: Less checks in sdricoh_init_mmc() after, error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-29 22:10 +0100
[PATCH 3/3] mtd-rfd_ftl: Refactoring for erase_block() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-31 21:30 +0100
[PATCH 1/3] mtd-rfd_ftl: Replace a variable initialisation by assignments in move_block_contents() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-31 21:30 +0100
[PATCH 2/3] mtd-rfd_ftl: Refactoring for move_block_contents() SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-31 21:30 +0100
[PATCH 0/3] mtd-rfd_ftl: Fine-tuning for two function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-31 21:30 +0100
[PATCH] net-thunder: One check less in nicvf_register_interrupts() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2015-12-31 22:50 +0100
[PATCH] be2net: Delete an unnecessary check in two functions SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 00:30 +0100
[PATCH 0/3] net-gianfar: Fine-tuning for gfar_ethflow_to_filer_table() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 13:20 +0100
[PATCH 2/3] net-gianfar: Delete unnecessary variable initialisations in gfar_ethflow_to_filer_table() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 13:30 +0100
[PATCH 3/3] net-gianfar: Extend an initialisation clause of a for loop in gfar_ethflow_to_filer_table() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 13:30 +0100
[PATCH 1/3] net-gianfar: Less function calls in gfar_ethflow_to_filer_table() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 13:30 +0100
Re: [PATCH 1/3] net-gianfar: Less function calls in gfar_ethflow_to_filer_table() after error detection Julia Lawall <julia.lawall@lip6.fr> - 2016-01-01 13:40 +0100
Re: [PATCH 1/3] net-gianfar: Less function calls in gfar_ethflow_to_filer_table() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 14:00 +0100
Re: [PATCH 1/3] net-gianfar: Less function calls in gfar_ethflow_to_filer_table() after error detection Julia Lawall <julia.lawall@lip6.fr> - 2016-01-01 14:10 +0100
Re: [PATCH 1/3] net-gianfar: Less function calls in gfar_ethflow_to_filer_table() after error detection Francois Romieu <romieu@fr.zoreil.com> - 2016-01-01 15:50 +0100
[PATCH v2 1/3] net-gianfar: Less function calls in gfar_ethflow_to_filer_table() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 14:10 +0100
[PATCH] net-i40e: Replace variable initialisations by assignments in i40e_vc_get_vf_resources_msg() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 15:40 +0100
[PATCH] net-huawei_cdc_ncm: Delete an unnecessary variable initialisation in huawei_cdc_ncm_bind() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 17:00 +0100
[PATCH 0/2] net-qmi_wwan: Fine-tuning for two function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 18:00 +0100
[PATCH 1/2] net-qmi_wwan: Refactoring for qmi_wwan_bind() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 18:00 +0100
[PATCH 2/2] net-qmi_wwan: Delete an unnecessary variable initialisation in qmi_wwan_register_subdriver() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 18:00 +0100
[PATCH 0/2] net-ath9k_htc: Fine-tuning for two function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 19:30 +0100
[PATCH 2/2] net-ath9k_htc: Replace a variable initialisation by an assignment in ath9k_htc_set_channel() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 19:30 +0100
Re: [PATCH 2/2] net-ath9k_htc: Replace a variable initialisation by an assignment in ath9k_htc_set_channel() Oleksij Rempel <linux@rempel-privat.de> - 2016-01-01 20:20 +0100
[PATCH 1/2] net-ath9k_htc: Delete an unnecessary variable initialisation in ath9k_hif_usb_rx_stream() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 19:30 +0100
Re: [PATCH 1/2] net-ath9k_htc: Delete an unnecessary variable initialisation in ath9k_hif_usb_rx_stream() Oleksij Rempel <linux@rempel-privat.de> - 2016-01-01 20:20 +0100
[PATCH] net-brcmfmac: Delete an unnecessary variable initialisation in brcmf_sdio_download_firmware() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 20:30 +0100
[PATCH 0/3] net-iwlegacy: Fine-tuning for il_eeprom_init() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 21:30 +0100
[PATCH 2/3] net-iwlegacy: One check less in il_eeprom_init() after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 21:40 +0100
[PATCH 1/3] net-iwlegacy: Refactoring for il_eeprom_init() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 21:40 +0100
[PATCH 3/3] net-iwlegacy: Another refactoring for il_eeprom_init() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 21:40 +0100
[PATCH] net-libertas: Better exception handling in if_spi_host_to_card_worker() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-01-01 22:40 +0100
Re: [PATCH] net-libertas: Better exception handling in if_spi_host_to_card_worker() Julia Lawall <julia.lawall@lip6.fr> - 2016-01-01 22:50 +0100
csiph-web