From: Tejun Heo <htejun@gmail.com>
To: Marek Podmaka <marki@marki-online.net>
Cc: jgarzik@pobox.com, linux-ide@vger.kernel.org
Subject: Re: AHCI and add_disk_randomness()
Date: Mon, 20 Nov 2006 11:35:24 +0900 [thread overview]
Message-ID: <4561146C.9030203@gmail.com> (raw)
In-Reply-To: <1124564609.20061115220130@marki-online.net>
Marek Podmaka wrote:
> Hello,
>
> I have server with Intel 5000V motherboard with integrated AHCI SATA
> controller. It works well with kernel 2.6.18.2. But I have problem
> with little entropy available and I'm not sure if one of the reasons
> is that AHCI driver does not use add_disk_randomness() to contribute
> to the kernel entropy pool.
>
> I'm not very skilled on kernel "hacking"... I tried finding where
> this is called (it's defined in drivers/char/random.c and used for
> example in Comapaq SmartArray driver in drivers/block/cciss.c). For
> the SCSI part, I found it in scsi_lib.c, but I was not able to
> determine if this is actually used by ahci/libata drivers.
>
> If not, would it be possible to implement it? I tried to figure out
> where to call it by looking at cciss.c, but it seems that this is
> totally different case, at least for me. I don't know where to add
> it, because its parameter is struct gendisk *disk and didn't find it
> used anywhere in ahci or libata.
libata currently attaches as a SCSI device and SCSI midlayer invokes
add_disk_randomness() in scsi_enq_request(). So, ahci is already
contributing to entropy pool. cciss needs explicit call to the function
because it's a separate block driver and has no one above to handle it
automatically. You probably need to look for another source if you need
more.
--
tejun
prev parent reply other threads:[~2006-11-20 2:35 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-15 21:01 AHCI and add_disk_randomness() Marek Podmaka
2006-11-16 12:40 ` Jens Axboe
2006-11-20 2:35 ` Tejun Heo [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=4561146C.9030203@gmail.com \
--to=htejun@gmail.com \
--cc=jgarzik@pobox.com \
--cc=linux-ide@vger.kernel.org \
--cc=marki@marki-online.net \
/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.