[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 11/13] block/export: convert vhost-user-blk server to bloc
Re: [PATCH v2 11/13] block/export: convert vhost-user-blk server to block export API
Thu, 8 Oct 2020 17:02:30 +0100
On Fri, Oct 02, 2020 at 07:20:48AM +0200, Markus Armbruster wrote:
> Stefan Hajnoczi <email@example.com> writes:
> > On Wed, Sep 30, 2020 at 04:33:18PM +0200, Markus Armbruster wrote:
> >> Stefan Hajnoczi <firstname.lastname@example.org> writes:
> >> > On Wed, Sep 30, 2020 at 07:28:58AM +0200, Markus Armbruster wrote:
> >> >> Stefan Hajnoczi <email@example.com> writes:
> >> Hence my question: is it possible that address families other than unix
> >> become available here?
> >> When that happens, we have an introspection problem of the sort we
> >> common solve with a feature flag.
> >> > Accepting any SocketAddr seems reasonable to me since vhost-user
> >> > requires an address family that has file descriptor passing. Very few
> >> > address families support this feature and we don't expect to add new
> >> > ones often.
> >> Your answer appears to be "yes in theory, quite unlikely in practice".
> >> Correct?
> Keeping introspection "tight" would be nice, but since a real need for
> "tight" here seems quite unlikely, it doesn't seem to be worth the
> Perhaps this argument could be worked into the commit message. Up to
> Acked-by: Markus Armbruster <firstname.lastname@example.org>
Thanks, I will update the commit message when merging.
Description: PGP signature