linux-kernel - Re: [PATCH] [SCSI] libsas: Kconfig: Enable SATA compatibility by default

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 06 May 2015 07:23:30 -0700
From:	James Bottomley <James.Bottomley@...senPartnership.com>
To:	Ian Jackson <ian.jackson@...citrix.com>
Cc:	linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
	xen-devel@...ts.xenproject.org,
	James Bottomley <James.Bottomley@...e.de>,
	Donald D Dugger <donald.d.dugger@...el.com>,
	Pawel Baldysiak <pawel.baldysiak@...el.com>,
	Lukasz Dorau <lukasz.dorau@...el.com>,
	Artur Paszkiewicz <artur.paszkiewicz@...el.com>,
	Ian Campbell <ian.campbell@...rix.com>,
	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>,
	Boris Ostrovsky <boris.ostrovsky@...cle.com>,
	David Vrabel <david.vrabel@...rix.com>
Subject: Re: [PATCH] [SCSI] libsas: Kconfig: Enable SATA compatibility by
 default

On Fri, 2015-05-01 at 15:36 +0100, Ian Jackson wrote:
> SATA controllers support SATA disks.  The kernel should be able to
> drive these, by default.  It should not silently (apart from a
> debugging-only printk) ignore them.

To be honest, this is a bit pointless: no distribution takes the kernel
defaults anyway so changing it achieves nothing.

The original thought behind doing this is that libata was huge and is
pulled in by the sata code whether its actually used or not.  Certain
configurations might not want this.  However, I think we can all agree
that the major SAS user is enterprise and for them, kernel size isn't
usually a huge worry.  Even for enterprise, the predominant
configuration seems to be SAS expanders with SATA drives, so they can't
turn it off anyway.  Therefore a more useful question might be: is it
time to remove the option entirely?

James

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/