RE: Adding disk firmware programming capability to camcontrol

I have updated the patch according to feedbacks I received. The main modification was to warn the user and ask for confirmation before downloading a firmware image. A -y option has been added that suppresses the confirmation-- the same as the one for the format command.

The new patch is against HEAD and could be downloaded from:
http://people.freebsd.org/~emaste/patches/fwdownload.diff


Reviews/suggestions are greatly appreciated.


Nima Misaghian 
nmisaghian_at_sandvine.com


> -----Original Message-----
> From: crodr001_at_gmail.com [mailto:crodr001_at_gmail.com] On Behalf Of Craig
> Rodrigues
> Sent: Thursday, November 03, 2011 10:06 PM
> To: Nima Misaghian
> Cc: freebsd-current_at_freebsd.org
> Subject: Re: Adding disk firmware programming capability to camcontrol
> 
> On Fri, Oct 28, 2011 at 1:47 PM, Nima Misaghian
> <nmisaghian_at_sandvine.com> wrote:
> > Hi,
> >
> > I have got code developed by Andre Albsmeier that is capable of
> > programming firmware of hard drives from several vendors and  turned
> > it into a camcontrol command.
> 
> +1
> 
> I took a look at your patch and it looks great.  I have worked on a
> storage product before where there
> was a requirement to reprogram the firmware of hard drives.  On tha
> product,
> proprietary Windows tools had to be used.  It would have been nice to
> be able to use camcontrol in FreeBSD.
> 
> I think the patch is fine in its current form.  Very few "regular
> users" need to reprogram hard drive firmware.
> It is a special administrative operation.
> 
> --
> Craig Rodrigues
> rodrigc_at_crodrigues.org