[lkml]   [2005]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 1/6] PCI Express Advanced Error Reporting Driver
    long <> writes:

    I haven't read your code in detail, just a high level remark.

    > +6. Enabling AER Aware Support in PCI Express Device Driver
    > +
    > +To enable AER aware support requires a software driver to configure
    > +the AER capability structure within its device, to initialize its AER
    > +aware callback handle and to call pcie_aer_register. Sections 6.1,
    > +6.2, and 6.3 describe how to enable AER aware support in details.


    There is currently discussion underway for a generic portable PCI
    error reporting interface for drivers. This is already being worked
    on by some PPC64 and IA64 people. I don't think it would be a good idea
    to add another incompatible PCI-E specific interface.

    So I would recommend to not apply pcie_aer_register()
    and coordinate with the others working on this area on a common

    This would only impact the device driver interface; having
    a PCI Express specific interface in sysfs is probably ok.

    Otherwise we would end up with tons of ifdefs in the drivers
    supporting multiple error reporting interfaces for different platforms,
    which would be bad.

    Also in general I think the necessary callbacks should
    be part of the basic device; not provided in a separate structure.


    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 14:11    [W:0.047 / U:65.232 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site