All of lore.kernel.org
 help / color / mirror / Atom feed
From: "John Stoffel" <john@stoffel.org>
To: Boaz Harrosh <bharrosh@panasas.com>
Cc: John Stoffel <john@stoffel.org>,
	linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org,
	Jens Axboe <Jens.Axboe@oracle.com>,
	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Subject: Re: [PATCH] Make SCSI SG v4 driver enabled by default now since udev depends on it.
Date: Tue, 7 Jul 2009 11:18:48 -0400	[thread overview]
Message-ID: <19027.26456.343392.704453@stoffel.org> (raw)
In-Reply-To: <4A53644E.2090205@panasas.com>

>>>>> "Boaz" == Boaz Harrosh <bharrosh@panasas.com> writes:

Boaz> On 07/07/2009 05:50 PM, John Stoffel wrote:
>> Signed-off-by: John Stoffel <john@stoffel.org>
>> ---
>> block/Kconfig |    6 +++---
>> 1 files changed, 3 insertions(+), 3 deletions(-)
>> 
>> diff --git a/block/Kconfig b/block/Kconfig
>> index e7d1278..22bf6eb 100644
>> --- a/block/Kconfig
>> +++ b/block/Kconfig
>> @@ -46,8 +46,8 @@ config LBD
>> 
>> config BLK_DEV_BSG
>> bool "Block layer SG support v4 (EXPERIMENTAL)"

Boaz> Please also remove the (EXPERIMENTAL) from the prompt as well

Duh... sorry for this oversight.  I'll respin the patch ASAP.

>> -	depends on EXPERIMENTAL
>> -	---help---
>> +	default y
>> +	help
>> Saying Y here will enable generic SG (SCSI generic) v4 support
>> for any block device.
>> 
>> @@ -57,7 +57,7 @@ config BLK_DEV_BSG
>> protocols (e.g. Task Management Functions and SMP in Serial
>> Attached SCSI).
>> 
>> -	  If unsure, say N.
>> +	  If unsure, say Y.
>> 
>> config BLK_DEV_INTEGRITY
>> bool "Block layer data integrity support"

Boaz> Thanks I needed that too.

Boaz> BTW distro(s) have been shipping with it on for ages

I ran into this since I was trying to get Udev to properly find and
name my tape drives, and it wasn't until I finally figured out that
BSG wasn't enabled on any of my systems that I got around to getting
it working properly.

I've also put in a patch to the UDEV people to mention that this
should be turned on to make Udev work properly.

John

      reply	other threads:[~2009-07-07 15:19 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-07 14:50 [PATCH] Make SCSI SG v4 driver enabled by default now since udev depends on it John Stoffel
2009-07-07 15:05 ` Boaz Harrosh
2009-07-07 15:18   ` John Stoffel [this message]

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=19027.26456.343392.704453@stoffel.org \
    --to=john@stoffel.org \
    --cc=Jens.Axboe@oracle.com \
    --cc=bharrosh@panasas.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.