VirtualBox

Changeset 64172 in vbox


Ignore:
Timestamp:
Oct 6, 2016 6:38:00 PM (8 years ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
111155
Message:

bugref:8614: Additions/common/VBoxVideo: make the code more self-contained: remove all trace logging from drm vboxvideo, especially as there are better ways of doing that, such as ftrace in the Linux kernel.

Location:
trunk/src/VBox/Additions/linux/drm
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Additions/linux/drm/files_vboxvideo_drv

    r64164 r64172  
    2828    ${PATH_ROOT}/include/iprt/latin1.h=>include/iprt/latin1.h \
    2929    ${PATH_ROOT}/include/iprt/list.h=>include/iprt/list.h \
    30     ${PATH_ROOT}/include/iprt/log.h=>include/iprt/log.h \
    3130    ${PATH_ROOT}/include/iprt/param.h=>include/iprt/param.h \
    3231    ${PATH_ROOT}/include/iprt/stdarg.h=>include/iprt/stdarg.h \
    … …  
    3837    ${PATH_ROOT}/include/VBox/cdefs.h=>include/VBox/cdefs.h \
    3938    ${PATH_ROOT}/include/VBox/err.h=>include/VBox/err.h \
    40     ${PATH_ROOT}/include/VBox/log.h=>include/VBox/log.h \
    4139    ${PATH_ROOT}/include/VBox/ostypes.h=>include/VBox/ostypes.h \
    4240    ${PATH_ROOT}/include/VBox/param.h=>include/VBox/param.h \

  • trunk/src/VBox/Additions/linux/drm/vbox_drv.h

    r63771 r64172  
    5151#define LOG_GROUP LOG_GROUP_DEV_VGA
    5252
    53 #include "the-linux-kernel.h"
    54 
    5553#include <VBox/VBoxVideoGuest.h>
    56 #include <VBox/log.h>
    5754
    5855#include <drm/drmP.h>

  • trunk/src/VBox/Additions/linux/drm/vbox_fb.c

    r62550 r64172  
    8989    struct drm_clip_rect rect;
    9090
    91     LogFunc(("vboxvideo: %d\n", __LINE__));
    9291    obj = fbdev->afb.obj;
    9392    bo = gem_to_vbox_bo(obj);
    … …  
    126125        fbdev->y2 = y2;
    127126        spin_unlock_irqrestore(&fbdev->dirty_lock, flags);
    128         LogFunc(("vboxvideo: %d\n", __LINE__));
    129127        return;
    130128    }
    … …  
    155153    rect.y2 = y2 + 1;
    156154    vbox_framebuffer_dirty_rectangles(&fbdev->afb.base, &rect, 1);
    157     LogFunc(("vboxvideo: %d, bo->kmap.virtual=%p, fbdev->sysram=%p, x=%d, y=%d, x2=%d, y2=%d, unmap=%RTbool\n",
    158              __LINE__, bo->kmap.virtual, fbdev->sysram, (int)x, (int)y, (int)x2, (int)y2, unmap));
    159155    if (unmap)
    160156        ttm_bo_kunmap(&bo->kmap);
    … …  
    201197{
    202198    struct vbox_fbdev *fbdev = info->par;
    203     LogFunc(("vboxvideo: %d\n", __LINE__));
    204199    sys_fillrect(info, rect);
    205200    vbox_dirty_update(fbdev, rect->dx, rect->dy, rect->width,
    … …  
    211206{
    212207    struct vbox_fbdev *fbdev = info->par;
    213     LogFunc(("vboxvideo: %d\n", __LINE__));
    214208    sys_copyarea(info, area);
    215209    vbox_dirty_update(fbdev, area->dx, area->dy, area->width,
    … …  
    221215{
    222216    struct vbox_fbdev *fbdev = info->par;
    223     LogFunc(("vboxvideo: %d\n", __LINE__));
    224217    sys_imageblit(info, image);
    225218    vbox_dirty_update(fbdev, image->dx, image->dy, image->width,
    … …  
    256249
    257250    int ret = 0;
    258     LogFunc(("vboxvideo: %d\n", __LINE__));
    259251    drm_fb_get_bpp_depth(mode_cmd->pixel_format, &depth, &bpp);
    260252
    … …  
    265257
    266258    *gobj_p = gobj;
    267     LogFunc(("vboxvideo: %d\n", __LINE__));
    268259    return ret;
    269260}
    … …  
    284275    struct drm_gem_object *gobj = NULL;
    285276    struct vbox_bo *bo = NULL;
    286     LogFunc(("vboxvideo: %d\n", __LINE__));
    287277    mode_cmd.width = sizes->surface_width;
    288278    mode_cmd.height = sizes->surface_height;
    … …  
    369359              fb->width, fb->height);
    370360
    371     LogFunc(("vboxvideo: %d\n", __LINE__));
    372361    return 0;
    373362out:
    374     LogFunc(("vboxvideo: %d\n", __LINE__));
    375363    return ret;
    376364}
    … …  
    401389    struct fb_info *info;
    402390    struct vbox_framebuffer *afb = &fbdev->afb;
    403     LogFunc(("vboxvideo: %d\n", __LINE__));
    404391    if (fbdev->helper.fbdev) {
    405392        info = fbdev->helper.fbdev;
    … …  
    421408#endif
    422409    drm_framebuffer_cleanup(&afb->base);
    423     LogFunc(("vboxvideo: %d\n", __LINE__));
    424410}
    425411
    … …  
    430416    int ret;
    431417
    432     LogFunc(("vboxvideo: %d\n", __LINE__));
    433418    fbdev = kzalloc(sizeof(struct vbox_fbdev), GFP_KERNEL);
    434419    if (!fbdev)
    … …  
    458443        goto fini;
    459444
    460     LogFunc(("vboxvideo: %d\n", __LINE__));
    461445    return 0;
    462446fini:
    … …  
    465449    kfree(fbdev);
    466450    vbox->fbdev = NULL;
    467     LogFunc(("vboxvideo: %d, ret=%d\n", __LINE__, ret));
    468451    return ret;
    469452}
    … …  
    476459        return;
    477460
    478     LogFunc(("vboxvideo: %d\n", __LINE__));
    479461    vbox_fbdev_destroy(dev, vbox->fbdev);
    480462    kfree(vbox->fbdev);
    … …  
    486468    struct vbox_private *vbox = dev->dev_private;
    487469
    488     LogFunc(("vboxvideo: %d\n", __LINE__));
    489470    if (!vbox->fbdev)
    490471        return;

  • trunk/src/VBox/Additions/linux/drm/vbox_irq.c

    r62550 r64172  
    111111        hints = &vbox->last_mode_hints[vbox_connector->vbox_crtc->crtc_id];
    112112        if (hints->magic == VBVAMODEHINT_MAGIC) {
    113             LogFunc(("vboxvideo: %d: crtc_id=%u, mode %hdx%hd(enabled:%d),%hdx%hd\n",
    114                      __LINE__, (unsigned)vbox_connector->vbox_crtc->crtc_id,
    115                      (short)hints->cx, (short)hints->cy, (int)hints->fEnabled,
    116                      (short)hints->dx, (short)hints->dy));
    117113            disconnected = !(hints->fEnabled);
    118114            crtc_id = vbox_connector->vbox_crtc->crtc_id;
    … …  
    150146                                             hotplug_work);
    151147
    152     LogFunc(("vboxvideo: %d: vbox=%p\n", __LINE__, vbox));
    153148    vbox_update_mode_hints(vbox);
    154149    drm_kms_helper_hotplug_event(vbox->dev);
    … …  
    159154    int ret;
    160155
    161     LogFunc(("vboxvideo: %d: vbox=%p\n", __LINE__, vbox));
    162156    vbox_update_mode_hints(vbox);
    163157#if LINUX_VERSION_CODE >= KERNEL_VERSION(3, 16, 0)
    … …  
    173167    INIT_WORK(&vbox->hotplug_work, vbox_hotplug_worker);
    174168    vbox->isr_installed = true;
    175     LogFunc(("vboxvideo: %d: vbox=%p\n", __LINE__, vbox));
    176169    return 0;
    177170}
    … …  
    179172void vbox_irq_fini(struct vbox_private *vbox)
    180173{
    181     LogFunc(("vboxvideo: %d: vbox=%p\n", __LINE__, vbox));
    182174    if (vbox->isr_installed) {
    183175        drm_irq_uninstall(vbox->dev);

  • trunk/src/VBox/Additions/linux/drm/vbox_main.c

    r63771 r64172  
    6060        drm_gem_object_unreference_unlocked(vbox_fb->obj);
    6161
    62     LogFunc(("vboxvideo: %d: vbox_fb=%p, vbox_fb->obj=%p\n", __LINE__,
    63              vbox_fb, vbox_fb->obj));
    6462    drm_framebuffer_cleanup(fb);
    6563    kfree(fb);
    … …  
    7270    uint32_t vram_map_offset = vbox->available_vram_size - vbox->vram_map_start;
    7371
    74     AssertLogRelReturnVoid(vbox->vbva_info != NULL);
     72    if (vbox->vbva_info == NULL) {  /* Should never happen... */
     73        printk(KERN_ERR "vboxvideo: failed to set up VBVA.\n");
     74        return;
     75    }
    7576    for (i = 0; i < vbox->num_crtcs; ++i) {
    7677        if (vbox->vbva_info[i].pVBVA == NULL) {
    77             LogFunc(("vboxvideo: enabling VBVA.\n"));
    7878            vbva = (struct VBVABUFFER *) (  ((uint8_t *)vbox->mapped_vram)
    7979                                           + vram_map_offset
    … …  
    115115    unsigned i;
    116116
    117     LogFunc(("vboxvideo: %d: fb=%p, num_rects=%u, vbox=%p\n", __LINE__, fb,
    118              num_rects, vbox));
    119117    mutex_lock(&vbox->hw_mutex);
    120118    list_for_each_entry(crtc, &fb->dev->mode_config.crtc_list, head) {
    … …  
    148146    }
    149147    mutex_unlock(&vbox->hw_mutex);
    150     LogFunc(("vboxvideo: %d\n", __LINE__));
    151148}
    152149
    … …  
    177174    int ret;
    178175
    179     LogFunc(("vboxvideo: %d: dev=%p, vbox_fb=%p, obj=%p\n", __LINE__, dev,
    180              vbox_fb, obj));
    181176    drm_helper_mode_fill_fb_struct(&vbox_fb->base, mode_cmd);
    182177    vbox_fb->obj = obj;
    … …  
    184179    if (ret) {
    185180        DRM_ERROR("framebuffer init failed %d\n", ret);
    186         LogFunc(("vboxvideo: %d\n", __LINE__));
    187181        return ret;
    188182    }
    189     LogFunc(("vboxvideo: %d\n", __LINE__));
    190183    return 0;
    191184}
    … …  
    203196    int ret;
    204197
    205     LogFunc(("vboxvideo: %d\n", __LINE__));
    206198#if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 7, 0)
    207199    obj = drm_gem_object_lookup(filp, mode_cmd->handles[0]);
    … …  
    224216        return ERR_PTR(ret);
    225217    }
    226     LogFunc(("vboxvideo: %d\n", __LINE__));
    227218    return &vbox_fb->base;
    228219}
    … …  
    245236{
    246237    unsigned i;
    247     LogFunc(("vboxvideo: %d: vbox=%p, vbox->num_crtcs=%u, vbox->vbva_info=%p\n",
    248              __LINE__, vbox, (unsigned)vbox->num_crtcs, vbox->vbva_info));
    249238    if (!vbox->vbva_info)
    250239    {
    … …  
    261250                                   vbox->available_vram_size + i * VBVA_MIN_BUFFER_SIZE,
    262251                                   VBVA_MIN_BUFFER_SIZE);
    263     LogFunc(("vboxvideo: %d: vbox->vbva_info=%p, vbox->available_vram_size=%u\n",
    264              __LINE__, vbox->vbva_info, (unsigned)vbox->available_vram_size));
    265252    return 0;
    266253}
    … …  
    360347    int ret = 0;
    361348
    362     LogFunc(("vboxvideo: %d: dev=%p\n", __LINE__, dev));
    363349    if (!VBoxHGSMIIsSupported())
    364350        return -ENODEV;
    … …  
    400386    if (ret)
    401387        goto out_free;
    402     LogFunc(("vboxvideo: %d: vbox=%p, vbox->mapped_vram=%p, vbox->full_vram_size=%u\n",
    403              __LINE__, vbox, vbox->mapped_vram, (unsigned)vbox->full_vram_size));
    404388    return 0;
    405389out_free:
    406390    vbox_driver_unload(dev);
    407     LogFunc(("vboxvideo: %d: ret=%d\n", __LINE__, ret));
    408391    return ret;
    409392}
    … …  
    413396    struct vbox_private *vbox = dev->dev_private;
    414397
    415     LogFunc(("vboxvideo: %d\n", __LINE__));
    416398    vbox_fbdev_fini(dev);
    417399    vbox_irq_fini(vbox);
    … …  
    426408    kfree(vbox);
    427409    dev->dev_private = NULL;
    428     LogFunc(("vboxvideo: %d\n", __LINE__));
    429410    return 0;
    430411}
    … …  
    454435    int ret;
    455436
    456     LogFunc(("vboxvideo: %d: dev=%p, size=%u, iskernel=%u\n", __LINE__,
    457              dev, (unsigned)size, (unsigned)iskernel));
    458437    *obj = NULL;
    459438
    … …  
    469448    }
    470449    *obj = &vboxbo->gem;
    471     LogFunc(("vboxvideo: %d: obj=%p\n", __LINE__, obj));
    472450    return 0;
    473451}
    … …  
    481459    u32 handle;
    482460
    483     LogFunc(("vboxvideo: %d: args->width=%u, args->height=%u, args->bpp=%u\n",
    484              __LINE__, (unsigned)args->width, (unsigned)args->height,
    485              (unsigned)args->bpp));
    486461    args->pitch = args->width * ((args->bpp + 7) / 8);
    487462    args->size = args->pitch * args->height;
    … …  
    498473
    499474    args->handle = handle;
    500     LogFunc(("vboxvideo: %d: args->handle=%u\n", __LINE__,
    501              (unsigned)args->handle));
    502475    return 0;
    503476}
    … …  
    508481             uint32_t handle)
    509482{
    510     LogFunc(("vboxvideo: %d: dev=%p, handle=%u\n", __LINE__, dev,
    511              (unsigned)handle));
    512483    return drm_gem_handle_delete(file, handle);
    513484}
    … …  
    521492        return;
    522493
    523     LogFunc(("vboxvideo: %d: bo=%p\n", __LINE__, bo));
    524494    tbo = &((*bo)->bo);
    525495    ttm_bo_unref(&tbo);
    … …  
    532502    struct vbox_bo *vbox_bo = gem_to_vbox_bo(obj);
    533503
    534     LogFunc(("vboxvideo: %d: vbox_bo=%p\n", __LINE__, vbox_bo));
    535504    vbox_bo_unref(&vbox_bo);
    536505}
    … …  
    555524    struct vbox_bo *bo;
    556525
    557     LogFunc(("vboxvideo: %d: dev=%p, handle=%u\n", __LINE__,
    558              dev, (unsigned)handle));
    559526    mutex_lock(&dev->struct_mutex);
    560527#if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 7, 0)
    … …  
    573540    drm_gem_object_unreference(obj);
    574541    ret = 0;
    575     LogFunc(("vboxvideo: %d: bo=%p, *offset=%llu\n", __LINE__,
    576              bo, (unsigned long long)*offset));
    577542out_unlock:
    578543    mutex_unlock(&dev->struct_mutex);

  • trunk/src/VBox/Additions/linux/drm/vbox_mode.c

    r64165 r64172  
    7575    uint16_t flags;
    7676
    77     LogFunc(("vboxvideo: %d: vbox_crtc=%p, CRTC_FB(crtc)=%p\n", __LINE__,
    78              vbox_crtc, CRTC_FB(crtc)));
    7977    vbox = crtc->dev->dev_private;
    8078    width = mode->hdisplay ? mode->hdisplay : 640;
    … …  
    107105                                pitch, width, height,
    108106                                vbox_crtc->blanked ? 0 : bpp, flags);
    109     LogFunc(("vboxvideo: %d\n", __LINE__));
    110107}
    111108
    … …  
    116113    void *p;
    117114
    118     LogFunc(("vboxvideo: %d: vbox_crtc=%p\n", __LINE__, vbox_crtc));
    119115    /* Tell the host about the view.  This design originally targeted the
    120116     * Windows XP driver architecture and assumed that each screen would have
    … …  
    140136    else
    141137        return -ENOMEM;
    142     LogFunc(("vboxvideo: %d: p=%p\n", __LINE__, p));
    143138    return 0;
    144139}
    … …  
    154149    struct vbox_private *vbox = crtc->dev->dev_private;
    155150
    156     LogFunc(("vboxvideo: %d: vbox_crtc=%p, mode=%d\n", __LINE__, vbox_crtc,
    157              mode));
    158151    switch (mode) {
    159152    case DRM_MODE_DPMS_ON:
    … …  
    169162    vbox_do_modeset(crtc, &crtc->hwmode);
    170163    mutex_unlock(&vbox->hw_mutex);
    171     LogFunc(("vboxvideo: %d\n", __LINE__));
    172164}
    173165
    … …  
    192184    u64 gpu_addr;
    193185
    194     LogFunc(("vboxvideo: %d: fb=%p, vbox_crtc=%p\n", __LINE__, fb, vbox_crtc));
    195186    /* push the previous fb to system ram */
    196187    if (!atomic && fb) {
    … …  
    235226        vbox->input_mapping_height = CRTC_FB(crtc)->height;
    236227    }
    237     LogFunc(("vboxvideo: %d: vbox_fb=%p, obj=%p, bo=%p, gpu_addr=%u\n",
    238              __LINE__, vbox_fb, obj, bo, (unsigned)gpu_addr));
    239228    return 0;
    240229}
    … …  
    243232                 struct drm_framebuffer *old_fb)
    244233{
    245     LogFunc(("vboxvideo: %d\n", __LINE__));
    246234    return vbox_crtc_do_set_base(crtc, old_fb, x, y, 0);
    247235}
    … …  
    256244    int rc = 0;
    257245
    258     LogFunc(("vboxvideo: %d: vbox=%p\n", __LINE__, vbox));
    259246    vbox_crtc_mode_set_base(crtc, x, y, old_fb);
    260247    mutex_lock(&vbox->hw_mutex);
    … …  
    267254                                vbox->input_mapping_height);
    268255    mutex_unlock(&vbox->hw_mutex);
    269     LogFunc(("vboxvideo: %d\n", __LINE__));
    270256    return rc;
    271257}
    … …  
    324310    struct vbox_crtc *vbox_crtc;
    325311
    326     LogFunc(("vboxvideo: %d\n", __LINE__));
    327312    vbox_crtc = kzalloc(sizeof(struct vbox_crtc), GFP_KERNEL);
    328313    if (!vbox_crtc)
    … …  
    333318    drm_mode_crtc_set_gamma_size(&vbox_crtc->base, 256);
    334319    drm_crtc_helper_add(&vbox_crtc->base, &vbox_crtc_helper_funcs);
    335     LogFunc(("vboxvideo: %d: crtc=%p\n", __LINE__, vbox_crtc));
    336320
    337321    return vbox_crtc;
    … …  
    340324static void vbox_encoder_destroy(struct drm_encoder *encoder)
    341325{
    342     LogFunc(("vboxvideo: %d: encoder=%p\n", __LINE__, encoder));
    343326    drm_encoder_cleanup(encoder);
    344327    kfree(encoder);
    … …  
    358341    int enc_id = connector->encoder_ids[0];
    359342
    360     LogFunc(("vboxvideo: %d: connector=%p\n", __LINE__, connector));
    361343    /* pick the encoder ids */
    362344    if (enc_id)
    363345        return drm_encoder_find(connector->dev, enc_id);
    364     LogFunc(("vboxvideo: %d\n", __LINE__));
    365346    return NULL;
    366347}
    … …  
    412393    struct vbox_encoder *vbox_encoder;
    413394
    414     LogFunc(("vboxvideo: %d: dev=%d\n", __LINE__));
    415395    vbox_encoder = kzalloc(sizeof(struct vbox_encoder), GFP_KERNEL);
    416396    if (!vbox_encoder)
    … …  
    426406
    427407    vbox_encoder->base.possible_crtcs = 1 << i;
    428     LogFunc(("vboxvideo: %d: vbox_encoder=%p\n", __LINE__, vbox_encoder));
    429408    return &vbox_encoder->base;
    430409}
    … …  
    498477    int preferred_width, preferred_height;
    499478
    500     LogFunc(("vboxvideo: %d: connector=%p\n", __LINE__, connector));
    501479    vbox_connector = to_vbox_connector(connector);
    502480    vbox = connector->dev->dev_private;
    … …  
    547525    struct vbox_connector *vbox_connector = NULL;
    548526
    549     LogFunc(("vboxvideo: %d: connector=%p\n", __LINE__, connector));
    550527    vbox_connector = to_vbox_connector(connector);
    551528#if LINUX_VERSION_CODE < KERNEL_VERSION(3, 17, 0)
    … …  
    564541
    565542    (void) force;
    566     LogFunc(("vboxvideo: %d: connector=%p\n", __LINE__, connector));
    567543    vbox_connector = to_vbox_connector(connector);
    568544    return vbox_connector->mode_hint.disconnected ?
    … …  
    576552    struct drm_display_mode *mode, *iterator;
    577553
    578     LogFunc(("vboxvideo: %d: connector=%p, max_x=%lu, max_y = %lu\n", __LINE__,
    579              connector, (unsigned long)max_x, (unsigned long)max_y));
    580554    vbox_connector = to_vbox_connector(connector);
    581555    dev = vbox_connector->base.dev;
    … …  
    608582    struct drm_connector *connector;
    609583
    610     LogFunc(("vboxvideo: %d: dev=%p, encoder=%p\n", __LINE__, dev,
    611              encoder));
    612584    vbox_connector = kzalloc(sizeof(struct vbox_connector), GFP_KERNEL);
    613585    if (!vbox_connector)
    … …  
    639611    drm_mode_connector_attach_encoder(connector, encoder);
    640612
    641     LogFunc(("vboxvideo: %d: connector=%p\n", __LINE__, connector));
    642613    return 0;
    643614}
    … …  
    650621    unsigned i;
    651622    /* vbox_cursor_init(dev); */
    652     LogFunc(("vboxvideo: %d: dev=%p\n", __LINE__, dev));
    653623    for (i = 0; i < vbox->num_crtcs; ++i)
    654624    {
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette