Re: [PATCH] enclosure: add support for enclosure services

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Kristen Carlson Accardi
Date: Wednesday, February 13, 2008 - 10:45 am

On Tue, 12 Feb 2008 13:28:15 -0600
James Bottomley <James.Bottomley@HansenPartnership.com> wrote:


I don't think I'm arguing whether or not your solution may work, what I
am arguing is really a more philosophical point.  Not "can we do it
this way", but "should we do it way".  I am of the opinion that
management belongs in userspace.  I also am of the opinion that if you
can successfully accomplish something in user space, you should.  I
also believe that even if you provide this basic interface, all system
vendors are going to provide libraries on top of that to customize it,
so you've not added much value to just a simple message passing
interface.

So, I'm happy to defer to Jeff's judgement call here - I just want to
do what's right for our customers and get an enclosure management
interface for SATA exposed, preferrably in time for the 2.6.26 merge
window.  If he prefers your design, I'll disagree, but commit to his
decision and try to get this to work for SATA. If he'd rather see
something along the lines of what I proposed, then since it is 100% self
contained in the SATA subsystem, it shouldn't impact whatever you
want to do in the SCSI subsystem.

Jeff?

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] enclosure: add support for enclosure services, James Bottomley, (Sun Feb 3, 2:40 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Sun Feb 3, 5:16 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Mon Feb 4, 5:41 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Mon Feb 4, 7:14 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Mon Feb 4, 9:37 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Tue Feb 5, 8:01 am)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Tue Feb 5, 1:29 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Tue Feb 5, 7:57 pm)
Re: [PATCH] enclosure: add support for enclosure services, Kristen Carlson Accardi, (Tue Feb 12, 11:22 am)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Tue Feb 12, 11:45 am)
Re: [PATCH] enclosure: add support for enclosure services, Kristen Carlson Accardi, (Tue Feb 12, 12:07 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Tue Feb 12, 12:28 pm)
Re: [PATCH] enclosure: add support for enclosure services, Luben Tuikov, (Tue Feb 12, 12:45 pm)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Wed Feb 13, 9:04 am)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Wed Feb 13, 9:43 am)
Re: [PATCH] enclosure: add support for enclosure services, Kristen Carlson Accardi, (Wed Feb 13, 10:45 am)
Re: [PATCH] enclosure: add support for enclosure services, James Bottomley, (Wed Feb 13, 11:17 am)