Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-2800

build: clean out old autoconf options

Details

    • 3
    • 6778

    Description

      It currently takes longer to run configure on the Lustre source than it does to build it. There are a ton of configure tests that can be removed in light of the support matrix for 2.4+

      Attachments

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. LU-3080 Clean up code for ancient kernels <2.6.32

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            [LU-2800] build: clean out old autoconf options
            yujian Jian Yu added a comment -

            Patch http://review.whamcloud.com/8581 landed on Lustre b2_4 branch for 2.4.2.

            yujian Jian Yu added a comment - Patch http://review.whamcloud.com/8581 landed on Lustre b2_4 branch for 2.4.2.
            pjones Peter Jones added a comment -

            Fixed in 2.6

            pjones Peter Jones added a comment - Fixed in 2.6
            simmonsja James A Simmons added a comment -

            All patches have been landed to master. This ticket can be closed.

            simmonsja James A Simmons added a comment - All patches have been landed to master. This ticket can be closed.
            simmonsja James A Simmons added a comment -

            Only three patches to go. Please review the following patches.

            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/8102

            http://review.whamcloud.com/#/c/5380 - depends on patch http://review.whamcloud.com/#/c/7135 from LU-3319

            simmonsja James A Simmons added a comment - Only three patches to go. Please review the following patches. http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/8102 http://review.whamcloud.com/#/c/5380 - depends on patch http://review.whamcloud.com/#/c/7135 from LU-3319
            simmonsja James A Simmons added a comment -

            We are down to the last few patches. They are:

            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/5383
            http://review.whamcloud.com/#/c/5546
            http://review.whamcloud.com/#/c/8102

            http://review.whamcloud.com/#/c/5380 - depends on patch http://review.whamcloud.com/#/c/7135 for LU-3319

            simmonsja James A Simmons added a comment - We are down to the last few patches. They are: http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5383 http://review.whamcloud.com/#/c/5546 http://review.whamcloud.com/#/c/8102 http://review.whamcloud.com/#/c/5380 - depends on patch http://review.whamcloud.com/#/c/7135 for LU-3319
            simmonsja James A Simmons added a comment -

            The patches ready for inspection are:

            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/5374
            http://review.whamcloud.com/#/c/5406

            http://review.whamcloud.com/#/c/5371
            http://review.whamcloud.com/#/c/5546

            simmonsja James A Simmons added a comment - The patches ready for inspection are: http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5374 http://review.whamcloud.com/#/c/5406 http://review.whamcloud.com/#/c/5371 http://review.whamcloud.com/#/c/5546
            simmonsja James A Simmons added a comment -

            If you have not done so please review the patches below. All of them are ready to be merged. Thank you.

            http://review.whamcloud.com/#/c/5339
            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/5411
            http://review.whamcloud.com/#/c/5367
            http://review.whamcloud.com/#/c/5403

            http://review.whamcloud.com/#/c/5370
            http://review.whamcloud.com/#/c/5374
            http://review.whamcloud.com/#/c/5406

            simmonsja James A Simmons added a comment - If you have not done so please review the patches below. All of them are ready to be merged. Thank you. http://review.whamcloud.com/#/c/5339 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5411 http://review.whamcloud.com/#/c/5367 http://review.whamcloud.com/#/c/5403 http://review.whamcloud.com/#/c/5370 http://review.whamcloud.com/#/c/5374 http://review.whamcloud.com/#/c/5406
            simmonsja James A Simmons added a comment -

            Next series is ready. They will not conflict with current master or each other.

            http://review.whamcloud.com/#/c/5339
            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/5411
            http://review.whamcloud.com/#/c/5367
            http://review.whamcloud.com/#/c/5403

            The following patches are dependent on the above and have to be applied in the specific order below.

            http://review.whamcloud.com/#/c/5370
            http://review.whamcloud.com/#/c/5374
            http://review.whamcloud.com/#/c/5406

            Besides these only a few more patches will be needed after these land. The reason for not posting those patches is that they conflict with multiple patches above.

            simmonsja James A Simmons added a comment - Next series is ready. They will not conflict with current master or each other. http://review.whamcloud.com/#/c/5339 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5411 http://review.whamcloud.com/#/c/5367 http://review.whamcloud.com/#/c/5403 The following patches are dependent on the above and have to be applied in the specific order below. http://review.whamcloud.com/#/c/5370 http://review.whamcloud.com/#/c/5374 http://review.whamcloud.com/#/c/5406 Besides these only a few more patches will be needed after these land. The reason for not posting those patches is that they conflict with multiple patches above.
            simmonsja James A Simmons added a comment -

            New series ready to merge. They will not conflict with master or each other. Please review if you haven't already.

            http://review.whamcloud.com/#/c/5340
            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/5348
            http://review.whamcloud.com/#/c/5360
            http://review.whamcloud.com/#/c/5411

            simmonsja James A Simmons added a comment - New series ready to merge. They will not conflict with master or each other. Please review if you haven't already. http://review.whamcloud.com/#/c/5340 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5348 http://review.whamcloud.com/#/c/5360 http://review.whamcloud.com/#/c/5411
            simmonsja James A Simmons added a comment -

            The follow patches are the next series to merge. They will not conflict with master or each other. Please review if you haven't already.

            http://review.whamcloud.com/#/c/5338
            http://review.whamcloud.com/#/c/5340
            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/5359
            http://review.whamcloud.com/#/c/5369
            http://review.whamcloud.com/#/c/5411

            simmonsja James A Simmons added a comment - The follow patches are the next series to merge. They will not conflict with master or each other. Please review if you haven't already. http://review.whamcloud.com/#/c/5338 http://review.whamcloud.com/#/c/5340 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5359 http://review.whamcloud.com/#/c/5369 http://review.whamcloud.com/#/c/5411

            People

              bogl Bob Glossman (Inactive)
              jeffm Jeff Mahoney (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: