public inbox for linux-next@vger.kernel.org 
 help / color / mirror / Atom feed

• * Re: [PATCH] scsi: osd_uld: fix mismerge
         [not found] <20170419174415.4029467-1-arnd@arndb.de>
  @ 2017-04-19 23:26 ` Stephen Rothwell
         [not found] ` <1492624001.2543.9.camel@sandisk.com>
    1 sibling, 0 replies; 2+ messages in thread
  From: Stephen Rothwell @ 2017-04-19 23:26 UTC (permalink / raw)
    To: Arnd Bergmann
    Cc: Boaz Harrosh, Benny Halevy, James E.J. Bottomley,
  	Martin K. Petersen, Dan Carpenter, Bart Van Assche, linux-scsi,
  	LKML, Greg KH, Linux-Next Mailing List
  
  Hi Arnd,
  
  On Wed, 19 Apr 2017 19:44:01 +0200 Arnd Bergmann <arnd@arndb•de> wrote:
  >
  > A mismerge between two branches in linux-next reintroduced a warning that was
  > previously resolved:
  > 
  > drivers/scsi/osd/osd_uld.c: In function 'osd_probe':
  > drivers/scsi/osd/osd_uld.c:457:2: error: ignoring return value of 'scsi_device_get', declared with attribute warn_unused_result [-Werror=unused-result]
  > 
  > The original fix had more complex error unrolling, but as far as I
  > can tell, this simpler version is now sufficient.
  > 
  > Fixes: c02465fa13b6 ("scsi: osd_uld: Check scsi_device_get() return value")
  > Fixes: ac1ddc584e98 ("scsi: utilize new cdev_device_add helper function")
  > Signed-off-by: Arnd Bergmann <arnd@arndb•de>
  > ---
  >  drivers/scsi/osd/osd_uld.c | 3 ++-
  >  1 file changed, 2 insertions(+), 1 deletion(-)
  > 
  > diff --git a/drivers/scsi/osd/osd_uld.c b/drivers/scsi/osd/osd_uld.c
  > index 8b9941a5687a..0e56f1eb05dc 100644
  > --- a/drivers/scsi/osd/osd_uld.c
  > +++ b/drivers/scsi/osd/osd_uld.c
  > @@ -454,7 +454,8 @@ static int osd_probe(struct device *dev)
  >  	/* hold one more reference to the scsi_device that will get released
  >  	 * in __release, in case a logout is happening while fs is mounted
  >  	 */
  > -	scsi_device_get(scsi_device);
  > +	if (scsi_device_get(scsi_device))
  > +		goto err_retract_minor;
  >  	osd_dev_init(&oud->od, scsi_device);
  >  
  >  	/* allocate a disk and set it up */
  
  I will add this as a merge fix patch for the merge of the char-misc and
  scsi trees today.  Someone needs to let Linus know when these trees are
  merged.
  
  -- 
  Cheers,
  Stephen Rothwell
  
  ^ permalink raw reply	[flat|nested] 2+ messages in thread

• [parent not found: <1492624001.2543.9.camel@sandisk.com>]

• • * Re: [PATCH] scsi: osd_uld: fix mismerge
           [not found] ` <1492624001.2543.9.camel@sandisk.com>
    @ 2017-04-19 23:29   ` Stephen Rothwell
      0 siblings, 0 replies; 2+ messages in thread
    From: Stephen Rothwell @ 2017-04-19 23:29 UTC (permalink / raw)
      To: Bart Van Assche
      Cc: arnd@arndb•de, jejb@linux•vnet.ibm.com,
    	linux-scsi@vger•kernel.org, ooo@electrozaur•com,
    	dan.carpenter@oracle•com, linux-kernel@vger•kernel.org,
    	bhalevy@primarydata•com, martin.petersen@oracle•com, Greg KH,
    	Linux-Next Mailing List
    
    Hi Bart,
    
    On Wed, 19 Apr 2017 17:46:43 +0000 Bart Van Assche <Bart.VanAssche@sandisk•com> wrote:
    >
    > On Wed, 2017-04-19 at 19:44 +0200, Arnd Bergmann wrote:
    > > Fixes: c02465fa13b6 ("scsi: osd_uld: Check scsi_device_get() return value")
    > > Fixes: ac1ddc584e98 ("scsi: utilize new cdev_device_add helper function")  
    > 
    > Both these patches are fine as far as I can tell. Shouldn't the
    > "Fixes" tag refer to the merge commit instead of to the commits
    > that have been merged?
    
    Unfortunately, the merge commit is redone every day and will be redone
    by Linus during the merge window.  This fix should be incorporated into
    the merge commit itself.  I should have done it originally, but the
    conflict was such a mess that I took the easy way out :-)
    -- 
    Cheers,
    Stephen Rothwell
    
    ^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-04-19 23:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20170419174415.4029467-1-arnd@arndb.de>
2017-04-19 23:26 ` [PATCH] scsi: osd_uld: fix mismerge Stephen Rothwell
     [not found] ` <1492624001.2543.9.camel@sandisk.com>
2017-04-19 23:29   ` Stephen Rothwell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox