On Thu, Oct 24, 2013 at 04:47:21AM -0400, Dave Jones wrote: > A few years ago, in 4183122dbc7c489f11971c5afa8e42011bca7fa2 > this code was added to drivers/scsi/be2iscsi/be_main.c > > + if (abrt_task->sc->device->lun != abrt_task->sc->device->lun) > + continue; > > Which doesn't make a lot of sense. > What was the intent here ? Also elsewhere we have this.. 6733b39a (Jayamohan Kallickal 2009-09-05 07:36:35 +0530 1583) phys_addr.u.a64.address = 6733b39a (Jayamohan Kallickal 2009-09-05 07:36:35 +0530 1584) *((unsigned long long *)(&phys_addr.u.a64.address)); which looks like a pretty convoluted way to assign a var to itself. Dave -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
