All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Christoph Hellwig <hch@lst.de>
Cc: Andy Grover <agrover@redhat.com>, linux-scsi@vger.kernel.org
Subject: Re: [PATCH 2/3] scsi: Move user-shareable stuff in scsi/scsi.h to uapi/scsi/scsi.h
Date: Fri, 09 Jan 2015 09:44:11 -0800	[thread overview]
Message-ID: <1420825451.2064.12.camel@HansenPartnership.com> (raw)
In-Reply-To: <20150109171148.GA17380@lst.de>

On Fri, 2015-01-09 at 18:11 +0100, Christoph Hellwig wrote:
> On Fri, Jan 09, 2015 at 08:33:00AM -0800, James Bottomley wrote:
> > Yes, but they have to be delivered to users somehow.  If you look at
> > debian, they deliver the headers through the linux-dev-libc package
> > which installs into the /usr/include directory.
> 
> linux-libc-dev despite the name is provided from the kernel source package:
> 
> https://packages.debian.org/sid/linux-libc-dev

I know ... I actually took a look in the source to see if they'd
commented why they weren't including any SCSI files ... it's not just
the clashing files, it's also things like scsi/fc.  The reason is that
glibc is slowly incorporating those.  So first question is: do we want
to supply the linux-libc-dev package or are we happy with the glibc
route?

> > Since glibc currently
> > owns the entirety of /usr/include/scsi, we have to help the distros
> > figure out how we deliver the headers, otherwise all uapi exports of
> > SCSI stuff just gets ignored ...
> 
> That's why I've been arguing for adding a new linux/scsi_ioctl.h header
> to bypass that whole conflict since my first mail in this thread.

If we go that route, I have a marginal preference for the linux/scsi/
namespace.  We still also need a co-existence and transition plan,
because if we don't export the opcodes, glibc is going to have to supply
them in scsi/scsi.h anyway and we're going to have to not clash with
some of its extraneous definitions, like device types, status codes and
some ioctls.

James



  reply	other threads:[~2015-01-09 17:44 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-08 19:47 [PATCH 0/3] Add uapi/scsi/scsi.h Andy Grover
2015-01-08 19:47 ` [PATCH 1/3] scsi: add WRITE_VERIFY_16 to scsi.h Andy Grover
2015-01-08 19:47 ` [PATCH 2/3] scsi: Move user-shareable stuff in scsi/scsi.h to uapi/scsi/scsi.h Andy Grover
2015-01-09 10:14   ` Christoph Hellwig
2015-01-09 15:27     ` James Bottomley
2015-01-09 15:46       ` Christoph Hellwig
2015-01-09 15:50         ` James Bottomley
2015-01-09 16:01           ` Christoph Hellwig
2015-01-09 16:33             ` James Bottomley
2015-01-09 17:11               ` Christoph Hellwig
2015-01-09 17:44                 ` James Bottomley [this message]
2015-01-08 19:47 ` [PATCH 3/3] scsi: Remove scsi_ioctl.h Andy Grover
2015-01-08 20:35   ` James Bottomley
2015-01-08 21:26     ` Andy Grover
2015-01-09  9:05     ` Christoph Hellwig

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=1420825451.2064.12.camel@HansenPartnership.com \
    --to=james.bottomley@hansenpartnership.com \
    --cc=agrover@redhat.com \
    --cc=hch@lst.de \
    --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.