Hi, Peter,
I have a question regarding to one of your comments below.
On 3/12/18 3:01 PM, Peter Zijlstra wrote:
On Mon, Mar 12, 2018 at 01:39:56PM -0700, Song Liu wrote:
+static void stack_map_get_build_id_offset(struct bpf_map *map,
+ struct
> On Mar 12, 2018, at 3:47 PM, Song Liu wrote:
>
>
>
>> On Mar 12, 2018, at 2:31 PM, Alexei Starovoitov wrote:
>>
>> On 3/12/18 2:12 PM, Song Liu wrote:
>>>
On Mar 12, 2018, at 2:00 PM, Alexei Starovoitov wrote:
On 3/12/18
Hi Song,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on next-20180309]
[also build test WARNING on v4.16-rc5]
[cannot apply to linus/master v4.16-rc4 v4.16-rc3 v4.16-rc2]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve
> On Mar 12, 2018, at 2:31 PM, Alexei Starovoitov wrote:
>
> On 3/12/18 2:12 PM, Song Liu wrote:
>>
>>> On Mar 12, 2018, at 2:00 PM, Alexei Starovoitov wrote:
>>>
>>> On 3/12/18 1:39 PM, Song Liu wrote:
+ page = find_get_page(vma->vm_file->f_mapping, 0);
>>>
On Mon, Mar 12, 2018 at 01:39:56PM -0700, Song Liu wrote:
> +static void stack_map_get_build_id_offset(struct bpf_map *map,
> + struct stack_map_bucket *bucket,
> + u64 *ips, u32 trace_nr)
> +{
> + int i;
> +
On 3/12/18 2:12 PM, Song Liu wrote:
On Mar 12, 2018, at 2:00 PM, Alexei Starovoitov wrote:
On 3/12/18 1:39 PM, Song Liu wrote:
+ page = find_get_page(vma->vm_file->f_mapping, 0);
did you test it with config_debug_atomic_sleep ?
it should have complained...
Yeah, I
> On Mar 12, 2018, at 2:00 PM, Alexei Starovoitov wrote:
>
> On 3/12/18 1:39 PM, Song Liu wrote:
>> +page = find_get_page(vma->vm_file->f_mapping, 0);
>
> did you test it with config_debug_atomic_sleep ?
> it should have complained...
Yeah, I have CONFIG_DEBUG_ATOMIC_SLEEP=y.
On 3/12/18 1:39 PM, Song Liu wrote:
+ page = find_get_page(vma->vm_file->f_mapping, 0);
did you test it with config_debug_atomic_sleep ?
it should have complained...