Re: [PATCH 0/7] Restore code to allow unpriv_sgio for hostdev SCSI generic

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 16, 2015 at 03:51:23PM -0400, John Ferlan wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1072736
> 
> This series of patches unreverts the functionality from commit id 'ce346623'
> which reverted the original functionality.
> 

What is the motivation to revert the revert?

The commit message of that commit says:

    The kernel didn't support the unprivileged SGIO for SCSI generic
    device finally, and since it's unknow whether the way to support
    unprivileged SGIO for SCSI generic device will be similar as for
    SCSI block device or not, even it's simliar (I.e. via sysfs, for
    SCSI block device, it's /sys/dev/block/8\:0/queue/unpriv_sgio,
    for example), the file name might be different, So it's better not
    guess what it should be like currently.

But 'git grep unpriv_sgio' gives me no results on linux-next,
I only see it in the source of the dowstream kernel of a certain
enterprise distro.

If unpriv_sgio is RHEL-only, maybe we should revert it from upstream
libvirt completely.

Jan

> Since pure revert caused too many conflicts and because the code has
> undergone a few changes since the prior reversion, I had to restore
> the code by rote method.  The reversion includes some refactorings to
> make the final check much easier to handle.  Of note patch 3 covers much
> of what was adjusted in the reverted patch 'qemuCheckSharedDevice'.
> Patch 4 expands the driver lock to cover the same space as the
> similar disk code - I can take the other as well and shorten the
> time the disk code has the lock. Keeping them similar is mostly a
> sanity thing.
> 

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]