Hi Shyam, As per Mike (Christie)'s suggestion I am working on getting all the hardware handlers to do async pg_init(), EMC handler is getting little cumbersome. Working on it. Will try to post the set of patches by end of week. Testing and reporting the same (for as many hardware handler as possible) would certainly help these get accepted (of course after I post them :).... chandra On Sat, 2009-09-12 at 09:39 +0530, Shyam_Iyer@xxxxxxxx wrote: > > -----Original Message----- > > From: dm-devel-bounces@xxxxxxxxxx [mailto:dm-devel-bounces@xxxxxxxxxx] > > On Behalf Of Chandra Seetharaman > > Sent: Friday, May 08, 2009 6:06 AM > > To: Mike Christie > > Cc: device-mapper development; linux-scsi@xxxxxxxxxxxxxxx > > Subject: Re: [dm-devel] [PATCH 3/3] scsi_dh: rdac handler: Batch up > > MODE SELECTsand send few of them > > > > > > On Wed, 2009-05-06 at 22:39 -0500, Mike Christie wrote: > > > Chandra Seetharaman wrote: > > > > } > > > > > > > > -static int rdac_activate(struct scsi_device *sdev, > > activate_complete fn, void *data) > > > > +static int queue_mode_select(struct scsi_device *sdev, > > > > + activate_complete fn, void *data) > > > > +{ > > > > + struct rdac_queue_data *qdata; > > > > + struct rdac_controller *ctlr; > > > > + > > > > + qdata = kzalloc(sizeof(*qdata), GFP_KERNEL); > > > > > > > > > I think you want to use GFP_NOIO or GFP_ATOMIC here. If GFP_NOIO can > > > block and screw up other devices using that multipathd workqueue > > struct > > > maybe GFP_ATOMIC would be best since we can have dm-mpath retry > > later. > > > > Will do. > > > > Thanks > > > > > Chandra - Thanks for doing this. How can we help here to get this > upstream? > -- > 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 -- 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
