Re: [PATCH 1/1] staging: io: Remove unnecessary OOM message
On Thu, 5 Mar 2015, Quentin Lambert wrote: > > > On 05/03/2015 11:28, Julia Lawall wrote: > > It's i2o, not io, and it is on the way out of the kernel. > sorry about that, > since it is on its way out I guess there is no need for me to submit a correct > version of this patch? > > Are there resources to know whether or not a driver will be dropped or is it > enough to check if there is > a TODO file in its directory? These are the staging drivers that I know of: i2o line6 media/parport media/tlg2300 media/vino julia -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/1] staging: io: Remove unnecessary OOM message
On 05/03/2015 11:28, Julia Lawall wrote: It's i2o, not io, and it is on the way out of the kernel. sorry about that, since it is on its way out I guess there is no need for me to submit a correct version of this patch? Are there resources to know whether or not a driver will be dropped or is it enough to check if there is a TODO file in its directory? Quentin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/1] staging: io: Remove unnecessary OOM message
It's i2o, not io, and it is on the way out of the kernel. julia On Thu, 5 Mar 2015, Quentin Lambert wrote: > This patch reduces the kernel size by removing error messages that duplicate > the normal OOM message. > > Signed-off-by: Quentin Lambert > --- > drivers/staging/i2o/i2o_block.c | 1 - > drivers/staging/i2o/i2o_config.c | 7 +-- > drivers/staging/i2o/iop.c| 10 ++ > 3 files changed, 3 insertions(+), 15 deletions(-) > > diff --git a/drivers/staging/i2o/i2o_block.c b/drivers/staging/i2o/i2o_block.c > index 0a13c64..a26e2a0 100644 > --- a/drivers/staging/i2o/i2o_block.c > +++ b/drivers/staging/i2o/i2o_block.c > @@ -965,7 +965,6 @@ static struct i2o_block_device > *i2o_block_device_alloc(void) > > dev = kzalloc(sizeof(*dev), GFP_KERNEL); > if (!dev) { > - osm_err("Insufficient memory to allocate I2O Block disk.\n"); > rc = -ENOMEM; > goto exit; > } > diff --git a/drivers/staging/i2o/i2o_config.c > b/drivers/staging/i2o/i2o_config.c > index 5748663..29e90e2 100644 > --- a/drivers/staging/i2o/i2o_config.c > +++ b/drivers/staging/i2o/i2o_config.c > @@ -591,11 +591,8 @@ static int i2o_cfg_passthru32(struct file *file, > unsigned cmnd, > > rcode = -ENOMEM; > reply = kzalloc(reply_size, GFP_KERNEL); > - if (!reply) { > - printk(KERN_WARNING "%s: Could not allocate reply buffer\n", > -c->name); > + if (!reply) > goto out; > - } > > sg_offset = (msg->u.head[0] >> 4) & 0x0f; > > @@ -833,8 +830,6 @@ static int i2o_cfg_passthru(unsigned long arg) > > reply = kzalloc(reply_size, GFP_KERNEL); > if (!reply) { > - printk(KERN_WARNING "%s: Could not allocate reply buffer\n", > -c->name); > rcode = -ENOMEM; > goto out; > } > diff --git a/drivers/staging/i2o/iop.c b/drivers/staging/i2o/iop.c > index 52334fc..47c785c 100644 > --- a/drivers/staging/i2o/iop.c > +++ b/drivers/staging/i2o/iop.c > @@ -99,11 +99,8 @@ u32 i2o_cntxt_list_add(struct i2o_controller * c, void > *ptr) > c->name); > > entry = kmalloc(sizeof(*entry), GFP_ATOMIC); > - if (!entry) { > - osm_err("%s: Could not allocate memory for context list element" > - "\n", c->name); > + if (!entry) > return 0; > - } > > entry->ptr = ptr; > entry->timestamp = jiffies; > @@ -1047,11 +1044,8 @@ struct i2o_controller *i2o_iop_alloc(void) > char poolname[32]; > > c = kzalloc(sizeof(*c), GFP_KERNEL); > - if (!c) { > - osm_err("i2o: Insufficient memory to allocate a I2O controller." > - "\n"); > + if (!c) > return ERR_PTR(-ENOMEM); > - } > > c->unit = unit++; > sprintf(c->name, "iop%d", c->unit); > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 1/1] staging: io: Remove unnecessary OOM message
This patch reduces the kernel size by removing error messages that duplicate the normal OOM message. Signed-off-by: Quentin Lambert --- drivers/staging/i2o/i2o_block.c | 1 - drivers/staging/i2o/i2o_config.c | 7 +-- drivers/staging/i2o/iop.c| 10 ++ 3 files changed, 3 insertions(+), 15 deletions(-) diff --git a/drivers/staging/i2o/i2o_block.c b/drivers/staging/i2o/i2o_block.c index 0a13c64..a26e2a0 100644 --- a/drivers/staging/i2o/i2o_block.c +++ b/drivers/staging/i2o/i2o_block.c @@ -965,7 +965,6 @@ static struct i2o_block_device *i2o_block_device_alloc(void) dev = kzalloc(sizeof(*dev), GFP_KERNEL); if (!dev) { - osm_err("Insufficient memory to allocate I2O Block disk.\n"); rc = -ENOMEM; goto exit; } diff --git a/drivers/staging/i2o/i2o_config.c b/drivers/staging/i2o/i2o_config.c index 5748663..29e90e2 100644 --- a/drivers/staging/i2o/i2o_config.c +++ b/drivers/staging/i2o/i2o_config.c @@ -591,11 +591,8 @@ static int i2o_cfg_passthru32(struct file *file, unsigned cmnd, rcode = -ENOMEM; reply = kzalloc(reply_size, GFP_KERNEL); - if (!reply) { - printk(KERN_WARNING "%s: Could not allocate reply buffer\n", - c->name); + if (!reply) goto out; - } sg_offset = (msg->u.head[0] >> 4) & 0x0f; @@ -833,8 +830,6 @@ static int i2o_cfg_passthru(unsigned long arg) reply = kzalloc(reply_size, GFP_KERNEL); if (!reply) { - printk(KERN_WARNING "%s: Could not allocate reply buffer\n", - c->name); rcode = -ENOMEM; goto out; } diff --git a/drivers/staging/i2o/iop.c b/drivers/staging/i2o/iop.c index 52334fc..47c785c 100644 --- a/drivers/staging/i2o/iop.c +++ b/drivers/staging/i2o/iop.c @@ -99,11 +99,8 @@ u32 i2o_cntxt_list_add(struct i2o_controller * c, void *ptr) c->name); entry = kmalloc(sizeof(*entry), GFP_ATOMIC); - if (!entry) { - osm_err("%s: Could not allocate memory for context list element" - "\n", c->name); + if (!entry) return 0; - } entry->ptr = ptr; entry->timestamp = jiffies; @@ -1047,11 +1044,8 @@ struct i2o_controller *i2o_iop_alloc(void) char poolname[32]; c = kzalloc(sizeof(*c), GFP_KERNEL); - if (!c) { - osm_err("i2o: Insufficient memory to allocate a I2O controller." - "\n"); + if (!c) return ERR_PTR(-ENOMEM); - } c->unit = unit++; sprintf(c->name, "iop%d", c->unit); -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 1/1] staging: io: Remove unnecessary OOM message
This patch reduces the kernel size by removing error messages that duplicate the normal OOM message. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/staging/i2o/i2o_block.c | 1 - drivers/staging/i2o/i2o_config.c | 7 +-- drivers/staging/i2o/iop.c| 10 ++ 3 files changed, 3 insertions(+), 15 deletions(-) diff --git a/drivers/staging/i2o/i2o_block.c b/drivers/staging/i2o/i2o_block.c index 0a13c64..a26e2a0 100644 --- a/drivers/staging/i2o/i2o_block.c +++ b/drivers/staging/i2o/i2o_block.c @@ -965,7 +965,6 @@ static struct i2o_block_device *i2o_block_device_alloc(void) dev = kzalloc(sizeof(*dev), GFP_KERNEL); if (!dev) { - osm_err(Insufficient memory to allocate I2O Block disk.\n); rc = -ENOMEM; goto exit; } diff --git a/drivers/staging/i2o/i2o_config.c b/drivers/staging/i2o/i2o_config.c index 5748663..29e90e2 100644 --- a/drivers/staging/i2o/i2o_config.c +++ b/drivers/staging/i2o/i2o_config.c @@ -591,11 +591,8 @@ static int i2o_cfg_passthru32(struct file *file, unsigned cmnd, rcode = -ENOMEM; reply = kzalloc(reply_size, GFP_KERNEL); - if (!reply) { - printk(KERN_WARNING %s: Could not allocate reply buffer\n, - c-name); + if (!reply) goto out; - } sg_offset = (msg-u.head[0] 4) 0x0f; @@ -833,8 +830,6 @@ static int i2o_cfg_passthru(unsigned long arg) reply = kzalloc(reply_size, GFP_KERNEL); if (!reply) { - printk(KERN_WARNING %s: Could not allocate reply buffer\n, - c-name); rcode = -ENOMEM; goto out; } diff --git a/drivers/staging/i2o/iop.c b/drivers/staging/i2o/iop.c index 52334fc..47c785c 100644 --- a/drivers/staging/i2o/iop.c +++ b/drivers/staging/i2o/iop.c @@ -99,11 +99,8 @@ u32 i2o_cntxt_list_add(struct i2o_controller * c, void *ptr) c-name); entry = kmalloc(sizeof(*entry), GFP_ATOMIC); - if (!entry) { - osm_err(%s: Could not allocate memory for context list element - \n, c-name); + if (!entry) return 0; - } entry-ptr = ptr; entry-timestamp = jiffies; @@ -1047,11 +1044,8 @@ struct i2o_controller *i2o_iop_alloc(void) char poolname[32]; c = kzalloc(sizeof(*c), GFP_KERNEL); - if (!c) { - osm_err(i2o: Insufficient memory to allocate a I2O controller. - \n); + if (!c) return ERR_PTR(-ENOMEM); - } c-unit = unit++; sprintf(c-name, iop%d, c-unit); -- 1.9.1 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/1] staging: io: Remove unnecessary OOM message
On 05/03/2015 11:28, Julia Lawall wrote: It's i2o, not io, and it is on the way out of the kernel. sorry about that, since it is on its way out I guess there is no need for me to submit a correct version of this patch? Are there resources to know whether or not a driver will be dropped or is it enough to check if there is a TODO file in its directory? Quentin -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/1] staging: io: Remove unnecessary OOM message
On Thu, 5 Mar 2015, Quentin Lambert wrote: On 05/03/2015 11:28, Julia Lawall wrote: It's i2o, not io, and it is on the way out of the kernel. sorry about that, since it is on its way out I guess there is no need for me to submit a correct version of this patch? Are there resources to know whether or not a driver will be dropped or is it enough to check if there is a TODO file in its directory? These are the staging drivers that I know of: i2o line6 media/parport media/tlg2300 media/vino julia -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/1] staging: io: Remove unnecessary OOM message
It's i2o, not io, and it is on the way out of the kernel. julia On Thu, 5 Mar 2015, Quentin Lambert wrote: This patch reduces the kernel size by removing error messages that duplicate the normal OOM message. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/staging/i2o/i2o_block.c | 1 - drivers/staging/i2o/i2o_config.c | 7 +-- drivers/staging/i2o/iop.c| 10 ++ 3 files changed, 3 insertions(+), 15 deletions(-) diff --git a/drivers/staging/i2o/i2o_block.c b/drivers/staging/i2o/i2o_block.c index 0a13c64..a26e2a0 100644 --- a/drivers/staging/i2o/i2o_block.c +++ b/drivers/staging/i2o/i2o_block.c @@ -965,7 +965,6 @@ static struct i2o_block_device *i2o_block_device_alloc(void) dev = kzalloc(sizeof(*dev), GFP_KERNEL); if (!dev) { - osm_err(Insufficient memory to allocate I2O Block disk.\n); rc = -ENOMEM; goto exit; } diff --git a/drivers/staging/i2o/i2o_config.c b/drivers/staging/i2o/i2o_config.c index 5748663..29e90e2 100644 --- a/drivers/staging/i2o/i2o_config.c +++ b/drivers/staging/i2o/i2o_config.c @@ -591,11 +591,8 @@ static int i2o_cfg_passthru32(struct file *file, unsigned cmnd, rcode = -ENOMEM; reply = kzalloc(reply_size, GFP_KERNEL); - if (!reply) { - printk(KERN_WARNING %s: Could not allocate reply buffer\n, -c-name); + if (!reply) goto out; - } sg_offset = (msg-u.head[0] 4) 0x0f; @@ -833,8 +830,6 @@ static int i2o_cfg_passthru(unsigned long arg) reply = kzalloc(reply_size, GFP_KERNEL); if (!reply) { - printk(KERN_WARNING %s: Could not allocate reply buffer\n, -c-name); rcode = -ENOMEM; goto out; } diff --git a/drivers/staging/i2o/iop.c b/drivers/staging/i2o/iop.c index 52334fc..47c785c 100644 --- a/drivers/staging/i2o/iop.c +++ b/drivers/staging/i2o/iop.c @@ -99,11 +99,8 @@ u32 i2o_cntxt_list_add(struct i2o_controller * c, void *ptr) c-name); entry = kmalloc(sizeof(*entry), GFP_ATOMIC); - if (!entry) { - osm_err(%s: Could not allocate memory for context list element - \n, c-name); + if (!entry) return 0; - } entry-ptr = ptr; entry-timestamp = jiffies; @@ -1047,11 +1044,8 @@ struct i2o_controller *i2o_iop_alloc(void) char poolname[32]; c = kzalloc(sizeof(*c), GFP_KERNEL); - if (!c) { - osm_err(i2o: Insufficient memory to allocate a I2O controller. - \n); + if (!c) return ERR_PTR(-ENOMEM); - } c-unit = unit++; sprintf(c-name, iop%d, c-unit); -- 1.9.1 -- To unsubscribe from this list: send the line unsubscribe kernel-janitors in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/