Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

On Fri, Jan 11, 2008 at 11:40:02AM -0800, Arjan van de Ven wrote:
On Fri, 11 Jan 2008 12:28:20 -0700
Matthew Wilcox <matthew@xxxxxx> wrote:

On Fri, Jan 11, 2008 at 11:02:29AM -0800, Greg KH wrote:
Can you send me a follow-on patch that documents this in
Documentation/ABI please.

Greg, if you integrate Ivan's patch, you don't need Arjan's patch.


Personally I absolutely don't agree with that.
Ivan's patch is another attempt to make MMCONFIG work somewhat better,
but does not provide the explicit opt-in that I think is required at
this point; people have tried to get MMCONFIG stable for a really long time,
and failed still upto today. At least my patience is up and this needs
to be opt-in.

I think I agree with Arjan here, Ivan's patch should also work on top of
this one, and will help out some machines.

But as he hasn't asked for it to be included in the kernel tree, that's
a moot point right now :)

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Relevant Pages

  • Re: [PATCH 5/9] Make use of permissions, returned by kobj_lookup
    ... the -fix patch for this one. ... Greg: do it in udev ... but people want to run old distros in containers ... I have basically three objections against LSM. ...
    (Linux-Kernel)
  • Re: [PATCH] Memory leak in visor.c and ftdi_sio.c
    ... Greg KH wrote: ... >>A related problem with the current implementation is that is easy to ... the patch above) of the completion handler not being called might be. ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: [stable] [patch 00/50] -stable review
    ... Thank you, Greg. ... Normally, I would wait for a patch to appear in Linus' tree before I send it to -stable, ... Arjan van de Ven reported an illegal re-usage of the fileoperations struct ... const struct dvb_device *template, void *priv, int type) ...
    (Linux-Kernel)
  • Re: [PATCH] Dont use a klist for drivers set-of-devices
    ... > Greg and Pat: ... > This patch simplifies the driver-model core by replacing the klist ... Pat, any opinions on this patch? ...
    (Linux-Kernel)
  • Re: [patch] kref shrinkage patches -- 1 of 2 -- kref shrinkage
    ... > function pointer 'release' from the kref object, ... Patch will probably talk ... greg k-h ...
    (Linux-Kernel)