[PATCH] aacraid: apply commit config for reset_devices flag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Under some conditions associated with the unclean transition to kdump,
the aacraid adapters will view the array as foreign and not export it to
prevent access and data manipulation. The solution is to submit a commit
configuration to export the devices since this is a expected behavior
when transitioning to a kdump kernel.

This patch adds the aacraid.reset_devices flag and when either this or
the global reset_devices flag is set, ensures that a commit config is
issued and extends the startup_timeout if it is set less than 5 minutes.

This attached patch is against current scsi-misc-2.6 and was unit tested
on RHEL5 and 2.6.22. This is a bugfix, so this patch needs to propagate
quickly.

ObligatoryDisclaimer: Please accept my condolences regarding Outlook's
handling of patch attachments.

Signed-off-by: Mark Salyzyn <aacraid@xxxxxxxxxxx>

Sincerely -- Mark Salyzyn

-----Original Message-----
From: Salyzyn, Mark 
Sent: Wednesday, May 16, 2007 10:41 AM
To: 'vgoyal@xxxxxxxxxx'
Subject: RE: System does not boot if booted with "reset_devices"

Vivek, could you please try the enclosed patch.

This patch adds the aacraid.reset_devices flag and when either this or
the global reset_devices flag is set, ensures that a commit config is
issued and extends the startup_timeout if it is set less than 5 minutes.
IF you are satisfied with this, I will submit it to the scsi list.

Sincerely -- Mark Salyzyn

> -----Original Message-----
> From: Vivek Goyal [mailto:vgoyal@xxxxxxxxxx] 
> Sent: Wednesday, May 16, 2007 1:31 AM
> To: Salyzyn, Mark
> Subject: Re: System does not boot if booted with "reset_devices"
> 
> 
> On Tue, May 15, 2007 at 12:51:06PM -0400, Salyzyn, Mark wrote:
> > Been tossing it around here, and it appears I am the only 
> person that
> > considers this a risk to do a forced commit config in this 
> scenario. I
> > will send you a patch later that does this (basically 
> export commit from
> > aachba.c, and set it to a value of one if we had to reset 
> the adapter
> > during the initialization, but I will have to make sure I 
> resolve all
> > symbol name overlaps etc, so not a simple task; and few 
> minutes left in
> > my day to work on this).
> > 
> 
> Thanks Mark. I am waiting for your patch. :-)
> 
> Thanks
> Vivek
> 

Attachment: aacraid_commit_reset.patch
Description: aacraid_commit_reset.patch


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux