linux-kernel - Re: [set5 PATCH 0/6] scsi, async: asynchronous probing rework / fixes

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAA9_cmejubhZRXkUc_-FYhgD3B0sjR8p4YO3VkgcxWvisd-RWg@mail.gmail.com>
Date:	Thu, 28 Jun 2012 15:22:34 -0700
From:	Dan Williams <dan.j.williams@...el.com>
To:	JBottomley@...allels.com
Cc:	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org
Subject: Re: [set5 PATCH 0/6] scsi, async: asynchronous probing rework / fixes

On Thu, Jun 21, 2012 at 11:46 PM, Dan Williams <dan.j.williams@...el.com> wrote:
> Set5 of 5 patchsets to update scsi, libsas, and libata in
> support of the isci driver.
>
> Commit 43a8d39d "[SCSI] fix async probe regression" found that
> async_synchronize_full() was missing async work that was scheduled to
> its own domain.  This led James to note:
>
>      "My theory is that this is an init problem: The assumption in a lot of
>       our code is that async_synchronize_full() waits for everything ... even
>       the domain specific async schedules, which isn't true."
>
> ...and this set aims to make that assumption true, but also with the
> ability to opt-out for "private" async work.
>
> The other async probe fix is in the area of unplug events that occur in
> the scsi async scanning interval.  Essentially scsi_remove_target() can
> now see semi-initialized scsi_targets that have yet to be added via
> device_add().
>
> If there are no objections I'll put these in -next.  But I expect at
> least patch1 and patch2 will need an ack from Arjan before the set shows
> up in scsi.git.
>
> ---

James,

I caught a couple conflicts with scsi.git/for-next and was able to fix
them up without rebasing on top of your tree.  I'll resend the series
after letting these sit in -next for a while, or I can prepare a tag
for you to pull with the whole pending set.

>
> Dan Williams (6):
>      async: introduce 'async_domain' type

dropped the include of async.h

>      async: make async_synchronize_full() flush all work regardless of domain
>      scsi: queue async scan work to an async_schedule domain
>      scsi: cleanup usages of scsi_complete_async_scans

dropped the changes to scsi_wait_scan.c since that file is deleted in
your for-next branch.

--
Dan
--
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/