From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: Chris Boot <bootc@boo.tc>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>,
Johannes Thumshirn <Johannes.Thumshirn@wdc.com>,
Bart Van Assche <bvanassche@acm.org>,
Finn Thain <fthain@telegraphics.com.au>,
"linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
"target-devel@vger.kernel.org" <target-devel@vger.kernel.org>,
"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
"linux1394-devel@lists.sourceforge.net"
<linux1394-devel@lists.sourceforge.net>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Chuhong Yuan <hslester96@gmail.com>,
"Martin K . Petersen" <martin.petersen@oracle.com>,
Nicholas Bellinger <nab@linux-iscsi.org>,
Stefan Richter <stefanr@s5r6.in-berlin.de>
Subject: Re: [PATCH] scsi: target/sbp: remove firewire SBP target driver
Date: Tue, 16 Jun 2020 23:09:16 -0400 [thread overview]
Message-ID: <yq1366ul8o4.fsf@ca-mkp.ca.oracle.com> (raw)
In-Reply-To: <5e512185-45d1-61eb-9bec-91e9f9d53ea3@boo.tc> (Chris Boot's message of "Tue, 16 Jun 2020 18:59:58 +0100")
Chris,
> I don't especially want it to be gone, nor can I be sure there are no
> users of what is as far as I can tell a working piece of code. I can
> tell you that I never hear about it (other than the odd patch),
> whereas I do get emails out of the blue for some of my other (much
> smaller) stuff which clearly has users. I'd be just as happy for this
> to be orphaned or for nothing to happen to it.
>
> Honestly, I am totally ambivalent as to what happens to this code.
> Martin, however, clearly cares enough to have asked me to supply a
> patch to remove it.
I love it when people take ownership of old drivers. I think it's
wonderful that Finn and others are on the ball when it comes to the 5380
family. I don't care how old things are as long as they are being
actively used.
I am also very happy to keep things in the tree as long as there is a
healthy community around them. However, keeping code around is not
free. Core interfaces change frequently. Nobody enjoys having to tweak
host templates for 50 devices they have never even heard about. Also, we
now live in a reality where there is a constant barrage of build bots
and code analyzers sending mail. So the effective cost of keeping code
around in the tree is going up. I get a substantial amount of code
analysis mail about drivers nobody have touched in a decade or more.
Consequently, I am much more inclined to remove drivers than I have been
in the past. But I am also very happy to bring them back if somebody
uses them or - even better - are willing to step up and maintain them.
I don't particularly like the notion of a driver being orphaned because
all that really means is that the driver transitions from being (at
least partially) somebody else's problem to being mine and mine alone.
--
Martin K. Petersen Oracle Linux Engineering
next prev parent reply other threads:[~2020-06-17 3:10 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-13 8:37 [PATCH] scsi: target/sbp: remove SBP target driver Chris Boot
2020-06-14 0:03 ` [PATCH] scsi: target/sbp: remove firewire " Finn Thain
2020-06-14 10:34 ` Chris Boot
2020-06-14 23:28 ` Finn Thain
2020-06-15 15:00 ` Chris Boot
2020-06-16 9:42 ` Finn Thain
2020-06-16 14:08 ` Bart Van Assche
2020-06-16 14:13 ` Johannes Thumshirn
2020-06-16 15:34 ` James Bottomley
2020-06-16 17:59 ` Chris Boot
2020-06-17 3:09 ` Martin K. Petersen [this message]
2020-06-18 0:40 ` Finn Thain
2020-06-17 2:07 ` Finn Thain
2021-01-04 17:43 ` Bart Van Assche
2021-01-04 22:50 ` Finn Thain
2021-01-05 1:48 ` Bart Van Assche
2020-06-17 2:35 ` Martin K. Petersen
2020-06-17 4:21 ` Finn Thain
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=yq1366ul8o4.fsf@ca-mkp.ca.oracle.com \
--to=martin.petersen@oracle.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=Johannes.Thumshirn@wdc.com \
--cc=bootc@boo.tc \
--cc=bvanassche@acm.org \
--cc=fthain@telegraphics.com.au \
--cc=hslester96@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=linux1394-devel@lists.sourceforge.net \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=nab@linux-iscsi.org \
--cc=stefanr@s5r6.in-berlin.de \
--cc=target-devel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox