Bug #2711

kernel i386 linux/highmem.h:34:2: error: implicit declaration of function 'PHYS_TO_DMAP'

Added by davshao 2 months ago. Updated 2 months ago.

Status:FeedbackStart date:08/14/2014
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:-
Target version:-

Description

i386 kernel compilation using latest master after

commit 82046b5c4af24ef479241453db63a2f5883c5f50
Date: Thu Aug 14 09:09:59 2014 +0200

drm/i915: Fix display corruption issues on Haswell

fails:

In file included from /usr/src/sys/dev/drm/i915/i915_gem_execbuffer.c:31:0:
/usr/src/sys/dev/drm/i915/../../../dev/drm/include/linux/highmem.h: In function 'kmap_atomic':
/usr/src/sys/dev/drm/i915/../../../dev/drm/include/linux/highmem.h:34:2: error: implicit declaration of function 'PHYS_TO_DMAP' [-Werror=implicit-function-declaration]
/usr/src/sys/dev/drm/i915/../../../dev/drm/include/linux/highmem.h:34:2: error: nested extern declaration of 'PHYS_TO_DMAP' [-Werror=nested-externs]

restore_i386_i915.diff Magnifier (4.36 KB) davshao, 08/15/2014 08:08 PM

History

#1 Updated by ftigeot 2 months ago

  • Status changed from New to Feedback

This should be fixed with 4b892e6b4435f6ff8916ac755a6014e0ca737b0d .

#2 Updated by davshao 2 months ago

The attached patch restores i915 drm functionality at least for an i386 Lenovo Ideapad with Intel 945 graphics. It also should have identical functionality for x86_64 machines. The patch has been tested for x86_64 Intel integrated graphics G45 and HD Graphics. Basically the kmap / kunmap functions for i386 only are replaced by the previously used sf_buf functions.

The patch is against

commit ffcfd82d02034a17cb4c9a62369e77682cdba2ef
Date: Fri Aug 15 10:05:21 2014 -0700

kernel - Reduce ig4 receive errors

#3 Updated by ftigeot 2 months ago

The whole point of using the kmap API is to reduce differences with Linux; I won't be reintroducing sf_buf usage.

Besides, the i386 architecture is now deprecated.
If you still want to use i386 hardware with DragonFly, you should consider moving to a stable 3.8.x release.

Also available in: Atom PDF