Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1742855

Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when requested by drivers

Show all headers | View raw

On Sun, Oct 01, 2017 at 02:25:00PM +0000, Mario.Limonciello@dell.com wrote:
> > -----Original Message-----
> > From: Greg KH [mailto:gregkh@linuxfoundation.org]
> > Sent: Sunday, October 1, 2017 8:24 AM
> > To: Limonciello, Mario <Mario_Limonciello@Dell.com>
> > Cc: dvhart@infradead.org; andy.shevchenko@gmail.com; linux-
> > kernel@vger.kernel.org; platform-driver-x86@vger.kernel.org; luto@kernel.org;
> > quasisec@google.com; pali.rohar@gmail.com; rjw@rjwysocki.net;
> > mjg59@google.com; hch@lst.de
> > Subject: Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when
> > requested by drivers
> > 
> > On Sat, Sep 30, 2017 at 07:26:57PM +0000, Mario.Limonciello@dell.com wrote:
> > > I think what's Darren's most interested in is the that conceptually this is
> > > an approach you can agree with.
> > >
> > > "A WMI vendor driver binds to the WMI bus and requests the WMI bus to
> > create
> > > a character device.  The WMI bus creates a character device /dev/wmi/$driver
> > > which the WMI vendor driver will process all various file operations."
> > 
> > We've been over this before, let's see that actual implementation which
> > defines the "will process all..." implementation to see if that is
> > actually an acceptable thing.
> > 
> > thanks,
> > 
> > greg k-h
> 
> I'm working on changes still (that you'll be CC on), but I don't expect this specific 
> concept to change in the future changes:
> 
> Bus creating character device:
> https://patchwork.kernel.org/patch/9975283/

A bit hard to comment on web pages, and I would like to point out your
obvious memory leak, but it's hard to do that this way :(

> Driver requesting character device:
> https://patchwork.kernel.org/patch/9975263/

There's other issues with this patch as well, I'll wait for a real one
in my inbox to be able to actually review it :(

thanks,

greg k-h

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 4/8] platform/x86: wmi: create character devices when requested by drivers Mario Limonciello <mario.limonciello@dell.com> - 2017-09-28 06:10 +0200
  Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers Darren Hart <dvhart@infradead.org> - 2017-09-30 04:00 +0200
    Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-09-30 10:20 +0200
      RE: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers <Mario.Limonciello@dell.com> - 2017-09-30 21:30 +0200
        Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers Greg KH <gregkh@linuxfoundation.org> - 2017-10-01 15:30 +0200
          RE: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers <Mario.Limonciello@dell.com> - 2017-10-01 16:30 +0200
            Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers Greg KH <gregkh@linuxfoundation.org> - 2017-10-01 20:10 +0200
      Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers Darren Hart <dvhart@infradead.org> - 2017-10-02 03:00 +0200
        Re: [PATCH v3 4/8] platform/x86: wmi: create character devices when  requested by drivers Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-10-02 11:30 +0200

csiph-web