linux-kernel - Re: [PATCH] sysfs: add per pci device msi[x] irq listing (v4)

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: <CAErSpo4bEcubO56hZ+PWZc0SdVEroyOmsxDtc_bBAJrxBKmAOQ@mail.gmail.com>
Date:	Thu, 6 Oct 2011 11:57:12 -0600
From:	Bjorn Helgaas <bhelgaas@...gle.com>
To:	Neil Horman <nhorman@...driver.com>
Cc:	Jesse Barnes <jbarnes@...tuousgeek.org>,
	Stefan Richter <stefanr@...6.in-berlin.de>,
	linux-kernel@...r.kernel.org, Greg Kroah-Hartman <gregkh@...e.de>,
	linux-pci@...r.kernel.org
Subject: Re: [PATCH] sysfs: add per pci device msi[x] irq listing (v4)

On Thu, Oct 6, 2011 at 11:12 AM, Neil Horman <nhorman@...driver.com> wrote:
>
> On Thu, Oct 06, 2011 at 08:36:07AM -0700, Jesse Barnes wrote:
> > On Fri, 30 Sep 2011 12:54:03 -0400
> > Neil Horman <nhorman@...driver.com> wrote:
> >
> > > On Fri, Sep 30, 2011 at 10:33:36AM -0600, Bjorn Helgaas wrote:
> > > > On Fri, Sep 30, 2011 at 9:33 AM, Neil Horman <nhorman@...driver.com> wrote:
> > > > >
> > > > > On Fri, Sep 30, 2011 at 02:32:11PM +0200, Stefan Richter wrote:
> > > > > > On Sep 29 Neil Horman wrote:
> > > > > > > (v4)
> > > > > > > Fixed up some spelling mistakes, and added a scissors line with a good
> > > > > > > commitlog, so that git-am drops all the version logging
> > > > > > >
> > > > > > > Signed-off-by: Neil Horman <nhorman@...driver.com>
> > > > > > > CC: Greg Kroah-Hartman <gregkh@...e.de>
> > > > > > > CC: Jesse Barnes <jbarnes@...tuousgeek.org>
> > > > > > > CC: Bjorn Helgaas <bhelgaas@...gle.com>
> > > > > > > CC: linux-pci@...r.kernel.org
> > > > > >
> > > > > > It drops these last 5 lines then too, doesn't it?
> > > > > >
> > > > > I would have thought so, but I tested a git-format-patch / git-am operation here
> > > > > on a temporary branch, and it worked properly.
> > > > >
> > > > > > > --- a/Documentation/ABI/testing/sysfs-bus-pci
> > > > > > > +++ b/Documentation/ABI/testing/sysfs-bus-pci
> > > > > > > @@ -66,6 +66,24 @@ Description:
> > > > > > >             re-discover previously removed devices.
> > > > > > >             Depends on CONFIG_HOTPLUG.
> > > > > > >
> > > > > > > +What:              /sys/bus/pci/devices/.../msi_irqs/
> > > > > > > +Date:              September, 2011
> > > > > > > +Contact:   Neil Horman <nhorman@...driver.com>
> > > > > > > +Description:
> > > > > > > +           The /sys/devices/.../msi_irqs directory contains a variable set
> > > > > > > +           sub-directories, with each sub-directory being named after a
> > > > > > > +           corresponding msi irq vector allocated to that device.
> > > > > >
> > > > > > "contains a variable set _of_ sub-directories"?
> > > > > Gah, bad ispell foo on my part.  Bjorn, can you add that in, or shall I repost?
> > > >
> > > > I'll try to remember to add that in.
> > > >
> > > Thank you, apologies for my poor writing skills :)
> > > Neil
> >
> > Is there a final patch available for me to queue up?
> >
> I was under the impression that v4 was the version to queue up.  Stefan had
> found that I left the word of out of the documentation at one location but Bjorn
> was kind enough to offer to fix it up on commit.

I intended to set up a git tree for the merge window, but Jesse came
back from vacation, so I don't think I'll do that after all.

Can you just repost it with the typo correction for Jesse?

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