linux-kernel - Re: [PATCH] drm/exynos: fix double call of drm_prime_(init/destroy)_file_private

Open Source and information security mailing list archives

Date:	Thu, 6 Sep 2012 17:30:22 +0900
From:	InKi Dae <inki.dae@...sung.com>
To:	Paul Menzel <paulepanter@...rs.sourceforge.net>
Cc:	Stéphane Marchesin <marcheu@...omium.org>,
	Kyungmin Park <kyungmin.park@...sung.com>,
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/exynos: fix double call of drm_prime_(init/destroy)_file_private

Hi,

2012/9/6 Paul Menzel <paulepanter@...rs.sourceforge.net>:
> Dear Inki Dae,
>
>
> Am Donnerstag, den 06.09.2012, 11:35 +0900 schrieb InKi Dae:
>
>> 2012/9/6 Mandeep Singh Baines <msb@...omium.org>:
>> > The double invocations are incorrect but seem to be safe so I don't
>> > think this will fix any bugs.
>> >
>> > Before:
>> >
>> > [    7.639366] drm_prime_init_file ee3675d0
>> > [    7.639377] drm_prime_init_file ee3675d0
>> > [    7.639507] drm_prime_destroy_file ee3675d0
>> > [    7.639518] drm_prime_destroy_file ee3675d0
>> > [    7.639802] drm_prime_init_file ee372390
>> > [    7.639810] drm_prime_init_file ee372390
>> > [    8.473316] drm_prime_init_file ee356390
>> > [    8.473331] drm_prime_init_file ee356390
>> >
>> > After:
>> >
>> > [    6.363842] drm_prime_init_file edc2e5d0
>> > [    6.363994] drm_prime_destroy_file edc2e5d0
>> > [    6.364260] drm_prime_init_file edc2e750
>> > [    8.004837] drm_prime_init_file ee36ded0
>> >
>> > Signed-off-by: Mandeep Singh Baines <msb@...omium.org>
>> > CC: Stéphane Marchesin <marcheu@...omium.org>
>> > CC: Pawel Osciak <posciak@...gle.com>
>> > CC: Inki Dae <inki.dae@...sung.com>
>> > CC: Joonyoung Shim <jy0922.shim@...sung.com>
>> > CC: Seung-Woo Kim <sw0312.kim@...sung.com>
>> > CC: Kyungmin Park <kyungmin.park@...sung.com>
>> > CC: David Airlie <airlied@...ux.ie>
>> > CC: dri-devel@...ts.freedesktop.org
>>
>> remove all CCs
>
> I guess they were generated by some script. So they should be fine, no?
>
> Mandeep, if you put CC in here those people should be CCed in real. `git
> send-email` should take care of that but I do not see everyone in the CC
> field. Or does `git send-email` use blind carbon copy (BCC) field?
>
>> and can you send it again using text mode?
>
> At least to the list it was send in plain text mode.
>
>> your patch is messed up when I try to get patch file.
>
> Everything is fine on my side. Especially since Mandeep used `git
> send-email` which should do everything correctly.
>

your patch was encoded with 'Content-Transfer-Encoding: base64' so
please use 7bit ascii like 'Content-Transfer-Encoding: 7bit'

>> Thanks.
>> Inki Dae
>
> In your From address your name is written InKi with capital K. Which one
> is correct?
>

Inki is correct :)

Thanks.
Inki Dae

>
> Thanks,
>
> Paul
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@...ts.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives