Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1570541

Re: scsi: BUG in scsi_init_io

Show all headers | View raw

On Tue, Jan 31, 2017 at 10:50:49AM +0100, Dmitry Vyukov wrote:
> On Tue, Jan 31, 2017 at 10:20 AM, Johannes Thumshirn <jthumshirn@suse.de> wrote:
> > On Tue, Jan 31, 2017 at 09:55:52AM +0100, Dmitry Vyukov wrote:

[...]

> Please-please-please, let's not use WARN for something that is not a
> kernel bug and is user-triggerable. This makes it impossible to
> automate kernel testing and requires hiring an army of people doing
> mechanical job of sorting out WARNING reports into kernel-bugs and
> non-kernel-bugs.
> If the message is absolutely necessary (while kernel does not
> generally explain every EINVAL on console), the following will do:
> 
>       if (!blk_rq_nr_phys_segments(rq)) {
>               pr_err("you are doing something wrong\n");
>               return -EINVAL;
>       }

Yes I understand that. OTOH having the WARN helps you finding the caller because
of to the stack trace. But arguably that could be accomplished with function
graph tracing as well. I'll re-send a v2 as a proper patch.

Thanks,
	Johannes

-- 
Johannes Thumshirn                                          Storage
jthumshirn@suse.de                                +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

scsi: BUG in scsi_init_io Dmitry Vyukov <dvyukov@google.com> - 2017-01-31 10:10 +0100
  Re: scsi: BUG in scsi_init_io Johannes Thumshirn <jthumshirn@suse.de> - 2017-01-31 10:30 +0100
    Re: scsi: BUG in scsi_init_io Dmitry Vyukov <dvyukov@google.com> - 2017-01-31 11:00 +0100
      Re: scsi: BUG in scsi_init_io Johannes Thumshirn <jthumshirn@suse.de> - 2017-01-31 11:00 +0100
        Re: scsi: BUG in scsi_init_io Dmitry Vyukov <dvyukov@google.com> - 2017-01-31 11:10 +0100
      Re: scsi: BUG in scsi_init_io James Bottomley <jejb@linux.vnet.ibm.com> - 2017-01-31 16:50 +0100

csiph-web